Skip to content

Conversation

@mathomp4
Copy link
Member

@mathomp4 mathomp4 commented Oct 9, 2025

GEOSgcm_GridComp still has the old ARIES and LatLon FV cores. These are dead codes. Let's remove them.

One of a series:

@mathomp4 mathomp4 requested a review from wmputman October 9, 2025 14:23
@mathomp4 mathomp4 self-assigned this Oct 9, 2025
@mathomp4 mathomp4 requested review from a team as code owners October 9, 2025 14:23
@mathomp4 mathomp4 added 0 diff The changes in this pull request have verified to be zero-diff with the target branch. 0 diff structural Structural changes to repository that are zero-diff labels Oct 9, 2025
@mathomp4 mathomp4 changed the title v12: Remove ARIES and LatLot dynamics GCs v12: Remove ARIES and LatLon dynamics GCs Nov 24, 2025
@mathomp4
Copy link
Member Author

Sigh. It looks like some bits of the model need it. I'll close for now.

@mathomp4 mathomp4 closed this Nov 24, 2025
@mathomp4 mathomp4 deleted the feature/v12-remove-old-dyn branch November 24, 2025 20:19
@mathomp4 mathomp4 restored the feature/v12-remove-old-dyn branch December 18, 2025 15:18
@mathomp4 mathomp4 reopened this Dec 18, 2025
@github-actions
Copy link

This PR is being prevented from merging because you have added one of our blocking labels: Contingent - DNA, Needs Lead Approval, Contingent -- Do Not Approve. You'll need to remove it before this PR can be merged.

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

Labels

0 diff structural Structural changes to repository that are zero-diff 0 diff The changes in this pull request have verified to be zero-diff with the target branch. Contingent - DNA These changes are contingent on other PRs (DNA=do not approve)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant