Skip to content

prc: update AMIP volcanic sulfer emissions to v202601 of Carn inventory#409

Open
pcolarco wants to merge 2 commits intodevelopfrom
feature/pcolarco/new_volcanic_emissions
Open

prc: update AMIP volcanic sulfer emissions to v202601 of Carn inventory#409
pcolarco wants to merge 2 commits intodevelopfrom
feature/pcolarco/new_volcanic_emissions

Conversation

@pcolarco
Copy link
Copy Markdown
Contributor

Updates volcanic sulfur emissions to v202601 of Carn database in SU/AMIP configuration only

@pcolarco pcolarco requested a review from a team as a code owner April 15, 2026 20:40
@pcolarco pcolarco added the enhancement New feature or request label Apr 15, 2026
@mathomp4 mathomp4 added the Non 0-diff The changes in this pull request are non-zero-diff label Apr 16, 2026
@mathomp4
Copy link
Copy Markdown
Member

mathomp4 commented Apr 16, 2026

@pcolarco I labeled with non-zero-diff as, well, I assume it is.

ETA: Well, for AMIP :)

Copy link
Copy Markdown
Contributor

@acollow acollow left a comment

Choose a reason for hiding this comment

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

I went to check the filepath to make sure the emissions files can be found and it does not appear that the files have been moved to the official location yet.

Also, shouldn't there be an analogous update for the ops version of the instance file in order to have the newer degassing emissions included for v12?

@pcolarco
Copy link
Copy Markdown
Contributor Author

pcolarco commented Apr 16, 2026 via email

@acollow
Copy link
Copy Markdown
Contributor

acollow commented Apr 16, 2026

Simon's recommendation was to use the most recent year available for degassing rather than a permanent climatology. Based on your experiment results, I would recommend we update as well.

@pcolarco
Copy link
Copy Markdown
Contributor Author

pcolarco commented Apr 16, 2026 via email

@pcolarco
Copy link
Copy Markdown
Contributor Author

pcolarco commented Apr 16, 2026 via email

Copy link
Copy Markdown
Contributor

@acollow acollow left a comment

Choose a reason for hiding this comment

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

Tested and confirmed with both ops and AMIP configurations

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request 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