Skip to content

Update docs to reflect open_mfdataset default chunk behaviour #10567

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.

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

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

jemmajeffree
Copy link

Looking at #5704 and #9038 (and various other issues that reference these), it seems like there's no easy solution and the current open_mfdataset chunk default is going to stay for a while. i.e. chunks=None as a kwarg to open_mfdataset is overwritten with chunks={} before calling open_dataset, so the default chunks are disk chunks not single-file chunks.

open_kwargs = dict(engine=engine, chunks=chunks or {}, **kwargs)

In which case, we should probably update the docs to reflect this current behaviour.

Copy link

welcome bot commented Jul 25, 2025

Thank you for opening this pull request! It may take us a few days to respond here, so thank you for being patient.
If you have questions, some answers may be found in our contributing guidelines.

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

Successfully merging this pull request may close these issues.

1 participant