-
Notifications
You must be signed in to change notification settings - Fork 120
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 of executor.update_parameters
arguments
#1735
Comments
JoeZiminski
changed the title
Documentation of 1executor.update_parameters` arguments
Documentation of May 5, 2023
executor.update_parameters
arguments
I agree very much, although I feel that the keywords here are the keyword list of slurm([https://slurm.schedmd.com/sbatch.html]), but I often encounter warnings that I am not using the keywords properly, and I don't know if it is triggered by slurm or submitit. I hope the submitit devs can add a little clarification in the README. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Thanks for making this excellent tool! It works basically out of the box and has proved extremely useful.
Apologies if I am missing something obvious, but it would be great to have a full list of the arguments
executor.update_parameters(timeout_min=1, slurm_partition="dev")
accepts. I always end up flailing about looking through examples and source code until I find the name of a parameter I want to pass / if it is supported. Maybe there is a better way I'm missing!The text was updated successfully, but these errors were encountered: