forked from NOAA-EMC/fv3atm
-
Notifications
You must be signed in to change notification settings - Fork 14
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
gsl/develop RRFS-Smoke #141
Merged
SamuelTrahanNOAA
merged 13 commits into
NOAA-GSL:gsl/develop
from
haiqinli:gsl/develop-smoke
Apr 21, 2022
Merged
Changes from 3 commits
Commits
Show all changes
13 commits
Select commit
Hold shift + click to select a range
c49aeca
"add for smoke"
haiqinli 9102a85
"update to pass GNU compiler"
haiqinli c10e3c1
"following Joe's comments to add nchem,ndvel as input to MYNN"
haiqinli 85a9c56
"update fv3atm to point to gsl/develop-smoke"
haiqinli fe8701d
"to point to physics branch of gsl/develop-smoke"
haiqinli a450884
Point to gsl/develop-smoke for ccpp-physics
SamuelTrahanNOAA 0025dde
Merge branch 'gsl/develop-smoke' of ssh://github.com/haiqinli/fv3atm …
SamuelTrahanNOAA dbc8a7a
"point ot gsl/develop-smoke branch"
haiqinli 920b401
"update GFS_typedefs.meta with rrfs_smoke active flag"
haiqinli 0aeb15f
"update the 3rd dimension readin of GBBEPx for smoke"
haiqinli 5bd314f
Remove an extra read that went out of bounds
SamuelTrahanNOAA efc3846
Merge branch 'gsl/develop-smoke' of ssh://github.com/haiqinli/fv3atm …
SamuelTrahanNOAA 6219be9
Point to NOAA-GSL gsl/develop for ccpp-physics
SamuelTrahanNOAA File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
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
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
Oops, something went wrong.
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I must have missed it, but where is mix_chem set to true when rrfs_smoke is set to true? Or do you have to set both manually?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The mix_chem and fire_turb are false by default. Actually all the smoke related options are false by default.
Then they are read from the external namelist file of gfs_physics_nml. In standard alone RRFS (SDF of FV3_HRRR), they are false. In the RRFS-Smoke (SDF of FV3_HRRR_smoke), they are true.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
So, I think that means they both need to be set = true in the namelist.
Anyway, in the interest of time, I will approve these changes, but eventually "kdvel" will have to be specified outside of the MYNN wrapper and hard-coding "EMIS_ANT_NO" and "vdep" should also be removed from the MYNN wrapper. This won't hurt to leave them in the wrapper for now, but this really isn't the proper location to handle these non-mynn-specific variables.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, the are set = true in the namelist.
I agree with. In order to save time, we keep the current status of "kdvel", "EMIS_ANT_NO" and "vdep" this time.
We will get rid of "kdvel", "EMIS_ANT_NO" and "vdep" in the next commit.