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

Use 1st eccentricity instead of 2nd eccentricity #1324

Merged
merged 1 commit into from Mar 16, 2019

Conversation

kbevers
Copy link
Member

@kbevers kbevers commented Mar 16, 2019

By mistake the second eccentricity was used in a few places in the
Molodensky transform. According to the literature the first eccentricity
should always be used in the eccentricity.

This only affects the output slightly. Reported test coordinates
differed by less than a millimeter.

Fixes #1321

@kbevers kbevers mentioned this pull request Mar 16, 2019
By mistake the second eccentricity was used in a few places in the
Molodensky transform. According to the literature the first eccentricity
should always be used in the eccentricity.

This only affects the output slightly. Reported test coordinates
differed by less than a millimeter.

Fixes OSGeo#1321
@kbevers
Copy link
Member Author

kbevers commented Mar 16, 2019

Travis fail caused by error in doc generation. Taken care of in https://github.com/OSGeo/proj.4/pull/1325/files

@kbevers kbevers merged commit eab2279 into OSGeo:master Mar 16, 2019
@kbevers kbevers added this to the 6.0.1 milestone Mar 16, 2019
@kbevers kbevers deleted the molodensky-eccentricity branch March 16, 2019 13:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant