Skip to content

fix: single interpolator#881

Merged
mc4117 merged 1 commit intomainfrom
fix/single_interpolator
Feb 10, 2026
Merged

fix: single interpolator#881
mc4117 merged 1 commit intomainfrom
fix/single_interpolator

Conversation

@mc4117
Copy link
Member

@mc4117 mc4117 commented Feb 10, 2026

Description

This is a fix related to an issue in the single time-interpolator set-up

What problem does this change solve?

This solves a bug that the dimensions were wrong in y_pred in the single time interpolator

What issue or task does this change relate to?

#880

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.

@github-project-automation github-project-automation bot moved this from To be triaged to For merging in Anemoi-dev Feb 10, 2026
@JPXKQX
Copy link
Member

JPXKQX commented Feb 10, 2026

Isn't this tested in the integration tests?

@mc4117
Copy link
Member Author

mc4117 commented Feb 10, 2026

The problem is that it does run but the loss just explodes

@mc4117 mc4117 merged commit 4378de0 into main Feb 10, 2026
17 of 19 checks passed
@mc4117 mc4117 deleted the fix/single_interpolator branch February 10, 2026 10:55
@github-project-automation github-project-automation bot moved this from For merging to Done in Anemoi-dev Feb 10, 2026
@DeployDuck DeployDuck mentioned this pull request Feb 10, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ATS Approval Not Needed No approval needed by ATS training

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants