Skip to content

[Feat] Improve CI#395

Merged
puneetmatharu merged 1 commit into
ARM-software:mainfrom
Radu2k:improve-ci
Oct 30, 2025
Merged

[Feat] Improve CI#395
puneetmatharu merged 1 commit into
ARM-software:mainfrom
Radu2k:improve-ci

Conversation

@Radu2k
Copy link
Copy Markdown
Contributor

@Radu2k Radu2k commented Oct 27, 2025

  • Enable c8g instance type to enhance testing
  • Filter out fp16 and bf16 pytorch tests when ONEDNN_DEFAULT_FPMATH_MODE="BF16" to mitigate accuracy failures
  • Add TORCH_RELEASE_ID as part of build wheel to match PEP440 standard

Comment thread .github/workflows/pytorch.yml Outdated
@Radu2k Radu2k force-pushed the improve-ci branch 2 times, most recently from 8a03a5e to 21eca18 Compare October 27, 2025 15:19
Copy link
Copy Markdown
Contributor

@puneetmatharu puneetmatharu left a comment

Choose a reason for hiding this comment

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

I really like this change but we just need to be careful with artifact sizes and the retention period.

Comment thread .github/workflows/pytorch.yml Outdated
puneetmatharu
puneetmatharu previously approved these changes Oct 28, 2025
Copy link
Copy Markdown
Contributor

@puneetmatharu puneetmatharu left a comment

Choose a reason for hiding this comment

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

Looks great. Have popped an optional change below

Comment thread .github/workflows/pytorch.yml Outdated
- Enable c8g instance type to enhance testing
- Add builds as artifacts to re-use in testing stage
- Filter out fp16 and bf16 pytorch tests when
  ONEDNN_DEFAULT_FPMATH_MODE="BF16" to mitigate accuracy failures
- Add TORCH_RELEASE_ID as part of build wheel to match PEP440 standard
@puneetmatharu puneetmatharu merged commit 5909a5a into ARM-software:main Oct 30, 2025
9 of 13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants