Skip to content

fix: diffusion sampling#924

Merged
ssmmnn11 merged 3 commits intomainfrom
fix/diffusion_sampling
Feb 24, 2026
Merged

fix: diffusion sampling#924
ssmmnn11 merged 3 commits intomainfrom
fix/diffusion_sampling

Conversation

@ssmmnn11
Copy link
Member

@ssmmnn11 ssmmnn11 commented Feb 23, 2026

  • fix diffusion sampling after multi-dataset integration and multi out
  • sampler operations consistently in dtype
  • improve tests

@ssmmnn11 ssmmnn11 self-assigned this Feb 23, 2026
@ssmmnn11 ssmmnn11 added training ATS Approval Not Needed No approval needed by ATS labels Feb 23, 2026
@github-project-automation github-project-automation bot moved this to To be triaged in Anemoi-dev Feb 23, 2026
@ssmmnn11 ssmmnn11 changed the title Fix/diffusion sampling fix: diffusion sampling Feb 23, 2026
Copy link
Member

@JPXKQX JPXKQX 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 fixing this Simon! I think that at some point we need to update the diffusion section of our docs to show/explain how to choose these different diffusion samplers but I'm happy to do this in a later PR. Feel free to merge when the integration tests pass.

@github-project-automation github-project-automation bot moved this from To be triaged to For merging in Anemoi-dev Feb 24, 2026
@ssmmnn11 ssmmnn11 merged commit 9bd9cae into main Feb 24, 2026
13 checks passed
@ssmmnn11 ssmmnn11 deleted the fix/diffusion_sampling branch February 24, 2026 14:39
@github-project-automation github-project-automation bot moved this from For merging to Done in Anemoi-dev Feb 24, 2026
@DeployDuck DeployDuck mentioned this pull request Feb 24, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants