Skip to content
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

Bug fix for TOMAS sulfate production rates #1569

Merged
merged 4 commits into from
Apr 7, 2023

Conversation

BettyCroft
Copy link
Contributor

@BettyCroft BettyCroft commented Dec 20, 2022

These updates are a bug fix for TOMAS sulfate production rates as discussed in issue #1511. With the migration of sulfate chemistry to KPP, TOMAS does not have the correct sulfate production rates in v14. Primary updates are to KPP/fullchem/fullchem.eqn and GeosCore/fullchem_mod.F90 and tomas_mod.F90. Two new species are added in fullchem.eqn PSO4AQ and PH2SO4 and KPP was rebuilt. These species are not in the GC restart file - so replace EFYO with CYS for the SPC_ in HEMCO_COnfig.rc for the simulation to run.

@stale
Copy link

stale bot commented Jan 20, 2023

This issue has been automatically marked as stale because it has not had recent activity. If there are no updates within 7 days it will be closed. You can add the "never stale" tag to prevent the Stale bot from closing this issue.

@stale stale bot added the stale No recent activity on this issue label Jan 20, 2023
@BettyCroft
Copy link
Contributor Author

Hi @msulprizio - these updates are ready to be implemented as is. This fix is an important one to enable TOMAS to run correctly with the in-cloud sulfate chemistry migrated into KPP. Can you assist with this update and can we add a never stale label?

@msulprizio msulprizio added never stale Never label this issue as stale and removed stale No recent activity on this issue labels Jan 21, 2023
@msulprizio
Copy link
Contributor

@BettyCroft Sure thing. We plan to implement this shortly after 14.1.0 is released -- it's in the benchmark stages now.

@BettyCroft
Copy link
Contributor Author

Thanks @msulprizio!

@msulprizio msulprizio changed the title Feature/tomaswork Bug fix for TOMAS sulfate production rates Mar 20, 2023
@msulprizio msulprizio added this to the 14.2.0 milestone Mar 20, 2023
Copy link
Contributor

@msulprizio msulprizio left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

These updates will impact TOMAS only. Looks good to merge.

@msulprizio msulprizio changed the base branch from main to dev/14.2.0 April 5, 2023 13:00
@msulprizio msulprizio linked an issue Apr 5, 2023 that may be closed by this pull request
msulprizio added a commit that referenced this pull request Apr 7, 2023
In pull request #1569, PH2SO4 and PSO4AQ were added as species in
fullchem.eqn but were never defined in species_database.yml. This led to
an error when registering species in state_chm_mod.F90

Signed-off-by: Melissa Sulprizio <mpayer@seas.harvard.edu>
@msulprizio msulprizio merged commit 1cf8098 into geoschem:dev/14.2.0 Apr 7, 2023
@BettyCroft
Copy link
Contributor Author

Thank you @msulprizio !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
category: Bug Something isn't working never stale Never label this issue as stale
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Sulfur in-cloud chemistry disconnected from TOMAS in V14 [BUG/ISSUE]
3 participants