Skip to content

clean up programs etc related to regrid.pl #1057

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

Open
wants to merge 5 commits into
base: develop
Choose a base branch
from

Conversation

gmao-rreichle
Copy link
Contributor

Clean up programs, scripts, etc related to obsolete regrid.pl (replaced with remap_restarts.py in GEOS_Util repo)

Initial commit removes mk_GEOSldasRestarts.F90

More cleanup is probably needed.

Related PRs:
GEOS-ESM/GEOS_Util#113 (pointing to GCM v12 feature branch)

cc: @mathomp4 @weiyuan-jiang @biljanaorescanin

@gmao-rreichle gmao-rreichle added the 0 diff The changes in this pull request have verified to be zero-diff with the target branch. label Feb 12, 2025
@gmao-rreichle gmao-rreichle self-assigned this Feb 12, 2025
@biljanaorescanin
Copy link
Contributor

I've run remap testing package test_remap_restarts.py to check did this affect RST files and it is zero diff.
(NOTE: we did have to rebase tests before this test was done. )

@gmao-rreichle gmao-rreichle marked this pull request as ready for review April 9, 2025 14:38
@gmao-rreichle gmao-rreichle requested review from a team as code owners April 9, 2025 14:38
@sdrabenh sdrabenh added the Contingent - DNA These changes are contingent on other PRs (DNA=do not approve) label Apr 15, 2025
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.

@sdrabenh
Copy link
Collaborator

Added contingent label until GCM v12 is officially released.

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. 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.

5 participants