-
Notifications
You must be signed in to change notification settings - Fork 0
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
Default number of workers for multiprocessing #62
Comments
Thanks for summarizing @ieivanov . @edyoshikun I think you've been using multiprocessing on slurm the most, so I'd appreciate your comments. I just took a quick look with |
I think this now becomes a question for griznog to know what are the differences between these |
Thanks for finding this. When we run the bash script for slurm we use
I think griznog or chatgpt would be good candidates to ask. |
Also related but maybe part of different conversation, but I did find this other command
|
Sounds good, I'll trial I also recently discovered
|
We should unify the |
@talonchandler and I found that on
bruno
multiprocessing.cpu_count()
returns the total number of CPUs on the given machine, which is usually more than the user has reserved. In that sense,multiprocessing.cpu_count()
is not a good default value onbruno
(as @edyoshikun has also found). We should check if onbruno
there is an env variable that gives the number of nodes reserved and use that as default. If that variable doesn't exist (e.g. when not working onbruno
) we can fall back tomultiprocessing.cpu_count()
The text was updated successfully, but these errors were encountered: