Skip to content

Update paths for run_ufo_hofx_test.sh#1830

Draft
ADCollard wants to merge 7 commits intodevelopfrom
feature/ursa_paths
Draft

Update paths for run_ufo_hofx_test.sh#1830
ADCollard wants to merge 7 commits intodevelopfrom
feature/ursa_paths

Conversation

@ADCollard
Copy link
Collaborator

@ADCollard ADCollard commented Jul 31, 2025

Updating paths so that test data is visible from URSA.

Note this does not update the paths in config_gsi.yaml and config.jedi.yaml

GDASApp=${GDASApp:-/work2/noaa/da/$LOGNAME/git/GDASApp/} # Change this to your own branch
JCBinstall=${JCBinstall:-/work2/noaa/da/cmartin/CI/GDASApp/opt}
JCBpylib=$JCBinstall/lib/python3.7/site-packages
elif [ $machine = hera ]; then
Copy link
Contributor

Choose a reason for hiding this comment

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

Can we still support Hera? The compute nodes aren't going away immediately

Copy link
Contributor

Choose a reason for hiding this comment

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

Correct. Hera compute will remain past 8/5/2025

08/05* Hera scratch 1-2 disks decommissioned (Hera CPU nodes still available)
*CPU allocations remain on Hera and users can run jobs using the cross-mounted Ursa disks. Hera access
will remain for as long as there is funding.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Can we still support Hera? The compute nodes aren't going away immediately

OK, I added it back and also added hercules

elif [ $machine = hera ]; then
export Datapath='/scratch1/NCEPDEV/da/Emily.Liu/UFO_eval/data/gsi_geovals_l127/nofgat_aug2021/'$dataprocdate
FixDir=/scratch1/NCEPDEV/da/Cory.R.Martin/GDASApp/fix
elif [ $machine = ursa || $machine" = hera ]; then
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
elif [ $machine = ursa || $machine" = hera ]; then
elif [ $machine = ursa || $machine = hera ]; then

Copilot AI and others added 3 commits July 31, 2025 20:39
Co-authored-by: CoryMartin-NOAA <6354668+CoryMartin-NOAA@users.noreply.github.com>
Co-authored-by: CoryMartin-NOAA <6354668+CoryMartin-NOAA@users.noreply.github.com>
Copy link
Contributor

@RussTreadon-NOAA RussTreadon-NOAA left a comment

Choose a reason for hiding this comment

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

Didn't test but looks good to me.

Approve.

@ADCollard ADCollard marked this pull request as draft August 1, 2025 14:13
@ADCollard
Copy link
Collaborator Author

Converting to draft while I update the test data and re-test.

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