TUM-DAML / seml

SEML: Slurm Experiment Management Library
Other
168 stars 30 forks source link

Always use environment variables from add time + minor changes #140

Closed n-gao closed 2 months ago

n-gao commented 2 months ago

Functionality change:

Logging changes:

Performance changes:

n-gao commented 2 months ago

@dfuchsgruber, the last commit is an important fix. Seml would call scancel -t PENDING [job_id, ...] if you call seml cancel. However, if the collection is empty, it would just call scancel -t PENDING resulting in killing all pending jobs of a user. In case the user is a slurm admin, it will kill all pending jobs on the cluster.

We should merge this asap and make a new release!