Skip to content

refactor: rescale/convert filters to SingleFieldFilter#226

Merged
aaron-hopkinson merged 3 commits intomainfrom
refactor/rescale_filter
Feb 18, 2026
Merged

refactor: rescale/convert filters to SingleFieldFilter#226
aaron-hopkinson merged 3 commits intomainfrom
refactor/rescale_filter

Conversation

@aaron-hopkinson
Copy link
Contributor

Description

Converts rescale and convert filters to SingleFieldFilters, changes tests to ensure non-matching fields are not processed, and breaks up the class hierarchy to avoid instantiating non-leaf class .

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

@OpheliaMiralles OpheliaMiralles left a comment

Choose a reason for hiding this comment

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

Looks like a harmless refactor

@github-project-automation github-project-automation bot moved this from To be triaged to For merging in Anemoi-dev Feb 18, 2026
@aaron-hopkinson
Copy link
Contributor Author

Thanks @OpheliaMiralles!

@aaron-hopkinson aaron-hopkinson merged commit 42ecd3b into main Feb 18, 2026
134 of 135 checks passed
@aaron-hopkinson aaron-hopkinson deleted the refactor/rescale_filter branch February 18, 2026 09:45
@github-project-automation github-project-automation bot moved this from For merging to Done in Anemoi-dev Feb 18, 2026
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