Skip to content

v2.5: Fix start-stop and missing GMI_HNO3 in NI AMIP ExtData yaml#381

Merged
mathomp4 merged 2 commits intorelease/v2.5from
bugfix/v2.5-fix-ni-yaml
Feb 17, 2026
Merged

v2.5: Fix start-stop and missing GMI_HNO3 in NI AMIP ExtData yaml#381
mathomp4 merged 2 commits intorelease/v2.5from
bugfix/v2.5-fix-ni-yaml

Conversation

@mathomp4
Copy link
Member

Testing with GOCART v2.5.0 found that it fails to run with AMIP emissions. The culprit was GMI_HNO3 which was added to NI2G_GridComp_ExtData.yaml in #373 but it wasn't added to AMIP/NI2G_GridComp_ExtData.yaml

This PR adds it.

@mathomp4 mathomp4 self-assigned this Feb 13, 2026
@mathomp4 mathomp4 requested a review from a team as a code owner February 13, 2026 16:54
@mathomp4 mathomp4 added the 0 diff The changes in this pull request have verified to be zero-diff with the target branch. label Feb 13, 2026
vbuchard
vbuchard previously approved these changes Feb 13, 2026
Copy link
Collaborator

@vbuchard vbuchard left a comment

Choose a reason for hiding this comment

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

Thanks @mathomp4

@mathomp4 mathomp4 changed the title v2.5: Fix missing GMI_HNO3 in NI AMIP ExtData yaml v2.5: Fix start-stop and missing GMI_HNO3 in NI AMIP ExtData yaml Feb 17, 2026
@mathomp4 mathomp4 marked this pull request as ready for review February 17, 2026 18:38
@mathomp4 mathomp4 requested a review from vbuchard February 17, 2026 18:48
Copy link
Collaborator

@vbuchard vbuchard left a comment

Choose a reason for hiding this comment

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

Thanks Matt!

@mathomp4 mathomp4 merged commit 30a6d5b into release/v2.5 Feb 17, 2026
25 checks passed
@mathomp4 mathomp4 deleted the bugfix/v2.5-fix-ni-yaml branch February 17, 2026 19:59
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.

2 participants