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

Hard coded OMP threads? #30

Open
johannesgeibel opened this issue Feb 24, 2022 · 2 comments
Open

Hard coded OMP threads? #30

johannesgeibel opened this issue Feb 24, 2022 · 2 comments

Comments

@johannesgeibel
Copy link

Hi,
I need to run winnowmap for several samples in parallel. For this I limited the number of threads to 10 via the -t parameter to avoid interference between the jobs. However, each job then used up to 30 threads. So I went through the log files and realized that -t obviously only affects pthreads, but omp_threads are additionally set to 3:

[M::main] Version: 2.03, pthreads=10, omp_threads=3

I tried to limit this by explicitely exporting OMP_NUM_THREADS=1, but it did not have an effect. So I searched this repository for omp and found a hard coded part in /src/minimap.h:

image

Is there any way to get around this hard coding? If not, is it possible to mention in the README that winnowmap uses three times more threads than expected?

Thanks,
Johannes

@Jesson-mark
Copy link

Jesson-mark commented May 21, 2022

Agree with @johannesgeibel. I have encountered the same questions. My aligning programs by winnowmap always run computing nodes too busy.

Is there any better way to change this situation?

By the way, what is the default pthreads of -t? Is it possible to write the default pthreads in software usage?

Thanks,
Jesson-mark

@CanWood
Copy link

CanWood commented Mar 29, 2023

Hi folks,

I'm chiming in as an HPC admin. Our users are keen to use winnowmap on our infrastructure but are finding that they can not reliably limit their CPU use to work within their requested resources. Any movement on this issue would be greatly appreciated.

Thanks!

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

No branches or pull requests

3 participants