Skip to content

Conversation

@pcolarco
Copy link
Contributor

This merges two older branches (chem_coupling and photolysis_coupling) that extend capabilities if running with GMI chemistry.

  • one-way coupling of oxidants from GMI to GOCART sulfur chemistry
  • export sulfate surface area density for use in chemistry
  • create callback to allow chemistry to ask for optical properties of aerosols needed for photolysis (GMI and CloudJ)

pcolarco and others added 26 commits May 29, 2025 15:48
…ation of sulfate surface area and effective radius based on optical tables
…st settling call in Process Library accordingly
…e; add dedicated photolysis table entry in state; add hooks to use dedicated photolysis table and start to read legendre coefficients of P11
…ling' into feature/pcolarco/merge_chemistry

Merging chemistry coupling and new photolysis calculator
@pcolarco pcolarco requested a review from a team as a code owner November 18, 2025 17:40
@pcolarco pcolarco added the 0 diff The changes in this pull request have verified to be zero-diff with the target branch. label Nov 18, 2025
vbuchard
vbuchard previously approved these changes Dec 15, 2025
@vbuchard
Copy link
Collaborator

@mathomp4 It looks like the same test is failing as in the other PR. Can you please merge it. Thanks

@mathomp4
Copy link
Member

@mathomp4 It looks like the same test is failing as in the other PR. Can you please merge it. Thanks

This branch was out of date with develop. I've updated the branch. Once things build on circleci, I'll merge in.

@mathomp4
Copy link
Member

@mathomp4 It looks like the same test is failing as in the other PR. Can you please merge it. Thanks

This branch was out of date with develop. I've updated the branch. Once things build on circleci, I'll merge in.

Oh. Wait, the merge just brought in some changelog diffs:

44ab1d5

No need to wait. I'll merge.

@mathomp4 mathomp4 merged commit c96359a into develop Dec 15, 2025
8 of 15 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

0 diff The changes in this pull request have verified to be zero-diff with the target branch.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants