-
Notifications
You must be signed in to change notification settings - Fork 7
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 aerocom namelist #35
new aerocom namelist #35
Conversation
mvertens
commented
Aug 28, 2024
- remove AEROCOM as a cpp variable - and introduce a new namelist to turn it on
- add a new file oslo_aero_diagnostics.F90 that activates oslo_aero (and aercom namelists) and is called from cam_diagnostics.F90. This makes it much easier to modularize where the oslo-aero diagnostics fields are defined.
- verified that running SMS_Ln9.ne30pg3_ne30pg3_mtn14.NF1850.derecho_intel.cam-outfrq9s with use_aerocom=.false. have bfb the same history files
- verified that running SMS_Ln9.ne30pg3_ne30pg3_mtn14.NF1850.derecho_intel.cam-outfrq9s with use_aerocom=.true. added the aerocom variables.
- a question about the new aerocom output appears in Diagnostic changes with AEROCOM noresm3_dev_simulations#33
- NOTE: this PR must also be accompanied my PRs to https://github.com/NorESMhub/CAM and to https://github.com/NorESMhub/CAM-Nor-physics
a35a70b
to
0db5173
Compare
Add OSLO Aero diagnostics to physpkg
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.
Namelist variables should be protected
.
Several files have long lines that need to be trimmed (#36).
src/oslo_aero_control.F90
Outdated
! Namelist variables: | ||
real(r8) :: volc_fraction_coarse = 0.0_r8 !Fraction of volcanic aerosols in coarse mode | ||
! Public Namelist variables: | ||
logical, public :: use_aerocom = .false. ! If true, turn on aerocom output |
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.
This should be protected
src/oslo_aero_diagnostics.F90
Outdated
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 fields with unitless
units should be updated to something descriptive (see #37).
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.
Looks good now.