Skip to content
This repository was archived by the owner on Apr 2, 2025. It is now read-only.

Added gridded on RAW #103

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

Added gridded on RAW #103

wants to merge 2 commits into from

Conversation

thomafred
Copy link
Contributor

Changes

Note on Python version

Zarr depends on Python 3.10 or newer, however our SDK needs to support Python 3.9 or newer. As a result, zarr is an optional dependency for the odp-utils-package. If the zarr-package is not installed, gridded functionality will be disabled and the user will get a warning before the import error if they attempt to use it:

>>> from odp.utils.gridded import OdpStoreV3
/home/tom/project/odf/odp-sdk-python/src/utils/odp/utils/gridded/__init__.py:20: UserWarning: Zarr not installed. Gridded support not enabled
  warnings.warn("Zarr not installed. Gridded support not enabled")
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
ImportError: cannot import name 'OdpStoreV3' from 'odp.utils.gridded' (/home/tom/project/odf/odp-sdk-python/src/utils/odp/utils/gridded/__init__.py)

@thomafred thomafred added the enhancement New feature or request label Sep 18, 2024
@thomafred thomafred force-pushed the feature/gridded-on-raw branch from 066513b to 38c637a Compare September 24, 2024 08:11
@thomafred
Copy link
Contributor Author

Rebased with master

@thomafred thomafred marked this pull request as ready for review October 1, 2024 11:11
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant