Skip to content

fix(training): raise triton tolerence#870

Merged
anaprietonem merged 2 commits intomainfrom
fix/raise-triton-test-tolerance
Feb 9, 2026
Merged

fix(training): raise triton tolerence#870
anaprietonem merged 2 commits intomainfrom
fix/raise-triton-test-tolerance

Conversation

@cathalobrien
Copy link
Contributor

Description

sometimes triton tests fail due to random error because the tolerance is too high, this PR fixes that by raising the tolerance. 1e-4 precision is standard for fp16 computations

@github-project-automation github-project-automation bot moved this to To be triaged in Anemoi-dev Feb 9, 2026
@github-actions github-actions bot added the models label Feb 9, 2026
@cathalobrien cathalobrien added the ATS Approval Not Needed No approval needed by ATS label Feb 9, 2026
@cathalobrien cathalobrien changed the title fix(training):raise triton tolerence fix(training): raise triton tolerence Feb 9, 2026
Copy link
Member

@japols japols left a comment

Choose a reason for hiding this comment

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

LGTM

@github-project-automation github-project-automation bot moved this from To be triaged to For merging in Anemoi-dev Feb 9, 2026
@anaprietonem anaprietonem merged commit 81e913b into main Feb 9, 2026
13 checks passed
@anaprietonem anaprietonem deleted the fix/raise-triton-test-tolerance branch February 9, 2026 11:21
@github-project-automation github-project-automation bot moved this from For merging to Done in Anemoi-dev Feb 9, 2026
@DeployDuck DeployDuck mentioned this pull request Feb 9, 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 models

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants