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

use available processor for process not all on host #349

Merged
merged 3 commits into from
Feb 14, 2023

Conversation

EricDeveaud
Copy link

Hello

this PR fix probelem mentionned in #338

some sample output of megahit in taskset environnement once applied

maestro-builder:megahit/test_data > megahit -t 12 -1 r3_1.fa  -2 r3_2.fa -o out 2>&1| grep  -e cpu -e thread 
2023-01-26 18:21:10 - Start assembly. Number of CPU threads 12 

now run with 8 core constrain (using taskset)

maestro-builder:megahit/test_data > taskset -c 1-8 megahit -t 12 -1 r3_1.fa  -2 r3_2.fa -o out2 2>&1| grep  -e cpu -e thread
2023-01-26 18:21:47 - Maximum number of available CPU thread is 8.
2023-01-26 18:21:47 - Number of thread is reset to the 8.
2023-01-26 18:21:47 - Start assembly. Number of CPU threads 8 

no diff in the results

maestro-builder:megahit/test_data > diff -q out/final.contigs.fa out2/final.contigs.fa 
maestro-builder:megahit/test_data > echo $?
0

regards

Eric

@aquaskyline aquaskyline merged commit 5f329c6 into voutcn:master Feb 14, 2023
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.

None yet

2 participants