Skip to content

Conversation

@ricardoV94
Copy link
Member

@ricardoV94 ricardoV94 commented Nov 18, 2025

  • Always make the uint max test fail
  • Pin MLX version in CI

@ricardoV94 ricardoV94 force-pushed the fix_flaky_cis branch 2 times, most recently from 4964756 to fabfad1 Compare November 18, 2025 11:26
@ricardoV94 ricardoV94 force-pushed the fix_flaky_cis branch 2 times, most recently from d1ae8ec to d00db75 Compare November 18, 2025 11:50
@ricardoV94
Copy link
Member Author

Many things seem broken with MLX, I suspect it's related to their recent release? https://github.com/ml-explore/mlx/releases/tag/v0.29.4

@codecov
Copy link

codecov bot commented Nov 18, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 81.71%. Comparing base (8617558) to head (97101c4).
⚠️ Report is 30 commits behind head on main.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff            @@
##             main    #1729    +/-   ##
========================================
  Coverage   81.70%   81.71%            
========================================
  Files         246      251     +5     
  Lines       53632    54107   +475     
  Branches     9438     9468    +30     
========================================
+ Hits        43822    44212   +390     
- Misses       7329     7408    +79     
- Partials     2481     2487     +6     

see 4 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@jessegrabowski
Copy link
Member

Was the issue with pytorch tests resolved?

@jessegrabowski jessegrabowski merged commit 4e4f237 into pymc-devs:main Nov 18, 2025
57 checks passed
@ricardoV94
Copy link
Member Author

Was the issue with pytorch tests resolved?

Apparently. Testing the pin thing in #1720

@ricardoV94 ricardoV94 deleted the fix_flaky_cis branch November 18, 2025 13:01
Copy link
Contributor

@cetagostini cetagostini left a comment

Choose a reason for hiding this comment

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

Good 👍🏻 I was thinking in the same direction but didn't went to last release, I closed my PR. We can merge this

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants