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

Updated hashes of regional_workflow and UFS_UTILS #133

Closed
wants to merge 1 commit into from

Conversation

mdtoy
Copy link
Collaborator

@mdtoy mdtoy commented Apr 23, 2021

DESCRIPTION OF CHANGES:

This updates the hashes of regional_workflow and UFS_UTILS to work with PR #448 into regional_workflow

TESTS CONDUCTED:

See PR #448 into regional_workflow

DOCUMENTATION:

If this PR is contributing new capabilities that need to be documented, please also include updates to the RST files (docs/UsersGuide/source) as supporting material.

@@ -11,8 +11,7 @@ required = True
protocol = git
repo_url = https://github.com/NOAA-EMC/UFS_UTILS
# Specify either a branch name or a hash but not both.
#branch = develop
hash = 005f9a0a
hash = ea821358
local_path = src/UFS_UTILS
Copy link
Collaborator

Choose a reason for hiding this comment

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

Hi @mdtoy, can you keep the "#branch = develop" line in your version?

@JeffBeck-NOAA
Copy link
Collaborator

@mdtoy, now that your regional_workflow PR is merged, you can update the hash of that repo in the Externals.cfg file, and I think we're good to go after that.

@JulieSchramm
Copy link

The hash for UFS_UTILS has already been updated to ea821358 in the master branch. The regional_workflow hash needs to be updated.

@JeffBeck-NOAA
Copy link
Collaborator

The hash for UFS_UTILS has already been updated to ea821358 in the master branch. The regional_workflow hash needs to be updated.

Thanks, @JulieSchramm! @mdtoy, it looks like you can remove the UFS_UTILS hash from this PR (it went in with this PR, and just update the regional_workflow hash.

@JeffBeck-NOAA
Copy link
Collaborator

The hash changes mentioned here were covered in previous PRs, so closing this PR.

christopherwharrop-noaa pushed a commit to christopherwharrop-noaa/ufs-srweather-app that referenced this pull request Apr 12, 2022
natalie-perlin pushed a commit to natalie-perlin/ufs-srweather-app that referenced this pull request Apr 25, 2022
DESCRIPTION OF CHANGES:
Add a subdirectory to ush/ containing wrapper scripts

each wrapper script runs one workflow task
the experiment-generation step MUST be completed before running these scripts
an example batch submit script is provided for PBSpro (cheyenne) and Slurm (hera)
a README file is provided with basic instructions (including which tasks in which order)
users on other systems will need to customize a batch script for their system
TESTS CONDUCTED:
A single experiment was run using the standard rocoto job control, on hera and cheyenne. This same experiment was run using the wrapper scripts on hera and cheyenne.

ISSUE (optional):
Fixes issue ufs-community#133
mkavulich pushed a commit to mkavulich/ufs-srweather-app that referenced this pull request Aug 26, 2022
DESCRIPTION OF CHANGES:
Add a subdirectory to ush/ containing wrapper scripts

each wrapper script runs one workflow task
the experiment-generation step MUST be completed before running these scripts
an example batch submit script is provided for PBSpro (cheyenne) and Slurm (hera)
a README file is provided with basic instructions (including which tasks in which order)
users on other systems will need to customize a batch script for their system
TESTS CONDUCTED:
A single experiment was run using the standard rocoto job control, on hera and cheyenne. This same experiment was run using the wrapper scripts on hera and cheyenne.

ISSUE (optional):
Fixes issue ufs-community#133
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