Skip to content

Update modulefiles and UFS Weather model build option#6

Merged
zach1221 merged 20 commits into
ufs-community:developfrom
jkbk2004:feature/ufs-wm
May 1, 2023
Merged

Update modulefiles and UFS Weather model build option#6
zach1221 merged 20 commits into
ufs-community:developfrom
jkbk2004:feature/ufs-wm

Conversation

@jkbk2004
Copy link
Copy Markdown
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
Copy Markdown
Collaborator Author

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

@ulmononian
Copy link
Copy Markdown
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
Copy Markdown

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
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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.

Comment thread modulefiles/ufs_orion.intel.lua Outdated
Comment thread modulefiles/ufs_hera.intel.lua Outdated
@jkbk2004 jkbk2004 requested a review from zach1221 May 1, 2023 20:22
@jkbk2004
Copy link
Copy Markdown
Collaborator Author

jkbk2004 commented May 1, 2023

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

@zach1221
Copy link
Copy Markdown
Collaborator

zach1221 commented May 1, 2023

@jkbk2004 yes I'll merge

@zach1221
Copy link
Copy Markdown
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