Add PBS job and submission wrapper for bottom roughness generation on Gadi#117
Merged
minghangli-uni merged 5 commits intomainfrom Feb 26, 2026
Merged
Add PBS job and submission wrapper for bottom roughness generation on Gadi#117minghangli-uni merged 5 commits intomainfrom
minghangli-uni merged 5 commits intomainfrom
Conversation
Collaborator
Author
|
Below is the help message of $ ./submit_bottom_roughness.sh -h
Submit bottom roughness generation PBS job (Gadi).
./submit_bottom_roughness.sh -s PATH_TO_SAVE -r RESOLUTION -p true|false -g HGRID -t TOPOG -j PBS_SCRIPT
Note:
Reusing PATH_TO_SAVE across resolutions is recommended. The WOA intermediate
file (depth variance on the WOA grid) is resolution-independent and can be
reused. After the first run, subsequent runs in the same directory will
typically skip the intermediate step and only perform regridding much faster.
However the final output filename is always 'bottom_roughness.nc'.
Please run one resolution at a time and move or rename the output file
(e.g. to /g/data/vk83/) before running the next resolution to avoid
overwriting previous results.
-h Show this help message
Required arguments:
-s Path to save intermediate and final outputs
-r Resolution is one of: 100km, 25km, 8km, panan_4km.
-p Periodic mode: true (periodic longitude); false (aperiodic longitude) is not supported yet.
(Sets BOTH PERIODIC_REGRID and PERIODIC_LON_LAPLACE as a bundle)
-g Path to ocean_hgrid.nc
-t Path to ocean_topog.nc
-j Path to PBS script
Examples:
# periodic longitude
./submit_bottom_roughness.sh -s /path/to/save -r 25km -p true -g /path/to/hgrid.nc -t /path/to/topog.nc -j /path/to/pbs_bottom_roughness.pbs
# aperiodic longitude (not supported yet)Hence for 25km configs, one can do, $ ./submit_bottom_roughness.sh \
-s . \
-p true \
-g /g/data/vk83/configurations/inputs/access-om3/mom/grids/mosaic/global.25km/2025.09.02/ocean_hgrid.nc \
-t /g/data/vk83/configurations/inputs/access-om3/share/grids/global.25km/2025.11.27/topog.nc \
-r 25km \
-j pbs_bottom_roughness.pbs |
Collaborator
Author
|
Hi @aekiss would you mind reviewing this? |
aekiss
reviewed
Feb 26, 2026
aekiss
previously approved these changes
Feb 26, 2026
Contributor
aekiss
left a comment
There was a problem hiding this comment.
LGTM, thanks @minghangli-uni - I assume you've tested it?
Co-authored-by: Andrew Kiss <31054815+aekiss@users.noreply.github.com>
Collaborator
Author
Yep I've tested it. Could you please approve this again? |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
closes #116