Adding new VISBSN variable #52
Merged
Conversation
This file contains 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
|
@hu5970 this PR is ready for review. It adds a single new output field, visibility which includes effect of blowing snow. Output is identical to the top of EMC's develop branch, except for the addition of the new field. |
hu5970
approved these changes
Feb 15, 2023
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.
This PR introduces a new variables, VISBSN, which includes the effect of blowing snow on the GSL visibility algorithm. We would like to output both the original and new visibility to compare in realtime...once we are satisfied it is working as we like, the code can be modified again to only output a single visibility, which can be used in operations.
The code was tested for RRFS_CONUS_3km on Jet.