Skip to content

Add inline post option to the SRW App#150

Merged
chan-hoo merged 4 commits into
ufs-community:developfrom
chan-hoo:feature/in_post
Jul 7, 2021
Merged

Add inline post option to the SRW App#150
chan-hoo merged 4 commits into
ufs-community:developfrom
chan-hoo:feature/in_post

Conversation

@chan-hoo
Copy link
Copy Markdown
Collaborator

@chan-hoo chan-hoo commented Jul 6, 2021

DESCRIPTION OF CHANGES:

  • Add a compile argument for inline post to the ufs weather model in 'src/CMakeLists.txt'.
  • Update some modules in 'env' for Hera and WCOSS Dell to the latest.

TESTS CONDUCTED:

  • GST test + verification on Hera and WCOSS Dell.
  • New WE2E test for inline post on Hera and WCOSS Dell.

ISSUE:

@chan-hoo
Copy link
Copy Markdown
Collaborator Author

chan-hoo commented Jul 7, 2021

@JeffBeck-NOAA, please review this PR. Since the pr for inline post was merged in the regional workflow, this pr should be merged to run the latest regional workflow correctly.

@JeffBeck-NOAA
Copy link
Copy Markdown
Collaborator

@chan-hoo, do environment files for all other systems (aside from Hera and WCOSS) need to be updated to work with in-line post?

@chan-hoo
Copy link
Copy Markdown
Collaborator Author

chan-hoo commented Jul 7, 2021

@JeffBeck-NOAA, No, they don't. I updated the env files for Hera and WCOSS because I got a version change notice when I compiled the SRW App on them. These are not related to the inline post.

@JeffBeck-NOAA
Copy link
Copy Markdown
Collaborator

OK, thanks! Approving.

@chan-hoo chan-hoo merged commit fb48578 into ufs-community:develop Jul 7, 2021
@chan-hoo chan-hoo deleted the feature/in_post branch July 7, 2021 17:55
christinaholtNOAA pushed a commit to christinaholtNOAA/ufs-srweather-app that referenced this pull request May 23, 2022
1) use 127L BEC
2) use single domain for restart files on CONUS grid 3km grid
3) add wet1 in diag_table
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.

2 participants