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

Map the Behrmann projection to cae when converting ESRI CRSes #1986

Merged
merged 2 commits into from Mar 4, 2020

Conversation

nyalldawson
Copy link
Contributor

Adds the capability to force a fixed value for a non-specified CRS component while mapping ESRI projections, and maps the Behrman projection to cae with lat_ts=30, lon_0=0

I had to add the new "is_fixed_value" property for ESRIParamMapping in order to differentiate parameters which are specified in the ESRI WKT definition and have an expected value Vs parameters which are NOT included in the ESRI WKT definition and have to have a fixed value (such as the Standard_Parallel_1 parameter for Behrman, which is not part of the CRS definition but has to have the value of 30 so that the proj string gets the required +lat_ts=30 parameter)

@nyalldawson nyalldawson changed the title Map the Behrman projection to cae when coverting ESRI CRSes Map the Behrman projection to cae when converting ESRI CRSes Feb 26, 2020
@kbevers kbevers modified the milestones: 7.0.1, 7.1.0 Feb 26, 2020
@rouault
Copy link
Member

rouault commented Feb 26, 2020

A test would be welcome. Presumably as a new entry in the esriProjDefs[] array of test/unit/test_io.cpp

while mapping ESRI projections, and map the Behrman projection to
cae with lat_ts=30, lon_0=0
@nyalldawson
Copy link
Contributor Author

@rouault I've just added a bunch of missing tests

@rouault rouault merged commit c1fbdd8 into OSGeo:master Mar 4, 2020
@mwtoews mwtoews changed the title Map the Behrman projection to cae when converting ESRI CRSes Map the Behrmann projection to cae when converting ESRI CRSes May 17, 2021
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.

None yet

3 participants