Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@SimonKamuk
Copy link
Contributor

Description

The PR clarifies the error message given when a group does not have the expected number of parameters. Previously the error message would give the metadata of all groups instead of just the one causing the error.

What problem does this change solve?

bugfix

What issue or task does this change relate to?

Additional notes

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.

Copy link
Contributor

@aaron-hopkinson aaron-hopkinson left a comment

Choose a reason for hiding this comment

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

Thanks for the change. LGTM!

@aaron-hopkinson aaron-hopkinson merged commit 1cd0536 into ecmwf:main Oct 30, 2025
16 of 19 checks passed
@github-project-automation github-project-automation bot moved this from To be triaged to Done in Anemoi-dev Oct 30, 2025
aaron-hopkinson pushed a commit that referenced this pull request Oct 31, 2025
🤖 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.1.18](0.1.17...0.1.18)
(2025-10-30)


### Features

* Add AccumToInterval filter transform
([#168](#168))
([0801a4b](0801a4b))
* Generic cos sin from rad filter
([#180](#180))
([7fabff9](7fabff9))


### Bug Fixes

* Clarify error message
([#181](#181))
([1cd0536](1cd0536))
* Remove variable from xarray metadata when grouping
([#182](#182))
([46465cd](46465cd))

---
> [!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.

2 participants