Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

flow converts memory directive to float #482

Closed
atravitz opened this issue Mar 22, 2021 · 0 comments · Fixed by #484
Closed

flow converts memory directive to float #482

atravitz opened this issue Mar 22, 2021 · 0 comments · Fixed by #484
Labels
bug Something isn't working directives Operation directives
Milestone

Comments

@atravitz
Copy link
Collaborator

Description

flow passes the memory directive to SLURM as a float, which at least for greatlakes is invalid.

To reproduce

Passing memory=8 results in a SLURM script containing --mem=8.0G

Error output

sbatch: error: Invalid --mem specification

System configuration

Please complete the following information:

  • Operating System [e.g. macOS]:
  • Version of Python [e.g. 3.7]:
  • Version of signac [e.g. 1.0]:
  • Version of signac-flow: 0.13.0

Or copy & paste the output of: python -c 'import platform; print(platform.platform()); import sys; print(sys.version); import signac; print(signac.__version__); import flow; print(flow.__version__)'

@csadorf csadorf added bug Something isn't working directives Operation directives labels Mar 25, 2021
@b-butler b-butler added this to the v0.14.0 milestone Mar 30, 2021
@bdice bdice closed this as completed in #484 Apr 2, 2021
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
bug Something isn't working directives Operation directives
Projects
None yet
3 participants