-
Notifications
You must be signed in to change notification settings - Fork 0
pid needed #11
Copy link
Copy link
Open
Labels
non-priorityThis doesn't seem rightThis doesn't seem right
Description
Line 122 in 3a4b7ed
| if job in list(jobs.keys()): |
if True, this will cancel all but the oldest job for job (the smallest slurm_job_id). However, this if statement needs to return the correct pid (sorted(pids)[0]), which handles: if len(pids) >= 1, but not when len(pids)==0 where it will cause IndexError.
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
non-priorityThis doesn't seem rightThis doesn't seem right