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

Sync with latest WW3 develop #1668

Closed
JessicaMeixner-NOAA opened this issue Mar 21, 2023 · 3 comments
Closed

Sync with latest WW3 develop #1668

JessicaMeixner-NOAA opened this issue Mar 21, 2023 · 3 comments
Assignees
Labels
enhancement New feature or request

Comments

@JessicaMeixner-NOAA
Copy link
Collaborator

JessicaMeixner-NOAA commented Mar 21, 2023

Description

This is for a sync with the latest WW3 develop. Recent updates include:

Solution

Will provide a PR to dev/ufs-waether-model branch of WW3 and a PR to ufs-weather-model --- anticipated by 3/23

Alternatives

None

Related to

#1556

@JessicaMeixner-NOAA JessicaMeixner-NOAA added the enhancement New feature or request label Mar 21, 2023
@JessicaMeixner-NOAA JessicaMeixner-NOAA self-assigned this Mar 21, 2023
@DeniseWorthen
Copy link
Collaborator

@JessicaMeixner-NOAA Since I am the one doing the actual work on the unstructured mesh for UFS, I would ask that you not merge the wise PR into the ufs-weather-model branch until I have a chance to test it using the unstructured mesh. I already have a draft pr to update the current dev/ufs-weather-model w/ the current fixes that were just merged PR950 prior to wise.

@JessicaMeixner-NOAA
Copy link
Collaborator Author

JessicaMeixner-NOAA commented Mar 21, 2023

@DeniseWorthen I was actually doing this to help accelerate development for you, not to cause you problems, so I will certainly not do this at your request. @MatthewMasarik-NOAA and I will take care of merging from WW3 develop to dev/ufs-weather-model as the WW3 code managers. I'll back this out to just the two updates minus the WISE and wait utnil when you're ready for the WISE PR to be sync merged. I did confirm that the WISE PR does not break the existing ufs-weather-model regression tests. I'll wait for word from you about the unstructured grid development testing before syncing up develop again.

@DeniseWorthen
Copy link
Collaborator

I will let you know after I have tested the latest develop (including wise) for the unstructured case and let you know. It should only take a day. I want to make sure there are no unexpected surprises.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants