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-dev PR#42 #374

Merged
merged 3 commits into from
Apr 12, 2023
Merged

UFS-dev PR#42 #374

merged 3 commits into from
Apr 12, 2023

Conversation

grantfirl
Copy link
Collaborator

Changes to GFS_typedefs to work with ufs/dev #42. The changes are analogous to NCAR/fv3atm#89

@grantfirl
Copy link
Collaborator Author

Contains changes from #373 and depends on it being merged first. Also depends on NCAR/ccpp-physics#1007.

@dustinswales
Copy link
Collaborator

@grantfirl I think this is good to go?
It's a bit troublesome that we are seeing lots of BL changes in the UWM, but not in here. NCAR/ufs-weather-model#92?

@grantfirl
Copy link
Collaborator Author

@grantfirl I think this is good to go? It's a bit troublesome that we are seeing lots of BL changes in the UWM, but not in here. NCAR/ufs-weather-model#92?

Ya, this probably needs to be verified. I'm also not sure that the failures in UWM are all expected since it wasn't explicitly mentioned in the UFS fork which RTs should fail. I've asked Sam for some clues, but haven't heard back. I'll look at this a bit more later this afternoon.

@grantfirl
Copy link
Collaborator Author

@dustinswales Do you know when the last time ftp://ftp.rap.ucar.edu/pub/ccpp-scm/rt-baselines-Debug.zip was updated?

@grantfirl
Copy link
Collaborator Author

Should https://github.com/NCAR/ccpp-scm/blob/main/test/cmp_rt2bl.py be using nccmp rather than cmp?

@dustinswales
Copy link
Collaborator

@dustinswales Do you know when the last time ftp://ftp.rap.ucar.edu/pub/ccpp-scm/rt-baselines-Debug.zip was updated?

@grantfirl I updated them the last time they changed, which was a few weeks ago.

@dustinswales
Copy link
Collaborator

March 27th.

@grantfirl grantfirl merged commit 99efc12 into NCAR:main Apr 12, 2023
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