Skip to content

Adding constraint to zarr during install#113

Closed
fmalatino wants to merge 0 commit intoNOAA-GFDL:developfrom
fmalatino:update/submodules
Closed

Adding constraint to zarr during install#113
fmalatino wants to merge 0 commit intoNOAA-GFDL:developfrom
fmalatino:update/submodules

Conversation

@fmalatino
Copy link
Copy Markdown
Contributor

@fmalatino fmalatino commented Apr 25, 2025

Description
This PR sets a constraint for the requirement of the zarr package to < 3 in setup.py. This was per the recommendation of the NASA team who found a break in the API when zarr > 3.

🚧
Can be merged in after PR 114

How Has This Been Tested?
Tested using the current CI workflow

Checklist:

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • Any dependent changes have been merged and published in downstream modules
  • New check tests, if applicable, are included

@fmalatino fmalatino requested review from bensonr and oelbert April 25, 2025 21:11
@fmalatino fmalatino closed this Apr 28, 2025
@fmalatino fmalatino reopened this Apr 28, 2025
@fmalatino fmalatino closed this Apr 29, 2025
@fmalatino fmalatino deleted the update/submodules branch July 22, 2025 14:33
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.

1 participant