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

new fix for add_gusts #474

Merged
merged 4 commits into from
Jul 1, 2024
Merged

new fix for add_gusts #474

merged 4 commits into from
Jul 1, 2024

Conversation

jedwards4b
Copy link
Collaborator

Description of changes

The associated statement won't work because the association is done in the called subroutine.

Specific notes

Contributors other than yourself, if any:

CMEPS Issues Fixed (include github issue #):

Are changes expected to change answers? (specify if bfb, different at roundoff, more substantial)

Any User Interface Changes (namelist or namelist defaults changes)?

Testing performed

Please describe the tests along with the target model and machine(s)
If possible, please also added hashes that were used in the testing

@jedwards4b jedwards4b self-assigned this Jun 27, 2024
@jedwards4b
Copy link
Collaborator Author

@DeniseWorthen Please test this change with UFS.

@DeniseWorthen
Copy link
Collaborator

@jedwards4b I have our tests running w/ this fix as well as #477.

@DeniseWorthen
Copy link
Collaborator

@jedwards4b This is fine for UFS. Thanks.

@jedwards4b jedwards4b merged commit 3305e11 into ESCOMP:main Jul 1, 2024
1 of 2 checks passed
@jedwards4b jedwards4b deleted the add_gust_fix3 branch July 1, 2024 14:53
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.

None yet

2 participants