这是indexloc提供的服务,不要输入任何密码
Skip to content

Conversation

@eustas
Copy link
Contributor

@eustas eustas commented Nov 6, 2025

No description provided.

@eustas eustas force-pushed the test_Avx3 branch 6 times, most recently from 4570d05 to 31446f7 Compare November 6, 2025 10:45
@eustas eustas added the CI:full Label to attach to a PR to run the full CI workflow and not just the regular PR workflows label Nov 6, 2025
@eustas eustas force-pushed the test_Avx3 branch 10 times, most recently from b0a7c2b to 7548033 Compare November 7, 2025 10:38
@jonsneyers
Copy link
Member

what's tools/scripts/test_cost-x64.zip ?

@eustas
Copy link
Contributor Author

eustas commented Nov 7, 2025

After tests complete it contains how long each test run. Before running tests, if CMake have that on hands, it runs "heavy" tests first. On multicore systems that ensures faster test completion (not waiting for long running test being executed last and using just one core). In our case it helps shard balancing (both avoiding long tests being executed at the end, but also divide test set evenly).

Currently I'm running pipeline in "upload costs" mode. It uses only 1 shard, but produces costs files at the end. When it is done, I will switch back to normal mode and update costs files. The costs in current change-set is a stub made on my PC (without SDE emulation, so only loosely relevant to real pipeline costs).

@eustas
Copy link
Contributor Author

eustas commented Nov 7, 2025

(5613/8168 tests completed @ 3h11m; hopefully won't time-out; if we are lucky each of 8 shards should complete in ~30min)

Drive-by: update test-costs; clean-ups
@eustas eustas added this pull request to the merge queue Nov 10, 2025
Merged via the queue into libjxl:main with commit 3d095d9 Nov 10, 2025
120 of 132 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CI:full Label to attach to a PR to run the full CI workflow and not just the regular PR workflows

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants