Bugfix/FLX5 Tau#493
Closed
ukmo-ccbunney wants to merge 4 commits into
Closed
Conversation
Error introduced after recent FLX5 switch addition.
ukmo-nievesvaliente
approved these changes
Oct 12, 2021
Collaborator
ukmo-nievesvaliente
left a comment
There was a problem hiding this comment.
The bug has been corrected and the switch works fine now.
Merged
Collaborator
Author
|
This PR will be merged via the UKMO Staging Branch; see #499 |
kestonsmith-noaa
pushed a commit
to kestonsmith-noaa/WW3
that referenced
this pull request
Jun 17, 2025
…add export of openwater_frac_in_atm from fv3ATM when coupled (NOAA-EMC#475) * update CICE in ufs-weather for the latest emc/develop * fix ice_in_template; sw_redist has been moved to shortwave_nml and removed two unused settings * update fv3ATM to use esmf-bs47 and add the export of open-water fraction from fv3ATM * update module files to use bs47 (PR NOAA-EMC#493) * fix wcoss-cray for alt-compare functionality * update CMEPS and NEMS for added export field * update Dockerfile
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Pull Request Summary [SEE #499]
Fixes order of parameters in
W3SRCEsubroutine definition when compiled withFLX5switch.Description
During the addition of the
!/FLX5flags to PR #483 I unintentionally altered the order of the parameters in the subroutine definition ofW3SRCEin w3srcemd.The model still compiles and runs, but the results are incorrect.
This only affects models compiled with the FLX5 switch and didn't show up in the regression tests as there was no existing FLX5 test.
Check list
Commit Message
Testing
Regression tests results:
matrix_flx5_tau.zip
Summary:
Usual issues with mww3_test_03.
ww3_tp2.15: The two FLX5 tests are different (as expected).