Skip to content

some ew4 tutorial fixes#263

Merged
tennlee merged 5 commits into
ACCESS-Community-Hub:developfrom
stevehadd:ew4_tutorial_fixes
Jun 2, 2026
Merged

some ew4 tutorial fixes#263
tennlee merged 5 commits into
ACCESS-Community-Hub:developfrom
stevehadd:ew4_tutorial_fixes

Conversation

@stevehadd

Copy link
Copy Markdown
Collaborator

closes #262

I have have the nc4 and remote zarr bug, and also created a small tutorial for the data section, demonstrating loading a remote zarr using pet.

Getting Started

  • [ X] If there is not an existing issue, raise a new issue Issues with file loading for EW4 tutorial #262
  • In the issue, state that you are intending to work on a contribution. This gives everyone involved the opportunity to discuss the best way forward.

Docstrings

  • [X ] Docstrings complete and follow Napoleon (google) style

Test Coverage

  • [ X] All new code is covered by unit tests

Documentation

  • [X ] Documentation is updated as required

Final Checks:

  • [ X] If no generative AI was used, then tick this box

Alternatively, if generative AI was used, then confirm you have:

  • Attributed any generative AI (such as GitHub Copilot) that was used in this PR. See our contributing guide for more information.
  • Included the name and version of the tool or system in the pull request
  • Described the scope of that use

Finally,

  • [ X] Mark the PR as ready to review. Note - we encourage you to ask for feedback at the outset or at any time during the work.

stevehadd added 2 commits June 2, 2026 12:02
…t fesspec urls for remote zarr archives are not turned into pathlib objects (and thus become invalid)
@stevehadd stevehadd requested a review from tennlee June 2, 2026 13:55
@stevehadd stevehadd self-assigned this Jun 2, 2026
@tennlee tennlee merged commit 0fe686d into ACCESS-Community-Hub:develop Jun 2, 2026
4 checks passed
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.

Issues with file loading for EW4 tutorial

2 participants