Skip to content

Revert "Mark x86 CI as xfail"#2622

Merged
vchuravy merged 3 commits intomainfrom
revert-2614-wsmoses-patch-4
Sep 26, 2025
Merged

Revert "Mark x86 CI as xfail"#2622
vchuravy merged 3 commits intomainfrom
revert-2614-wsmoses-patch-4

Conversation

@vchuravy
Copy link
Member

Reverts #2614

after #2619 the address-space issue should be reduced.

@codecov
Copy link

codecov bot commented Sep 25, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 75.30%. Comparing base (83984b0) to head (d0a7b43).
⚠️ Report is 6 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2622      +/-   ##
==========================================
+ Coverage   74.33%   75.30%   +0.97%     
==========================================
  Files          57       57              
  Lines       17960    17751     -209     
==========================================
+ Hits        13351    13368      +17     
+ Misses       4609     4383     -226     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

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

@vchuravy
Copy link
Member Author

Hm despite splitting the work between multiple workers we run out of memory/address-space.

@vchuravy vchuravy force-pushed the revert-2614-wsmoses-patch-4 branch from 58c9944 to a5705e2 Compare September 25, 2025 20:52
@vchuravy vchuravy mentioned this pull request Sep 25, 2025
@vchuravy
Copy link
Member Author

       From worker 2:	[ Info: Skipped RenderDocument step (doctest only).
tests                   (2) |   407.11 |   3.82 |  0.9 |    6557.15 |  1143.62 |
make_zero               (2) |   139.83 |   1.52 |  1.1 |    5559.74 |  1479.14 |
sugar                   (2) |    30.63 |   0.24 |  0.8 |     854.52 |  1543.55 |
      From worker 3:	┌ Info: Testing against
      From worker 3:	└   Enzyme_jll.libEnzyme = "/home/runner/.julia/artifacts/138277611a96d7253557af10b0b4be3d6cb52584/lib/libEnzyme-15.so"

Nice we switch workers when we get close-ish to running out of memory

@vchuravy vchuravy linked an issue Sep 25, 2025 that may be closed by this pull request
@vchuravy vchuravy force-pushed the revert-2614-wsmoses-patch-4 branch from d20bf05 to d0a7b43 Compare September 26, 2025 13:36
@vchuravy vchuravy merged commit 8da98df into main Sep 26, 2025
34 of 39 checks passed
@vchuravy vchuravy deleted the revert-2614-wsmoses-patch-4 branch September 26, 2025 13:36
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.

32bit ci fails from address oom

2 participants