-
-
Notifications
You must be signed in to change notification settings - Fork 136
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
Documentation about memory
vs 'job_mem` could be improved
#634
Comments
memory
keyword argumentmemory
vs 'job_mem` could be improved
You are right, this is the
|
I agree with the improvements mentioned here. Something like this:
|
@guillaumeeb Would like to know your thoughts on the above comment regarding description of CPU parameters. |
Thanks @maneesh29s for the proposal and sorry for the delay. Just a small suggestion: |
I agree with your suggestion. And I am happy to raise a PR to close this issue. |
Sure, please do so! |
The SLRUMCluster documentation's description for the
memory
keyword argument is a bit ambiguous when comparing againstjob_mem
.We have:
memory
: Total amount of memory per jobjob_mem
: Amount of memory to request in SLURM. If None, defaults to worker processes * memoryThe documentation states that a "job" is a Slurm allocation. So, this leads us to some potentially confusing docs.
The text was updated successfully, but these errors were encountered: