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.

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

Updates to auto-rt scripts to run on Cheyenne etc #443

Closed
wants to merge 1 commit into from

Conversation

climbfuji
Copy link
Collaborator

WORK IN PROGRESS

Description

Updates to auto-rt scripts to run on Cheyenne, temporary workaround to use GNU instead of Intel on that system. This will be generalized in a future PR.

Issue(s) addressed

n/a

Testing

WORK IN PROGRESS

How were these changes tested?
What compilers / HPCs was it tested with?
Are the changes covered by regression tests? (If not, why? Do new tests need to be added?)
Have regression tests and unit tests (utests) been run? On which platforms and with which compilers? (Note that unit tests can only be run on tier-1 platforms)

Dependencies

This PR will be pulled into and merged as part of #425.

@climbfuji
Copy link
Collaborator Author

@BrianCurtis-NOAA Sorry for the back and forth, but given that this PR touches two files only, which are not used by anything else, it would be best to commit it separately afterwards. This way we can use the PR itself to demonstrate that the auto-rt updates are working as expected. Shouldn't be any more work.

@DusanJovic-NOAA
Copy link
Collaborator

@BrianCurtis-NOAA Sorry for the back and forth, but given that this PR touches two files only, which are not used by anything else, it would be best to commit it separately afterwards. This way we can use the PR itself to demonstrate that the auto-rt updates are working as expected. Shouldn't be any more work.

How about keeping these two files in a separate location (repository) during this early stages of development, to allow faster commits. Do they need to be located in the ufs-weather-model repository? If they are in some other location you will be able to make much more frequent changes. Just keep cron jobs pointing to the latest version.

@BrianCurtis-NOAA
Copy link
Collaborator

@DusanJovic-NOAA I can manually update the scripts on each machine through development, but the final additions Jun would like to keep in the ufs-weather-model repo.

@climbfuji I put the changes that were in this repo into PR#444. PR 444 only touches two files as well. The log files shouldn't get merged.

pjpegion pushed a commit to NOAA-PSL/ufs-weather-model that referenced this pull request Apr 4, 2023
…fs-community#443)

* Remove old comments from GFS_typedefs.F90.
* Update logic that sets Model%cnvcld = .false.
epic-cicd-jenkins pushed a commit that referenced this pull request Apr 17, 2023
* Add a move step to binaries for NCO purposes.

* Pass COMROOT to setpdy.

* Correct path of post dependency on forecat.

* Remove DATA_SHARED directory.

* Remove symlinks and use unique id for run_fcst instead.

* Remove forecast directory after post is done.

* Use raw workflow_id

* Add templated data paths to wcoss2 that use compath.py.

* Remove cron relaunch from config.nco

* Undo for_ICS/LBCS stuff in NCO mode.

* Remove RESTART symlinking because DATAFCST is accessible to all other tasks.

* Undo cyc addition to COMIN.

* Update devclean.

* Follow symlinks when tarring log files in Jenkins.

* Add memory otpion for get_extrn_ics/lbcs.

* Optimize fundamental test coverage.

* Reduce work load on orion.

* Add minimum memory requirement to all serial tasks.

* Simplify set_predef_grid_params.

* Simplify generate_FV3LAM_wflow.

* Simplify link_fix.

* Simplify to get rid of export_vars from setup.

* Import only necessary variables from environment.

* Use `eval echo` or `eval` depending on situation.

* Add cyc back to COMIN.

* Fix calculate_cost.

* Fix gaea test list.

* Remove a test case from cheyenne that is taking extremely long.

* Reduce memory requirement for serial job to 1G.

* Create COMINaws ics/lbcs staging directory.

* Also symlink gfs ics/lbcs if on disk and in community mode.

* Remove failing tests on orion.

* Exclude gaea from --mem specification.

* Add a run_vx modulefile for orion.

* Add separate fundamental list for cheyenne GNU runs.

* Bug fix for wcoss2 GFS version

* Load prod_util on wcoss2.

* Rename AWSROOT to EXTROOT.

* Remove leftover COMINaws.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants