Add additional variable check to SCRIP reader

[rajeeja]

Closes #1234

Unstructured SCRIP files may have grid_rank=1 grid_imask and/or grid_area. Some files are found to not have grid_rank and grid_area, so moving to grid_imask

The test file is rather big, so not adding it as our test set is already rather large.

[rajeeja]

fixes #1234

[aaronzedwick]

@rajeeja looks good, glad you figured this out. Just to clarify, grid_imask is in all scrip files?

[rajeeja]

Satisfies the ones we have on repo, and some I found on the ESMF repo

[aaronzedwick]

Satisfies the ones we have on repo, and some I found on the ESMF repo

Would it be possible to add an or statement to check for all 3: grid_rank=1 grid_imask and/or grid_area?

[rajeeja]

Good idea, that can be done and it'll be more transparent, might add a debugging statement saying what was found.

…

_________________________________________ Rajeev

On Fri, May 2, 2025, 7:17 PM Aaron Zedwick ***@***.***> wrote: *aaronzedwick* left a comment (UXARRAY/uxarray#1233) <#1233 (comment)> Satisfies the ones we have on repo, and some I found on the ESMF repo What if you added an or statement to check for all 3: grid_rank=1 grid_imask and/or grid_area? — Reply to this email directly, view it on GitHub <#1233 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AALF6AUGJVTPHF73TO5X4LT24P4GZAVCNFSM6AAAAAB4KWEFE2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDQNBYGI3DEMBWHA> . You are receiving this because you were mentioned.Message ID: ***@***.***>