-
Notifications
You must be signed in to change notification settings - Fork 171
cam6_4_112: Fix minor bugs in cloud chemistry #1369
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
cam6_4_112: Fix minor bugs in cloud chemistry #1369
Conversation
modified: src/chemistry/aerosol/aero_convproc.F90
modified: src/chemistry/aerosol/aero_wetdep_cam.F90
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.
Pull Request Overview
This PR fixes several bugs in the cloud chemistry module that were identified during refactoring, improving consistency and accuracy in cloud chemistry calculations. The changes address issues with physical constants, Henry's Law constant parameters, sulfur oxidation rates, and diagnostic outputs.
- Unified Henry's Law constant parameters between wet deposition and cloud chemistry modules
- Replaced hard-coded physical constants with precise values and removed inconsistent air density inputs
- Updated sulfur oxidation rate parameters and reaction filters to prevent over-reaction
Reviewed Changes
Copilot reviewed 8 out of 8 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
src/chemistry/aerosol/mo_setsox.F90 |
Major refactoring to use shared Henry's Law constants, precise physical constants, updated reaction rates, and corrected H2O2 formation calculation |
src/chemistry/modal_aero/sox_cldaero_mod.F90 |
Fixed filtering logic to ensure all relevant species are bounded by small_value |
src/chemistry/modal_aero/aero_model.F90 |
Removed air density parameter and added pbuf parameter to setsox call |
src/chemistry/carma_aero/aero_model.F90 |
Removed air density parameter and added pbuf parameter to setsox call |
src/chemistry/bulk_aero/aero_model.F90 |
Added missing diagnostics and updated setsox call signature |
src/chemistry/aerosol/aero_wetdep_cam.F90 |
Fixed wet deposition diagnostic output logic |
src/chemistry/aerosol/aero_convproc.F90 |
Corrected diagnostic field name for convective wet deposition |
doc/ChangeLog |
Documented the bug fixes and test results |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
cacraigucar
left a comment
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 believe that @fvitt is the proper person to officially review this PR. But while I was skimming through it, I came across a couple of items which I believe should be addressed.
fvitt
left a comment
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.
My initial review.
doc/ChangeLog
Outdated
| mattdawson@derecho3:/glade/derecho/scratch/mattdawson/aux_cam_intel_20250815100530> ./cs.status.aux_cam_intel_20250815100530 | grep -v PASS | grep -v TPUTCOMP | grep -v MEMCOMP | ||
| aux_cam_intel_20250815100530: 51 tests |
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.
These lines should not be copied into the change log.
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.
removed!
| derecho/intel/aux_cam: | ||
| mattdawson@derecho3:/glade/derecho/scratch/mattdawson/aux_cam_intel_20250815100530> ./cs.status.aux_cam_intel_20250815100530 | grep -v PASS | grep -v TPUTCOMP | grep -v MEMCOMP | ||
| aux_cam_intel_20250815100530: 51 tests | ||
| ERC_D_Ln9.f19_f19_mg17.QPC6.derecho_intel.cam-outfrq3s_cosp (Overall: DIFF) details: |
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 believe all of these baseline failures are expected -- configurations that include prognostic aerosols and aqueous chemistry.
doc/ChangeLog
Outdated
| CAM tag used for the baseline comparison tests if different than previous | ||
| tag: cam6_4_101 |
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 believe the regression tests will be re-run and compared against the latest cam tag when this PR is ready to merge -- when @cacraigucar gives the go ahead to merge into cam_development and create the next tag. The test list may change and your new baselines may change after you merge to the head of cam_development.
doc/ChangeLog
Outdated
|
|
||
| Tag name: tbd | ||
| Originator(s): Matt Dawson, Francis Vitt | ||
| Date: August 18, 2025 |
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 date will need to be updated
fvitt
left a comment
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 to me. Just need to update the ChangeLog when ready. Thanks.
|
Budget changes for FCnudged MAM5 (1 year nudged simulation 2016): With bug fix: |
Several minor bugs related to cloud chemistry were identified during refactoring, and addressed here:
closes #1345
@fvitt @tilmes