Skip to content
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

Fixed "Error with SLURM #26" #30

Merged
merged 3 commits into from
Apr 20, 2016
Merged

Fixed "Error with SLURM #26" #30

merged 3 commits into from
Apr 20, 2016

Conversation

mznkcyr
Copy link
Contributor

@mznkcyr mznkcyr commented Oct 21, 2015

Hi,

I was working on a cluster machine with SLURM and I encountered the same error as below.
#26

If a node list assigned by SLURM contains nonconsecutive ones, error occurs at parseSLURM() in scoop/utils.py because 'bmin,bmax = rng.split('-')' does not work. For example, when a node list is 'host[001,003-004,006]', '001' and '006' cannot be parsed. Thus I simply fixed this.

Best regards,

@soravux soravux merged commit 269bea1 into soravux:master Apr 20, 2016
@soravux
Copy link
Owner

soravux commented Apr 20, 2016

Thanks for your pull request!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants