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.

Sign up for GitHub

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

Jump to bottom

Generate forecast grid when quitling is off. #450

Closed
junwang-noaa opened this issue Mar 4, 2021 · 0 comments · Fixed by #364
Closed

Generate forecast grid when quitling is off. #450

junwang-noaa opened this issue Mar 4, 2021 · 0 comments · Fixed by #364
Assignees
@junwang-noaa
Labels
bug Something isn't working

Comments

@junwang-noaa
Copy link
Collaborator

Description

Developer reported that "the creation of the ESMF grid inside the fv3 NUOPC cap was conditioned on the write component being turned on (quilting=.true.). However, this is faulty logic since the grid in the fv3 cap might be needed for other purposes - such as coupling - or to pass data up to the driver to hand off to JEDI, etc. This needs to be rectified since during development it is not always practical or desirable to turn on the write component."

To Reproduce:

Running coupled model without turning on quilting will see the error.

Additional context

Fixes has been made in FV3/module_fcst_grid_comp.F90 in fv3 PR#229
@junwang-noaa junwang-noaa added the bug Something isn't working label Mar 4, 2021
@junwang-noaa junwang-noaa self-assigned this Mar 4, 2021
@junwang-noaa junwang-noaa mentioned this issue Mar 4, 2021
Merged
pjpegion pushed a commit to NOAA-PSL/ufs-weather-model that referenced this issue Apr 4, 2023
@DusanJovic-NOAA
Use 'model set run clock' routine in FV3 NUOPC cap. (ufs-community#450)
96a2554 
* Add 'SetRunClock' specialization routine to FV3 NUOPC cap, which sets fv3 model clock. FV3 cap does not keep local copy of clock (clock_fv3) as saved module variable anymore.
* Minor code cleanup.
epic-cicd-jenkins pushed a commit that referenced this issue Apr 17, 2023
@chan-hoo
Make WE2E tests work on the WCOSS Dell and Cray (#450)
045ad00 
* Add options for wcoss to we2e tests
* Make cron work on wcoss_dell_p3
* Modify the baseline list file
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@junwang-noaa junwang-noaa

Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Regional inlinepost junwang-noaa/ufs-weather-model
1 participant
@junwang-noaa