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

ufs-bundle update to support building ATM/NG-GODAS/S2S UFS, and running ATM warmstart configurations #9

Merged
merged 48 commits into from
Feb 7, 2023

Conversation

climbfuji
Copy link
Collaborator

@climbfuji climbfuji commented Oct 20, 2022

Description

This PR includes all the contributions from @mark-a-potts described in PRs #3 and #5, and on top of it many more updates that enable building the ufs-bundle in atmosphere-only (ATM), ocean-only (NG-GODAS) and fully coupled (S2S) configurations.

Due to recent updates to the ufs-weather-model (and in particular mom6, cice6), the NG-GODAS ctest is no longer working (need update to input data sets, amongst others). There is no ctest for the S3S configuration yet. For atmosphere-only, we have two working ctests for setting up warmstart and coldstart run directories, and two working warmstart runs (model and forecast). The coldstart runs need more work to enable reading UFS coldstart initial conditions in fv3-jedi.

Definition of Done

Can we merge this PR and mark it as completed, even though the code changes for the submodules are still in their own branches and not yet merged?

Issue(s) addressed

Fixes #12
Working towards #8
Working towards #15
Working towards #16

Dependencies

JEDI

https://github.com/JCSDA-internal/fv3-jedi-linearmodel/pull/16
https://github.com/JCSDA-internal/fv3-jedi/pull/697
JCSDA-internal/soca#824
JCSDA-internal/fv3-jedi-data#52
https://github.com/JCSDA-internal/saber/pull/385

UFS

NOAA-EMC/AQM#40
NOAA-EMC/MOM6#107
NOAA-GFDL/GFDL_atmos_cubed_sphere#231
NOAA-EMC/fv3atm#620
ufs-community/ufs-weather-model#1504

Impact

None

@climbfuji climbfuji self-assigned this Oct 20, 2022
Copy link
Collaborator

@shlyaeva shlyaeva left a comment

Choose a reason for hiding this comment

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

One quick comment on looking through the code changes, and I'll also build and test today.

CMakeLists.txt Show resolved Hide resolved
@shlyaeva
Copy link
Collaborator

I've started on orion which doesn't have fms/2022.04 yet. Is there an easy way to build it as part of the bundle? No worries if not, I don't have to build it there. If not, do you have a recommended platform to test (pcluster?)

@climbfuji
Copy link
Collaborator Author

(1) You should be able to build a new spack-stack environment from the latest spack-stack develop branch using the skylab-dev template
(2) You should also be able to add fms@2022.04 to your existing skylab-v3 environment (but take a backup first, just in case), even without updating the spack-stack and spack code that you checked out for v3. After activating your existing environment:

spack add fms@2022.04 +64bit +quad_precision +gfs_phys +openmp +fpic constants=GFS
spack concretize
spack install --verbose --source
spack module lmod refresh fms

After loading the compiler, mpi and python metamodules, load jedi-ufs-env followed by loading fms@2022.04 (which will swap out the already loaded fms@2022.02

shlyaeva
shlyaeva previously approved these changes Jan 28, 2023
Copy link
Collaborator

@shlyaeva shlyaeva left a comment

Choose a reason for hiding this comment

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

I built on my mac (with freshly built spack-stack from yesterday's develop using skylab-dev template) and ran fv3-jedi tests (only for ATM). I am approving based on this (I don't have expertise to review cmake changes). Some fv3-jedi tests are failing, but I think it's a known issue (lam tests). I'm starting to update JEDI components to develop.

@@ -1,2 +1,106 @@
# ufs-jedi-bundle
# ufs-bundle
Copy link
Collaborator

Choose a reason for hiding this comment

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

This readme is super informative and useful, made it so easy to build. Thank you in particular for notes on building on mac!

@shlyaeva
Copy link
Collaborator

Some fv3-jedi tests are failing, but I think it's a known issue (lam tests)

Actually, I'm not sure it's a known issue. @climbfuji do lam_cmaq tests pass for you with this build?

@climbfuji
Copy link
Collaborator Author

Some fv3-jedi tests are failing, but I think it's a known issue (lam tests)

Actually, I'm not sure it's a known issue. @climbfuji do lam_cmaq tests pass for you with this build?

No, they don't. That's something we need to figure out. But because this is the ufs-bundle and not the fv3-bundle, I am unclear which tests have to pass (and which can fail for what reason).

* Update jedi repos to develop as of Jan 28 2023

* Update branch for oops for fixing missing netcdf-c dependencies

* Update CMakeLists.txt

* Update oops tag in CMakeLists.txt

* Change branches back to feature/ufs_dom

---------

Co-authored-by: Dom Heinzeller <dom.heinzeller@icloud.com>
@climbfuji climbfuji requested a review from cmgas February 4, 2023 04:01
@climbfuji
Copy link
Collaborator Author

This is your weekly, friendly reminder ... can we get this reviewed please? Thanks!

@shlyaeva
Copy link
Collaborator

shlyaeva commented Feb 6, 2023

@mark-a-potts, @aerorahul, @cmgas do you have time to review in the next couple of days?

Copy link
Contributor

@aerorahul aerorahul left a comment

Choose a reason for hiding this comment

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

The changes to the CMakeLists.txt look reasonable.
The addition of README.md is much appreciated.

@shlyaeva
Copy link
Collaborator

shlyaeva commented Feb 6, 2023

Thank you so much @aerorahul!

Copy link
Contributor

@cmgas cmgas left a comment

Choose a reason for hiding this comment

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

I used this branch to build on my Mac and all is going well, thank you! And great ReadMe :)

@climbfuji
Copy link
Collaborator Author

Thanks everyone, I think we should merge this to get it out of the way and then work on the outstanding issues?

@shlyaeva
Copy link
Collaborator

shlyaeva commented Feb 7, 2023

I agree with @climbfuji

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.

Update ufs-bundle to build and run atmosphere-only configurations
7 participants