-
Notifications
You must be signed in to change notification settings - Fork 183
cam6_3_046: PUMAS Updates #409
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
Merged
Katetc
merged 22 commits into
ESCOMP:cam_development
from
PUMASDevelopment:gettelman_pumas_update0821
Feb 1, 2022
Merged
Changes from 20 commits
Commits
Show all changes
22 commits
Select commit
Hold shift + click to select a range
1ea4d5e
CAM Changes in support of PUMAS update, first round changes tested in…
7a328f1
Remove vapdepso_grid and just average on output as needed
Katetc 8485932
Updating CAM externals to use new pumas tag v1.18
3167e9c
Removing unused code in response to review
36a2fb7
Update Namelist Definition file, correcting an error in documentation…
b792420
Correcting typos in comments
2f2346b
updated implicit fall speed code
6bf851b
Merge pull request #6 from ESCOMP/cam_development
Katetc b7a94d0
Whitespace changes to prevent merge conflicts with Jian's code
Katetc a479234
update to the most recent PUMAS tag
Katetc c775201
Merge up to cam6_3_043
Katetc b5a9fb0
Move new pumas to cam_dev and update names from mg to pumas in more p…
Katetc 7fbafa1
Re-review noticed comments not updated for new namelist options
Katetc 1bf847d
Addressing review suggestions, changed micro_mg_cam to micro_pumas_cam
Katetc cb1bfb9
Merge remote-tracking branch 'origin/cam_development' into gettelman_…
Katetc 62d75d8
Missed some renaming changes in sp-cam
Katetc 6687a95
Roll back nucleate_ice changes for non-cam_dev configurations, first …
Katetc 3e828b5
Missed an end if
Katetc bbe500b
Change output history variable names to reflect new tendencies
Katetc 8a982ad
Update Outfld calls with new names
Katetc 9a95855
Updated ChangeLog
Katetc 0470e0d
Final updates to ChangeLog
Katetc 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 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
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
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
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
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
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
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,5 +1,110 @@ | ||
| =============================================================== | ||
|
|
||
| Tag name: cam6_3_046 | ||
| Originator(s): katec, andrewgettelman | ||
| Date: 26 Jan 2022 | ||
|
Collaborator
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Don't forget to update. |
||
| One-line Summary: PUMAS v1 release to cam_dev | ||
| Github PR URL:https://github.com/ESCOMP/CAM/pull/409 | ||
|
|
||
| Purpose of changes (include the issue number and title text for each relevant GitHub issue): | ||
|
|
||
| Fixes issue #408 (PUMAS/MG Science updates and bug fixes), and ESCOMP/PUMAS issues #20 and #28 | ||
| 1. Fixes issue Possible removal of ice number limiter....doesn't work with new ice nucleation. | ||
| 2. adds in vapor deposition onto snow as a process | ||
| 3. Fall speed correction for rain/snow/graupel | ||
| 4. Option for implicit fall speed | ||
| 5. Option for the accretion to see newly autoconverted rain (liquid only) | ||
| -- As part of this update to PUMAS we have renamed many micro_mg files and subroutines to | ||
| micro_pumas instead. This occured for the previous pumas code and this new update. | ||
|
|
||
| Describe any changes made to build system: | ||
|
|
||
| The previous code is now in a subdirectory called "pumas-frozen" and the newly released | ||
| pumas_v1 code is in the "pumas" directory, but only available with the cam_dev | ||
| configure option. | ||
|
|
||
| Describe any changes made to the namelist: | ||
|
|
||
| Two new namelist options: micro_mg_accre_sees_auto and micro_mg_implicit_fall. Both are | ||
| set to "true" as default but only available (applicable) in cam_dev pumas | ||
| configurations. | ||
|
|
||
| List any changes to the defaults for the boundary datasets: | ||
|
|
||
| None | ||
|
|
||
| Describe any substantial timing or memory changes: | ||
|
|
||
| None | ||
|
|
||
| Code reviewed by: JulioTBacmeister, nusbaume, gold2718, fvitt, cacraigucar, pverwhee | ||
|
|
||
| List all files eliminated: | ||
| - micro_mg_cam.F90 renamed to micro_pumas_cam.F90 | ||
|
|
||
| List all files added and what they do: | ||
| - micro_pumas_cam.F90 added to src/physics/cam_dev/ | ||
|
|
||
| List all existing files that have been modified, and describe the changes: | ||
| M .gitignore | ||
| - Added a line for moving the preview PUMAS to a new directory in src/physics/pumas-frozen | ||
|
|
||
| M Externals_CAM.cfg | ||
| - Update PUMAS externals tag to pumas_cam-release_v1.22, add | ||
| new external directory for previous PUMAS code in src/physics/pumas-frozen, | ||
| update tag for previous code to branch tag pumas_cam-release_v1.17_rename | ||
|
|
||
| M bld/build-namelist | ||
| - add_default calls for new namelist options (only available to cam_dev physics) | ||
|
|
||
| M bld/configure | ||
| - Build filepath for pumas depends on cam_dev configure option. | ||
| src/physics/pumas build with cam_dev, otherwise src/physics/pumas-frozen | ||
| is used in the build. | ||
|
|
||
| M bld/namelist_files/namelist_defaults_cam.xml | ||
| - New namelist options default to true in cam_dev physics (not applicable otherwise) | ||
|
|
||
| M bld/namelist_files/namelist_definition.xml | ||
| - Define two new namelist options micro_mg_implicit_fall and micro_mg_accre_sees_auto | ||
|
|
||
| M src/physics/cam/macrop_driver.F90 | ||
| - Rename micro_mg to micro_pumas | ||
|
|
||
| M src/physics/cam/micro_pumas_cam.F90 | ||
| - Rename micro_mg to micro_pumas, including micro_mg_tend3_0 -> micro_pumas_tend | ||
|
|
||
| M src/physics/cam/microp_driver.F90 | ||
| - Rename micro_mg to micro_pumas | ||
|
|
||
| M src/physics/cam/nucleate_ice.F90 | ||
| - Remove preexisting_ice | ||
|
|
||
| M src/physics/cam/nucleate_ice_cam.F90 | ||
| - Change number conentrations to number concentration tendencies in history files | ||
|
gold2718 marked this conversation as resolved.
Outdated
|
||
|
|
||
| M src/physics/cam/physpkg.F90 | ||
| - Rename micro_mg to micro_pumas | ||
|
|
||
| M src/physics/cam/zm_microphysics.F90 | ||
| - Rename micro_mg to micro_pumas | ||
|
|
||
| M src/physics/cam_dev/physpkg.F90 | ||
| - Rename micro_mg to micro_pumas | ||
|
|
||
| M src/physics/spcam/crm_physics.F90 | ||
| - Rename micro_mg to micro_pumas | ||
|
|
||
| If there were any failures reported from running test_driver.sh on any test | ||
| platform, and checkin with these failures has been OK'd by the gatekeeper, | ||
| then copy the lines from the td.*.status files for the failed tests to the | ||
| appropriate machine below. All failed tests must be justified. | ||
|
|
||
| cheyenne/intel/aux_cam: | ||
|
gold2718 marked this conversation as resolved.
|
||
|
|
||
|
|
||
| =============================================================== | ||
|
|
||
| Tag name: cam6_3_045 | ||
| Originator(s): fvitt, billsacks | ||
| Date: 19 Jan 2022 | ||
|
|
||
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
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.
Uh oh!
There was an error while loading. Please reload this page.