Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[auto-merge] branch-24.10 to branch-24.12 [skip ci] [bot] #766

Merged
merged 1 commit into from
Oct 18, 2024

Conversation

nvauto
Copy link
Collaborator

@nvauto nvauto commented Oct 18, 2024

auto-merge triggered by github actions on branch-24.10 to create a PR keeping branch-24.12 up-to-date. If this PR is unable to be merged due to conflicts, it will remain open until manually fix.

* parallelize pytest

* increase pytest -n 4 to -n 8 per ci uses local[gpu_number] as spark context

* -n 8 does not improve over -n 4, revised to -n 3

* -n 3 seems 3 mins faster, revised to -n 2

* add dynamic pytest parallelism calculation on machine hardware

* revise from using max gpu memory to minimum gpu memory

* revise to use minimum GPU memory to determine parallelism
handle corner case minimal 1 pytest worker
get CUDA_VISIBLE_DEVICES working

* revise back to 3

* add back automatic parallelism with removed memory parallel

* revise typo not using TEST_PARALLEL

---------

Signed-off-by: Jinfeng <[email protected]>
@nvauto nvauto merged commit 09e3b2f into branch-24.12 Oct 18, 2024
@nvauto
Copy link
Collaborator Author

nvauto commented Oct 18, 2024

SUCCESS - auto-merge

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