Skip to content

Conversation

climbfuji
Copy link
Collaborator

Description of Changes:

See #1151

Tests Conducted:

Tested the NRL version of this code.

Dependencies:

None

Documentation:

None

Issue (optional):

Closes #1151

Contributors (optional):

@areinecke and @matusmartini

@climbfuji
Copy link
Collaborator Author

@grantfirl @dustinswales Gentle reminder about this PR.

Copy link
Member

@dustinswales dustinswales left a comment

Choose a reason for hiding this comment

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

Looks fine to me.
I will make corresponding SCM PR. If RTs pass I will approve.

@grantfirl
Copy link
Collaborator

@climbfuji @dustinswales I'm finally getting around to testing #1141 in the UFS. For testing, I'll combine this PR into that (locally at first). Assuming all tests are OK, we can combine PRs and merge together. Then start a PR into ufs/dev to sync up.

@grantfirl
Copy link
Collaborator

Testing the combined code changes in the SCM and UFS is complete and was successful, so I've combined the PRs into another one and will merge shortly.

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.

Use of w3emc in GFS_time_vary_pre is not safe - no protection against argument mismatches
3 participants