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

Update modulefiles and UFS Weather model build option #6

Merged
merged 20 commits into from
May 1, 2023
Merged

Update modulefiles and UFS Weather model build option #6

merged 20 commits into from
May 1, 2023

Conversation

jkbk2004
Copy link
Collaborator

@jkbk2004 jkbk2004 commented Apr 18, 2023

Description

Subcomponents involved:

  • DA_update
  • vector2tile
  • ufs-land-driver
  • none

Linked PR's and Issues:

Closes #4

Testing (for CM's):

  • RDHPCS

    • [*] Hera
    • Orion
    • Jet
    • Gaea
    • Cheyenne
  • CI

    • Completed
  • PW-louds

    • AWS
    • AZURE
    • GCP
  • Need to update container according to the updated modulefiles

  • Tips for build test steps:

    • module use modulefiles -> module load ufs_hera.intel or ufs_orion.intel -> module load ufs_common
    • mkdir build -> cd build -> ecbuild .. -> make -j 4

@jkbk2004
Copy link
Collaborator Author

@ulmononian FYI: we need to update container to fully follow up with the modulefile updates.

@ulmononian
Copy link
Collaborator

@ulmononian FYI: we need to update container to fully follow up with the modulefile updates.

i believe @climbfuji is working on an official spack-stack/1.3.1 container for the ufs-wm. further, @mark-a-potts has created a container to run the WM RTs. however, i am not sure which container is/will be best suited for this case. any thoughts?

@climbfuji
Copy link

Just to confirm that I am working on it, yes.

rhaesung
rhaesung previously approved these changes Apr 18, 2023
ulmononian
ulmononian previously approved these changes Apr 19, 2023
Copy link
Collaborator

@ulmononian ulmononian left a comment

Choose a reason for hiding this comment

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

like butter! @jkbk2004

yichengt90
yichengt90 previously approved these changes Apr 19, 2023
Copy link
Contributor

@yichengt90 yichengt90 left a comment

Choose a reason for hiding this comment

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

Successfully build ufs-weather-model with externalProject build.

@jkbk2004 jkbk2004 requested review from barlage and removed request for clouden90 April 19, 2023 12:16
@jkbk2004
Copy link
Collaborator Author

@barlage FYI: we are adding ufs-weather model build option in the ecbuild-based cmake for the use of Noah-MP -DAPP=LND. We can start using Noah-MP weather model baseline as our starting point.

@jkbk2004 jkbk2004 requested a review from zach1221 May 1, 2023 20:22
@jkbk2004
Copy link
Collaborator Author

jkbk2004 commented May 1, 2023

@zach1221 I tested ok with singularity. can you merge in the pr?

@zach1221
Copy link
Collaborator

zach1221 commented May 1, 2023

@jkbk2004 yes I'll merge

@zach1221
Copy link
Collaborator

zach1221 commented May 1, 2023

@jkbk2004 are you able to mark the two pending conversations as resolved?

@zach1221 zach1221 merged commit 19a164d into ufs-community:develop May 1, 2023
jkbk2004 pushed a commit that referenced this pull request May 4, 2023
* update links & image file paths

* updates for PR#6

* update container build command

* update jedi-bundle ref and fix xlink

* update data bucket url
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 modulefiles to combine with UFS-WM NOAH-MP subcomponent
6 participants