Skip to content

tutorial hierarchical zarr stores#361

Open
eni-awowale wants to merge 18 commits into
xarray-contrib:mainfrom
eni-awowale:fundamentals-rio-backend
Open

tutorial hierarchical zarr stores#361
eni-awowale wants to merge 18 commits into
xarray-contrib:mainfrom
eni-awowale:fundamentals-rio-backend

Conversation

@eni-awowale

@eni-awowale eni-awowale commented Jul 8, 2026

Copy link
Copy Markdown
Contributor

@review-notebook-app

Copy link
Copy Markdown

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@github-actions

github-actions Bot commented Jul 8, 2026

Copy link
Copy Markdown

✅ Preview is ready!

PR preview ✅ Ready ✅ Ready
🔗 Preview https://xarray-contrib-xarray-tutorial-preview-pr-361.surge.sh
📝 Commitf7269b6
⏱️ Build time0.009s
📦 Size38.8 MB (+24.3 KB ⬆️) · 475 files
🪵 LogsView logs
📱 MobileScan to open preview on mobile

↩️ Previous: ⚡️ f7269b6 · xarray-contrib-xarray-tutorial-preview-pr-361.surge.sh (open ↗) · 2026-07-11 23:57:47 UTC

🤖 Powered by surge-preview

@eni-awowale eni-awowale marked this pull request as ready for review July 9, 2026 20:19
Comment thread intermediate/hierarchical_zarr_store.ipynb
Comment thread intermediate/hierarchical_zarr_store.ipynb
Comment thread intermediate/hierarchical_zarr_store.ipynb
Comment thread intermediate/hierarchical_zarr_store.ipynb
Comment thread intermediate/hierarchical_zarr_store.ipynb
@eni-awowale

eni-awowale commented Jul 10, 2026

Copy link
Copy Markdown
Contributor Author

Thanks for the review @keewis! Page is rendered here

@ianhi

ianhi commented Jul 11, 2026

Copy link
Copy Markdown
Contributor

THis is overall very nice. One thought:

I felt a little lost jumping into the datatree as it is. In the tutorial itself we may have already introduced the datastructure. But for someone reading this page as a start I think it would be super helpful to have a bit of a problem statement at the start + description of the dataset:

THis is the dataset + this is why we need a datatree to support it

It might also be nice to state a bit more concretely that the data is not on their computer! but in the cloud and we only download what we actually need

Sign up for free to 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.

3 participants