Get Slurm options in a separate function

E3SM-Project / polaris

Testing and analysis for OMEGA, MPAS-Ocean, MALI and MPAS-Seaice

BSD 3-Clause "New" or "Revised" License

6 stars 13 forks source link

Get Slurm options in a separate function #83

Closed altheaden closed 1 year ago

altheaden commented 1 year ago

I made this change so that we can access the Slurm options without actually writing a job script, which may be useful for task parallelism.

Checklist

[x] API documentation in the Developer's Guide (api.md) has any new or modified class, method and/or functions listed
[x] Testing comment in the PR documents testing used to verify the changes

altheaden commented 1 year ago

This is a draft/in progress because I am currently testing, but I wanted to put together this PR in the meantime. I am setting up the PR suite that I will run against a baseline, but @xylar please let me know if you think this will require additional tests.

altheaden commented 1 year ago

Also @xylar please take a look at the changes in api.md, I copied what I'd seen for the other modules but I want you to make sure it's set up the way you would like.

altheaden commented 1 year ago

I just tested this with the PR suite against a baseline, and it passed.

xylar commented 1 year ago

Yep, the pr suite ran fine for me, too.