Skip to content

feat: Deprecate ai-models.json in favour of anemoi.json#247

Merged
HCookie merged 4 commits intomainfrom
fix/migrate-to-proper-name
Jan 20, 2026
Merged

feat: Deprecate ai-models.json in favour of anemoi.json#247
HCookie merged 4 commits intomainfrom
fix/migrate-to-proper-name

Conversation

@HCookie
Copy link
Member

@HCookie HCookie commented Dec 17, 2025

Description

The default metadata name was ai-models.json, this PR deprecates this in favour of anemoi.json with a warning logged to the console.
Logging was done as this should be a seemless change for users with old checkpoints working fine.

What problem does this change solve?

Removes old naming

What issue or task does this change relate to?

Closes #208

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 HCookie requested a review from anaprietonem December 17, 2025 16:07
@HCookie HCookie self-assigned this Dec 17, 2025
@github-project-automation github-project-automation bot moved this to To be triaged in Anemoi-dev Dec 17, 2025
@github-actions github-actions bot added the tests label Dec 17, 2025
@HCookie HCookie added the enhancement New feature or request label Dec 17, 2025
@HCookie HCookie moved this from To be triaged to Reviewers needed in Anemoi-dev Dec 17, 2025
@HCookie HCookie requested review from b8raoult and gmertes January 6, 2026 17:33
Copy link
Collaborator

@anaprietonem anaprietonem left a comment

Choose a reason for hiding this comment

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

Changes LGTM, thanks Harrison!

@github-project-automation github-project-automation bot moved this from Reviewers needed to For merging in Anemoi-dev Jan 7, 2026
@anaprietonem
Copy link
Collaborator

I assume this won't lead to problems in the catalogue or prepml - but maybe worth checking that with @b8raoult !

Copy link
Member

@gmertes gmertes left a comment

Choose a reason for hiding this comment

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

Good point Ana: I think the catalogue uses these functions to extract checkpoint info, so we should update anemoi-utils on the catalogue immediately at the next release. I think prepml shouldn't be affected (but we can update utils there too just in case).

@HCookie HCookie merged commit b5b1437 into main Jan 20, 2026
69 checks passed
@HCookie HCookie deleted the fix/migrate-to-proper-name branch January 20, 2026 13:53
@github-project-automation github-project-automation bot moved this from For merging to Done in Anemoi-dev Jan 20, 2026
anaprietonem pushed a commit that referenced this pull request Jan 21, 2026
🤖 Automated Release PR

This PR was created by `release-please` to prepare the next release.
Once merged:

1. A new version tag will be created
2. A GitHub release will be published
3. The changelog will be updated

Changes to be included in the next release:
---


##
[0.4.43](0.4.42...0.4.43)
(2026-01-21)


### Features

* Deprecate `ai-models.json` in favour of `anemoi.json`
([#247](#247))
([b5b1437](b5b1437))


### Bug Fixes

* Review provenance information
([#250](#250))
([0c23efe](0c23efe))

---
> [!IMPORTANT]
> Please do not change the PR title, manifest file, or any other
automatically generated content in this PR unless you understand the
implications. Changes here can break the release process.
> ⚠️ Merging this PR will:
> - Create a new release
> - Trigger deployment pipelines
> - Update package versions

 **Before merging:**
 - Ensure all tests pass
 - Review the changelog carefully
 - Get required approvals

[Release-please
documentation](https://github.com/googleapis/release-please)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

Change name from ai-models

3 participants