Skip to content

AERO_PROVIDER can now be GOCART2G, GMICHEM; get SZA from MAPL #6

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
merged 26 commits into from
Apr 24, 2023

Conversation

mmanyin
Copy link
Contributor

@mmanyin mmanyin commented Apr 24, 2023

This PR is non-zero-diff for GMI simulations due to SZA.
Solar zenith angle had been computed locally for Emission and Deposition, but now SZA is
taken from MAPL, as had been done for Photolysis. NOTE: this requires CHEMISTRY release v1.13.1 or later.

Aerosols from GOCART2G are now supported. Also, GMI can provide its own aerosols (via ExtData).

Details:

  • Added capability for AERO_PROVIDER=GOCART2G
  • Added capability for AERO_PROVIDER=GMICHEM
  • Added capability for AERO_PROVIDER=CARMA (uses GMICHEM right now)
  • Changed default aerosdust filename (for case AERO_PROVIDER=GMICHEM)
  • Removed code that might have eventually run GOCART-like aerosols from within the GMI framework (note: doubtful this ever worked and would take significant effort to get working
  • Instead of calling the Gmi routines for SZA (in Chem_Shared), now call a wrapper for the corresponding MAPL routine

Steve Steenrod and others added 26 commits March 1, 2023 09:15
AERO_PROVIDER=GMICHEM and cleaned up non-working code
… wrapper for the MAPL version, in Depos and Emiss (lines in Photolysis became the wrapper routine, which is now in Chem_Shared)
Reset default aerodust filenames (for AERO_PROVIDER=GMI)
…b.com:GEOS-ESM/GMI into feature/vvalenti/gocart2g_to_gmi_connectivity
…connectivity

Feature/vvalenti/gocart2g to gmi connectivity
…3_04_24

Edited CHANGELOG prior to v1.1.0 release
@mmanyin mmanyin added the Non 0-diff The changes in this pull request are non-zero-diff label Apr 24, 2023
@mmanyin mmanyin requested a review from a team as a code owner April 24, 2023 15:08
@mathomp4 mathomp4 merged commit 1d1945c into main Apr 24, 2023
@mathomp4
Copy link
Member

Super powered in due to this version of GMI needing a Chem GC that the CI can't pick up

@mmanyin
Copy link
Contributor Author

mmanyin commented Apr 24, 2023

@mathomp4 Many thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Non 0-diff The changes in this pull request are non-zero-diff
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants