Skip to content

feat: deprecate metadata commands - #336

Open
HCookie wants to merge 3 commits into
mainfrom
refactor/migrate-to-metadata
Open

feat: deprecate metadata commands#336
HCookie wants to merge 3 commits into
mainfrom
refactor/migrate-to-metadata

Conversation

@HCookie

@HCookie HCookie commented Jul 29, 2026

Copy link
Copy Markdown
Member

Description

Deprecate metadata commands in line with ecmwf/anemoi-core#1223
Some slight fixes to the metadata functions to not drop old arrays and write to the updated path in replacement

As a contributor to the Anemoi framework, please ensure that your changes include unit tests, updates to any affected dependencies and documentation, and have been tested in a parallel setting (i.e., with multiple GPUs). As a reviewer, you are also responsible for verifying these aspects and requesting changes if they are not adequately addressed. For guidelines about those please refer to https://anemoi.readthedocs.io/en/latest/

By opening this pull request, I affirm that all authors agree to the Contributor License Agreement.

HCookie added 2 commits July 29, 2026 14:32
- Use metadata package instead
- write to new path
- keep old arrays

@gmertes gmertes left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The saving/loading of the metadata json will still use the functions from here, right? The command lives in anemoi-metadata, it validates the schema and then passes the serialised schema to anemoi-utils save_metadata ?

@HCookie

HCookie commented Aug 4, 2026

Copy link
Copy Markdown
Member Author

@gmertes

The saving/loading of the metadata json will still use the functions from here, right? The command lives in anemoi-metadata, it validates the schema and then passes the serialised schema to anemoi-utils save_metadata ?

Yep, exactly correct, functions live here, validation lives there

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: To be triaged

Development

Successfully merging this pull request may close these issues.

2 participants