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.

Sign up for GitHub

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

Jump to bottom

Look into deleting the --mem options from all of the slurm machines #80

Closed
malloryprow opened this issue Oct 22, 2021 · 3 comments
Closed

Look into deleting the --mem options from all of the slurm machines #80

malloryprow opened this issue Oct 22, 2021 · 3 comments
Assignees
@malloryprow
Labels
update Updates related libraries, paths, or configurations

Comments

@malloryprow
Copy link
Collaborator

After consulting with @DavidHuber-NOAA, it is advised to remove the --mem options from the Slum machines (Orion, Hera, Jet, and S4). This has been tested on S4.
@malloryprow malloryprow added the update Updates related libraries, paths, or configurations label Oct 22, 2021
@malloryprow malloryprow self-assigned this Oct 22, 2021
malloryprow added a commit that referenced this issue Oct 25, 2021
@malloryprow
Remove memory requirment for Slurm machines
6aaaf78 
Refs: #80
malloryprow added a commit that referenced this issue Oct 26, 2021
@malloryprow
Merge branch 'develop' into feature_slurm_mem
f01fe2c 
Refs: #80
@malloryprow
Copy link
Collaborator Author

@DavidHuber-NOAA I have a branch "feature_slurm_mem" that I tested successfully on Hera, Orion, and Jet with removing the "--mem" line in the job card/script. Not sure if you would like to do any further testing on S4 or not.

@DavidHuber-NOAA
Copy link
Contributor

@malloryprow No, I think it's good. Thanks!

malloryprow added a commit that referenced this issue Oct 26, 2021
@malloryprow
Merge branch 'feature_slurm_mem' into develop
fc578a7 
Refs: #80
malloryprow added a commit that referenced this issue Oct 26, 2021
@malloryprow
Merge branch 'develop' into main
80facb1 
Refs: #80
@malloryprow
Copy link
Collaborator Author

verif_global_v2.6.7 created

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@malloryprow malloryprow

Labels
update Updates related libraries, paths, or configurations
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@malloryprow @DavidHuber-NOAA