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

GTIFGetDefn(): add missing normalization of angular units to degree #118

Merged
merged 2 commits into from
Jun 11, 2024

Conversation

rouault
Copy link
Member

@rouault rouault commented Jun 7, 2024

Contrary to what the documentation of GTIFDefn::ProjParm[] mentionned,
we failed to normalize angular measures to degrees when reading them
from projection parameters (ProjXXXXXGeoKey's) when ProjCoordTransGeoKey
was present (but we did normalize them when reading them from the PROJ
database when there were was only a EPSG PCS code)

Relates to OSGeo/gdal#10154 and OSGeo/gdal#10158

rouault added 2 commits June 7, 2024 18:30
Contrary to what the documentation of GTIFDefn::ProjParm[] mentionned,
we failed to normalize angular measures to degrees when reading them
from projection parameters (ProjXXXXXGeoKey's) when ProjCoordTransGeoKey
was present (but we did normalize them when reading them from the PROJ
database when there were was only a EPSG PCS code)

Relates to OSGeo/gdal#10154 and OSGeo/gdal#10158
@rouault rouault added this to the 1.7.4 milestone Jun 7, 2024
rouault added a commit to rouault/gdal that referenced this pull request Jun 7, 2024
rouault added a commit to rouault/gdal that referenced this pull request Jun 7, 2024
rouault added a commit to rouault/gdal that referenced this pull request Jun 7, 2024
rouault added a commit to rouault/gdal that referenced this pull request Jun 8, 2024
@rouault rouault merged commit af121ef into OSGeo:master Jun 11, 2024
7 checks passed
rouault added a commit to OSGeo/gdal that referenced this pull request Jun 11, 2024
rouault added a commit to rouault/gdal that referenced this pull request Jun 11, 2024
rouault added a commit to OSGeo/gdal that referenced this pull request Jun 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant