Bugfix/hack: Always use EPSG:4326 with GeocodedFile to fix crash

[nate-kean]

Description

Adds a line that transforms all bounding boxes the GeocodedFile AOI receives to EPSG:4326.

Motivation and Context

Rest of the code is not properly set up to handle GeocodedFile AOIs that are not in EPSG:4326, which causes problems with EPSG:32617 interferograms like from ASF On Demand. I lack the time to fix this properly, but I did end up doing this much so that RAiDER would work with my code for the gemlab ASF On Demand notebook jlmaurer/gemlab#11. I offer what I have in case you might want to accept it and/or use it as a jumping-off point for a more in-depth solution.

In the case when the user inputs a geocoded file with projection other than EPSG:4326, this change fixes a crash, at the cost of more transformations in this code path, and any precision errors or other drawbacks that may come with it. A more thoughtful solution may be able to avoid doing a conversion here.
In all other cases (other projections, other input AOI types), nothing is changed/affected.

How Has This Been Tested?

Screenshots (if appropriate):

Type of change

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist:

• I have added an explanation of what your changes do and why you'd like us to include them.
• I have written new tests for your core changes, as applicable.
• I have successfully ran tests with your changes locally.
• My change requires a change to the documentation.
• I have updated the documentation accordingly.

[dbekaert]

Hi Nate, Suggest we create/update a ticket where this support is flagged to be added or add reference to this pr change here. If i understand this here refers purely to the bounding box and the export projection as chosen by user is still supported. D.

…

On Mon, Sep 15, 2025 at 8:46 AM Nate Kean ***@***.***> wrote: Description Adds a line that transforms all bounding boxes the GeocodedFile AOI receives to EPSG:4326, if they Motivation and Context Rest of the code is not properly set up to handle GeocodedFile AOIs that are not in EPSG:4326, which causes problems with EPSG:32617 interferograms like from ASF On Demand. I lack the time to fix this properly, but I did end up doing this much so that RAiDER would work with my code for the gemlab ASF On Demand notebook jlmaurer/gemlab#11 <jlmaurer/gemlab#11>. I offer what I have in case you might want to accept it and/or use it as a jumping-off point for a more in-depth solution. In the case when the user inputs a geocoded file with projection other than EPSG:4326, this change fixes a crash, at the cost of more transformations in this code path, and any precision errors or other drawbacks that may come with it. A more thoughtful solution may be able to avoid doing a conversion here. In all other cases (other projections, other input AOI types), nothing is changed/affected. How Has This Been Tested? Screenshots (if appropriate): Type of change - Bug fix (non-breaking change which fixes an issue) - New feature (non-breaking change which adds functionality) - Breaking change (fix or feature that would cause existing functionality to not work as expected) Checklist: - I have added an explanation of what your changes do and why you'd like us to include them. - I have written new tests for your core changes, as applicable. - I have successfully ran tests with your changes locally. - My change requires a change to the documentation. - I have updated the documentation accordingly. ------------------------------ You can view, comment on, or merge this pull request online at: #770 Commit Summary - 179216f <179216f> Hack: always project GeocodedFile to EPSG:4326 File Changes (2 files <https://github.com/dbekaert/RAiDER/pull/770/files>) - *M* tools/RAiDER/llreader.py <https://github.com/dbekaert/RAiDER/pull/770/files#diff-c8a0d3a3ead389f15882b2124f93287cb9fe83f79db304b470cb5feba09c8f10> (7) - *M* tools/RAiDER/utilFcns.py <https://github.com/dbekaert/RAiDER/pull/770/files#diff-9ec6d15bedfbc18ed3e206b4985dd60d44bd30142411041af62724e8fb8b36de> (10) Patch Links: - https://github.com/dbekaert/RAiDER/pull/770.patch - https://github.com/dbekaert/RAiDER/pull/770.diff — Reply to this email directly, view it on GitHub <#770>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AESZPSJLSS7J3XQCYPSX4ID3SZODNAVCNFSM6AAAAACGQKU45GVHI2DSMVQWIX3LMV43ASLTON2WKOZTGQYTMNJUGU4TKMI> . You are receiving this because you are subscribed to this thread.Message ID: ***@***.***>