ESCOMP / ESCOMP/CTSM

A few improvements to gen_mksurfdata_jobscript_single/multi

Open
#2,446 0 comments 0 reactions 0 assignees View on GitHub
code health enhancement priority: low test: mksurfdata
Dominant language
Fortran
Stars
352
Forks
361
Avg merge
6d 6h
Merged PRs (30d)
8

Description

There are a few improvements to the gen_mksurfdata_jobscript_single/gen_mksurfdata_jobscript_mult scripts that would be helpful either for clarity, readability, or code cleanup.

This is in addition to #2439 and mostly are simple.

This mostly came up from review with @samrabin

https://github.com/ESCOMP/CTSM/pull/2427/files#r1543744893
https://github.com/ESCOMP/CTSM/pull/2427/files#r1543712951

- [x] To make it more clear that the output shell scripts are generated have them end with .sh (then it's more clear what's what)
- [x] Going with that change the name of the gen_mksurfdata_build.sh to gen_mksurfdata_build
- [ ] Improve names of some of the subroutines
- [ ] Improve names of the failure tests

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.