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

Update slurm requirements #98

Merged
merged 23 commits into from
Nov 9, 2023
Merged

Update slurm requirements #98

merged 23 commits into from
Nov 9, 2023

Conversation

angehung5
Copy link
Collaborator

Test run on Hopper:

State: COMPLETED (exit code 0)
Cores: 1
CPU Utilized: 01:10:19
CPU Efficiency: 92.16% of 01:16:18 core-walltime
Job Wall-clock time: 01:16:18
Memory Utilized: 236.06 GB
Memory Efficiency: 78.69% of 300.00 GB
  • python/README and run_canopy_slurm.sh updated by setting time=2:00:00 and mem=256G.
  • Example filenames and time steps modified to be consistent.
  • Output filename in global process script changed to gfs.t'+HH+'z.'+YY+MM+DD+'.sfcf'+FH+'.canopy.nc, consistent with README.

drnimbusrain and others added 22 commits November 7, 2023 11:58
@drnimbusrain
Initial commits of restructured NL and README for clarity.
f91237c
@drnimbusrain
Updated README.
5a2671d
@drnimbusrain
Update README
78a2c26
@drnimbusrain
Update README
ad359ee
@drnimbusrain
Updated README
5476d52
@drnimbusrain
Updated README
4ddb64e
@drnimbusrain
Also added python/README and example slurm script for global.
eb3b1a7
@drnimbusrain
Updated example slurm and python global for new updates.
cd974bf
@angehung5
Update README.md
2447507 
change required cpu time and mem for global script
@angehung5
Update run_canopy_slurm.sh
1cb46aa 
change required cpu time and mem
@angehung5
Update README.md
b6a2543
@angehung5
Update run_canopy_slurm.sh
8686cb4
@drnimbusrain
Updated python/README to be specific to Example 1 global processing
e7091dd
@drnimbusrain
Merge branch 'feature/nl_format' of https://github.com/noaa-oar-arl/c…
f640d2f 
…anopy-app into feature/nl_format
@drnimbusrain
Merge branch 'develop' into feature/nl_format
54096f7
@drnimbusrain
Updated python README
d4801b9
@angehung5
Update README.md
d07ef3f
@angehung5
Update output filename
b9403fb
@angehung5
Update run_canopy_slurm.sh
88c73a4
@angehung5
Update README.md
6e0b948
@angehung5
Update run_canopy_slurm.sh
d6aa07f
@angehung5
Update README.md
a8b8801
@drnimbusrain
Copy link
Member

@angehung5 Rather not a separate PR. Can you make these updates to the latest feature/nl_format branch/PR that is already there?

@drnimbusrain drnimbusrain reopened this Nov 9, 2023
@drnimbusrain
Copy link
Member

I suppose I can merge this one if it started from my latest changes on feature/nl_format

drnimbusrain
drnimbusrain requested changes Nov 9, 2023
View reviewed changes
Copy link
Member

@drnimbusrain drnimbusrain left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@angehung5 Great, thanks! Looks good, just a small change.

python/README.md Outdated Show resolved Hide resolved
@drnimbusrain drnimbusrain self-requested a review November 9, 2023 23:45
@drnimbusrain drnimbusrain merged commit 9d23a33 into develop Nov 9, 2023
@drnimbusrain drnimbusrain deleted the slurm_test branch November 9, 2023 23:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@drnimbusrain drnimbusrain drnimbusrain approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@angehung5 @drnimbusrain