Skip to content

Conversation

@agnesLeroy
Copy link
Contributor

@agnesLeroy agnesLeroy commented Sep 8, 2025

closes: please link all relevant issues

PR content/description

We're getting out of memory in doc tests because we're not restraining the number of test threads used on 4090.

Check-list:

  • Tests for the changes have been added (for bug fixes / features)
  • Docs have been added / updated (for bug fixes / features)
  • Relevant issues are marked as resolved/closed, related issues are linked in the description
  • Check for breaking changes (including serialization changes) and add them to commit message following the conventional commit specification

@agnesLeroy
Copy link
Contributor Author

agnesLeroy commented Sep 8, 2025

We're still seeing out of memory in 4090 tests in ilog2, because doc tests don't have a restriction on the number of test threads.
https://github.com/zama-ai/tfhe-rs/actions/runs/17537037454

@agnesLeroy agnesLeroy force-pushed the al/reduce_test_threads_4090 branch from dae4880 to 231370b Compare September 9, 2025 06:50
@agnesLeroy agnesLeroy changed the title chore(gpu): reduce test threads for 4090 tests chore(gpu): reduce test threads for 4090 doc tests Sep 9, 2025
@agnesLeroy agnesLeroy added the 4090_test Launch test on our CI 4090 desktop label Sep 9, 2025
Copy link
Member

@IceTDrinker IceTDrinker left a comment

Choose a reason for hiding this comment

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

Thanks !

@agnesLeroy agnesLeroy merged commit daee3f1 into main Sep 10, 2025
180 checks passed
@agnesLeroy agnesLeroy deleted the al/reduce_test_threads_4090 branch September 10, 2025 08:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

4090_test Launch test on our CI 4090 desktop approved cla-signed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants