-
Notifications
You must be signed in to change notification settings - Fork 13
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
docs: fix documentation to refer to anemoi datasets instead of zarr datasets #154
docs: fix documentation to refer to anemoi datasets instead of zarr datasets #154
Conversation
It may be worth renaming the |
@JPXKQX so it's that something you'd prefer to wait to fix it because it would be more breaking changes? Cause if so, at least we could have an issue opened to capture that |
I have no preference, but in agreement with Florian PR, I just realised that it is not the correct name. All anemoi datasets are zarr datasets, but not vice versa, and we only support anemoi datasets in this node builder. |
Description
Refering to Zarr outside of the anemoi-datasets package breaks the encapsulation principle.
Type of Change
Issue Number
Code Compatibility
Code Performance and Testing
Dependencies
Documentation
Additional Notes
📚 Documentation preview 📚: https://anemoi-training--154.org.readthedocs.build/en/154/
📚 Documentation preview 📚: https://anemoi-graphs--154.org.readthedocs.build/en/154/
📚 Documentation preview 📚: https://anemoi-models--154.org.readthedocs.build/en/154/