Skip to content
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

findsOpsInRegistryWithIntermediate(): restrict to using known source/target CRS that have the same originating authority #4252

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

rouault
Copy link
Member

@rouault rouault commented Sep 17, 2024

Fixes https://lists.osgeo.org/pipermail/proj/2024-September/011531.html
i.e.

projinfo -s EPSG:4269 -t EPSG:6318 --3d --spatial-test intersects

The fix consists in making sure that we recognize that the 3d-promoted object EPSG:4269 (NAD83(86)) is still linked to EPSG, and thus discard ESRI 3D objects in findsOpsInRegistryWithIntermediate()

Fixes a "regression" of #4244 (one could argue which of the results is the best, given that NAD83(86) as a 3D geographic CRS has no solid foundation)

@rouault rouault added the backport 9.5 Backport to 9.5 branch label Sep 17, 2024
@rouault rouault force-pushed the tweak_findsOpsInRegistryWithIntermediate branch from 2afefa0 to 25310e2 Compare September 17, 2024 14:30
…target CRS that have the same originating authority

Fixes https://lists.osgeo.org/pipermail/proj/2024-September/011531.html
i.e.
```
projinfo -s EPSG:4269 -t EPSG:6318 --3d --spatial-test intersects
```

The fix consists in making sure that we recognize that the 3d-promoted
object EPSG:4269 (NAD83(86)) is still linked to EPSG, and thus discard
ESRI 3D objects in findsOpsInRegistryWithIntermediate()

Fixes a "regression" of OSGeo#4244
(one could argue which of the results is the best, given that NAD83(86)
as a 3D geographic CRS has no solid foundation)
@rouault rouault force-pushed the tweak_findsOpsInRegistryWithIntermediate branch from 25310e2 to 78d5d84 Compare September 17, 2024 14:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport 9.5 Backport to 9.5 branch
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant