Skip to content

feat: remove single out time interpolator#915

Merged
mc4117 merged 5 commits intomainfrom
deprecation/single_out_interpolator
Feb 19, 2026
Merged

feat: remove single out time interpolator#915
mc4117 merged 5 commits intomainfrom
deprecation/single_out_interpolator

Conversation

@mc4117
Copy link
Member

@mc4117 mc4117 commented Feb 18, 2026

Description

Following discussion on issue #886, this PR deprecates the time interpolator for single outputs

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.


📚 Documentation preview 📚: https://anemoi-training--915.org.readthedocs.build/en/915/


📚 Documentation preview 📚: https://anemoi-graphs--915.org.readthedocs.build/en/915/


📚 Documentation preview 📚: https://anemoi-models--915.org.readthedocs.build/en/915/

@mc4117 mc4117 added this to the Time-Interpolation milestone Feb 18, 2026
@mc4117 mc4117 requested a review from VeraChristina February 18, 2026 15:25
@mc4117 mc4117 self-assigned this Feb 18, 2026
@mc4117 mc4117 added the ATS Approved Approved by ATS label Feb 18, 2026
@github-project-automation github-project-automation bot moved this to To be triaged in Anemoi-dev Feb 18, 2026
@mc4117 mc4117 changed the title feature/remove single out time interpolator feat:remove single out time interpolator Feb 18, 2026
@mc4117 mc4117 changed the title feat:remove single out time interpolator feat: remove single out time interpolator Feb 18, 2026
VeraChristina
VeraChristina previously approved these changes Feb 19, 2026
Copy link
Collaborator

@VeraChristina VeraChristina left a comment

Choose a reason for hiding this comment

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

This looks good to me, thanks! Just two small clarification questions for my understanding

@mc4117 mc4117 force-pushed the deprecation/single_out_interpolator branch from 6535a25 to 40f450c Compare February 19, 2026 11:07
@VeraChristina VeraChristina self-requested a review February 19, 2026 11:12
Copy link
Collaborator

@VeraChristina VeraChristina 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 answering the questions and removing the schemas! LGTM

@github-project-automation github-project-automation bot moved this from To be triaged to For merging in Anemoi-dev Feb 19, 2026
@mc4117 mc4117 merged commit 80713fb into main Feb 19, 2026
14 checks passed
@mc4117 mc4117 deleted the deprecation/single_out_interpolator branch February 19, 2026 11:38
@github-project-automation github-project-automation bot moved this from For merging to Done in Anemoi-dev Feb 19, 2026
@DeployDuck DeployDuck mentioned this pull request Feb 19, 2026
HCookie pushed a commit to ecmwf/anemoi-inference that referenced this pull request Feb 19, 2026
## Description
Deprecating `InterpolatorRunner` following the deprecation in [training](ecmwf/anemoi-core#915)
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