Skip to content

Support rechunking to a frequency. - #9109

Merged
dcherian merged 12 commits into
pydata:mainfrom
dcherian:chunk-by-frequency
Jul 29, 2024
Merged

Support rechunking to a frequency.#9109
dcherian merged 12 commits into
pydata:mainfrom
dcherian:chunk-by-frequency

Conversation

@dcherian

@dcheriandcherian commented Jun 12, 2024

Copy link
Copy Markdown
Contributor

Now uses syntax ds.chunk(time=TimeResampler("YE"))

@shoyer

Copy link
Copy Markdown
Member

Are frequency strings unambiguous? Rechunking already supports memory sizes for Dask using strings.

@dcherian
dcherian marked this pull request as draft June 13, 2024 17:55
@dcherian
dcherianforce-pushed the chunk-by-frequency branch from 566fd37 to 5ffb9e4CompareJune 22, 2024 02:48
@dcherian
dcherianforce-pushed the chunk-by-frequency branch from 5ffb9e4 to e38a9b9CompareJuly 18, 2024 04:15
@dcherian
dcherian marked this pull request as ready for review July 18, 2024 07:31
@dcherian

Copy link
Copy Markdown
ContributorAuthor

Ready for review!

The syntax is ds.chunk(time=TimeResampler("YE")).

* main:
Delete ``base`` and ``loffset`` parameters to resample (pydata#9233)
Update dropna docstring (pydata#9257)
@dcherian

Copy link
Copy Markdown
ContributorAuthor

This is pretty cool, and could use a review!

@spencerkclarkspencerkclark left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks great @dcherian! My only suggestion is to make the documentation a little clearer about how to import the TimeResampler object.

Comment threaddoc/user-guide/dask.rst Outdated
Comment threaddoc/user-guide/dask.rst
Co-authored-by: Spencer Clark <spencerkclark@gmail.com>
@dcherian
dcherian merged commit af12604 into pydata:mainJul 29, 2024
@dcherian
dcherian deleted the chunk-by-frequency branch July 29, 2024 17:38
dcherian added a commit to dcherian/xarray that referenced this pull request Jul 29, 2024
* main:
automate extracting the contributors (pydata#9288)
Support rechunking to a frequency. (pydata#9109)
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Support specifying chunk sizes using labels (e.g. frequency string)

3 participants

@dcherian@shoyer@spencerkclark