New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

copy the data of the `DatetimeIndex` in the `Dataset.chunk`-by-frequency tests #9419

Merged

dcherian merged 1 commit into pydata:main from keewis:chunk-frequency-read-only

Sep 3, 2024

Collaborator

keewis commented Sep 1, 2024 •

edited

Loading

Looks like pd.DatetimeIndex.to_numpy will return a immutable view by default, so we have to specify that we want a copy instead.

towards ⚠️ Nightly upstream-dev CI failed ⚠️ #9277

(Note: this, together with #9403, allows us to close the upstream-dev issue #9277)


          make sure to copy the data of the DatetimeIndex

b8d7ddc

keewis requested a review from dcherian

September 1, 2024 16:23

keewis added the run-upstream label

dcherian merged commit a539b72 into pydata:main

31 of 34 checks passed

keewis deleted the chunk-frequency-read-only branch

September 3, 2024 11:24

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels