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

apply autoformatting; update to latest develop for dev/ufs-weather-model branch; fix wave_elevation export for coupling with CICE6 #840

Conversation

DeniseWorthen
Copy link
Contributor

@DeniseWorthen DeniseWorthen commented Oct 26, 2022

Pull Request Summary

Apply autoformatting and white space removal to dev/ufs-weather-model branch. Update to latest develop. Fix export of wave_elevation spectrum to be updated at coupling frequency when coupled with CICE6.

Description

Apply autoformatting and white space removal to dev/ufs-weather-model branch. Update to latest develop. No change to baselines for UWM regression tests (ufs-community/ufs-weather-model#1474)

Issue(s) addressed

fixes issues #796 and #797 for ufs-weather-model branch
fixes issue #843

Commit Message

Apply autoformatting and white space removal to dev/ufs-weather-model branch. Update to latest develop. Fix export of wave_elevation spectrum to be updated at coupling frequency when coupled with CICE6.

co-author: @mvertens

Check list

Testing

  • How were these changes tested?
  • Are the changes covered by regression tests? (If not, why? Do new tests need to be added?)
  • Have the matrix regression tests been run (if yes, please note HPC and compiler)?
  • Please indicate the expected changes in the regression test output, (Note the list of known non-identical tests.)
  • Please provide the summary output of matrix.comp (matrix.Diff.txt, matrixCompFull.txt and matrixCompSummary.txt):

@DeniseWorthen DeniseWorthen changed the title apply autoformatting; update to latests develop apply autoformatting; update to latest develop Oct 26, 2022
@DeniseWorthen DeniseWorthen changed the title apply autoformatting; update to latest develop apply autoformatting; update to latest develop for dev/ufs-weather-model branch Oct 26, 2022
@DeniseWorthen DeniseWorthen marked this pull request as ready for review October 26, 2022 18:51
DeniseWorthen and others added 2 commits October 29, 2022 08:39
* calc ebd as scalar and use it directly as the wav_elevation_spectrum
* changes are b4b compared to the initial implementation of calcef
@DeniseWorthen DeniseWorthen changed the title apply autoformatting; update to latest develop for dev/ufs-weather-model branch apply autoformatting; update to latest develop for dev/ufs-weather-model branch; fix wave_elevation export for coupling with CICE6 Nov 1, 2022
Copy link
Collaborator

@MatthewMasarik-NOAA MatthewMasarik-NOAA left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Visual inspection looks good to me.

@DeniseWorthen
Copy link
Contributor Author

DeniseWorthen commented Nov 2, 2022

Update of WW3 in UWM has been rolled into UWM ufs-community/ufs-weather-model#1456

@BrianCurtis-NOAA
Copy link

We're ready for reviews/merge on the UFSWM side.

@MatthewMasarik-NOAA
Copy link
Collaborator

We're ready for reviews/merge on the UFSWM side.

Great. We'll merge now on this side.

Copy link
Collaborator

@MatthewMasarik-NOAA MatthewMasarik-NOAA left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

All looks well.

@MatthewMasarik-NOAA MatthewMasarik-NOAA merged commit cec631c into NOAA-EMC:dev/ufs-weather-model Nov 3, 2022
@MatthewMasarik-NOAA
Copy link
Collaborator

Thanks @DeniseWorthen, for this huge step in cleaning up the WW3 code base!

@DeniseWorthen DeniseWorthen deleted the feature/updatewm branch January 30, 2023 15:38
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.

4 participants