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
Remove proj_def.dat (#201) #1210
Conversation
test/gie/ellipsoid.gie
Outdated
operation proj=merc | ||
accept 0 0 | ||
expect 0 0 | ||
expect failure errno major_axis_not_given |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is this because we now don't have a default ellipsoid?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yes
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I guess that would leave many projections default-less with regards to ellipsoids. It seems to me that +proj=merc
not working without adding +ellps=...
is a step in the wrong direction. I think setting ellipsoid parameters for GRS80
when initialising a PJ
would be a sane default. If +ellps=bessel
(or whatever) is set it will obviously be overridden.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
OK, default to ellps=GRS80 per 6bb14fa
Looks good to me! |
This also fixes a regression recently added in PROJ master now that cs2cs is using underneath the new ISO19111 code to generate pipeline, which doesn't emit +no_defs, and thus cause issues with CRS using LCC_1SP projections that erroneously use the lat_2=45 default of proj_def.dat