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

GAEA support, authoritative branches, non-nested global grids #3

Merged
merged 32 commits into from
Dec 21, 2023

Conversation

SamuelTrahanNOAA
Copy link
Collaborator

@SamuelTrahanNOAA SamuelTrahanNOAA commented Dec 18, 2023

Description of changes

Switch to Authoritative Branches

This switches from HAFS code to the authoritative branches of:

  • tracker
  • ufs-weather-model
  • UFS_UTILS
  • UPP (via ufs-weather-model)

Disable Unused Components

Also, it disables compilation of components that aren't used:

  • GSI
  • hycom_utils
  • ww3_utils

Ports

The HAFS-AR runs on GAEA and Hera except graphics. Pygraf graphics are only installed on Hera.

Uniform and Stretched Global Options

The stretched global and uniform global work. That's a global FV3 with no nest. Access these with:

grid.gtype=uniform or stretch
grid.nest_grids=0

Workarounds

The UPP crashes on GAEA, so we're using the inline post instead.

There's a new CCPP suite FV3_global_nest_v0 which is the FV3_global_nest_v1 with the old gravity wave drag. This is a temporary measure for script testing. The workflow will be updated soon to use the right suite.

The sorc/customizations contains patches that the sorc/build_*.sh scripts apply before compiling. Here are the reasons for them:

  1. UFS_UTILS = GAEA support
  2. tracker = GAEA support
  3. ufs-weather-model = FV3_global_nest_v0 suite (temporary; will be removed soon)

Issues addressed (optional)

No issue submitted. Problem is listed in description.

Dependencies (optional)

None.

Contributors (optional)

None.

Tests conducted

Ran an atmospheric river case and verified the output looked like an atmosphere. Did this on GAEA and Hera.

Application-level regression test status

None.

@SamuelTrahanNOAA SamuelTrahanNOAA changed the title GAEA support + update to latest code GAEA support, authoritative branches, non-nested global grids Dec 18, 2023
Copy link
Collaborator

@guoqing-noaa guoqing-noaa left a comment

Choose a reason for hiding this comment

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

@SamuelTrahanNOAA Thanks for inviting me to review this PR. Great work! I was not able to test this on Gaea as I haven't set up a Gaea account yet but I visually went through the changes and it looks good to me. I asked a few questions but they may be general questions and not specific to this PR.

modulefiles/hafs.hera.lua Show resolved Hide resolved
parm/WaterInTheWestC384NoNest.conf Outdated Show resolved Hide resolved
parm/WaterInTheWestC384Uniform.conf Outdated Show resolved Hide resolved
parm/forecast/globnest/input.nml.nonest.tmp Outdated Show resolved Hide resolved
Copy link
Collaborator

Choose a reason for hiding this comment

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

I assume including "patch.diff" files is a PR requirement?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Yes. This is an unfortunate workaround for the lack of GAEA C5 support in the components.

I'll do a PR to update the tracker soon.

Copy link
Collaborator

@guoqing-noaa guoqing-noaa left a comment

Choose a reason for hiding this comment

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

LGTM. Thanks!

parm/WaterInTheWestC384NoNest.conf Outdated Show resolved Hide resolved
parm/WaterInTheWestC384Uniform.conf Show resolved Hide resolved
rocoto/WaterInTheWest.sh Outdated Show resolved Hide resolved
@@ -21,6 +21,8 @@ elif [ ${target} == "orion" ]; then
FIXROOT=/work/noaa/hwrf/noscrub/hafs-fix-files/hafs-${FIXversion}-fix/fix
elif [ ${target} == "jet" ]; then
FIXROOT=/lfs4/HFIP/hwrf-data/hafs-fix-files/hafs-${FIXversion}-fix/fix
elif [[ $target =~ gaea ]]; then
FIXROOT=/lustre/f2/dev/Samuel.Trahan/hafs/hafs-fix-files/hafs-${FIXversion}-fix/fix
Copy link
Collaborator

Choose a reason for hiding this comment

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

Is there a plan to move this out of your personal directory?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Do we have somewhere else to put it?

Choose a reason for hiding this comment

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

We should be able to put it in the role.epic space somewhere.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

@mark-a-potts -

Please copy this directory to a communal area:

/lustre/f2/dev/Samuel.Trahan/hafs/hafs-fix-files

and let me know when it is there. I'll update the link_fix.sh.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I don't want to wait any longer. I'll merge this now. If you provide me a directory later, I'll update the script.

@SamuelTrahanNOAA SamuelTrahanNOAA merged commit 737888d into ufs-community:develop Dec 21, 2023
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