-
Notifications
You must be signed in to change notification settings - Fork 802
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
Add new Conversion "Local Orthographic" #4228
Conversation
When it is ok I can merge the commits in one. |
Outputs of EPSG:10622
|
With the code from master, without this changes, the WKT1_GDAL is like this:
Is the name of the projection |
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.
LGTM !
One point to check though is that projinfo "+proj=ortho +k_0=0.9 +type=crs"
(or with +alpha != 0) instantiates a Local Orthographic projection and not a regular Orthographic.
You may need to fine tune PROJStringParser::Private::buildProjectedCRS() to add a special case, and a test for that in test_io.cpp
I believe we should rather fail. WKT1_GDAL is for what GDAL <= 2.4 and PROJ <= 5.x understood. Although it is not the end of the world if we generate that WKT1... |
Your are right. It is currently creating a |
Doing the test in test_io.cpp, I noticed that the proj string is this
using The documentation I wrote is all with |
This is due to PROJ/src/iso19111/operation/parammappings.cpp Line 311 in eec7351
|
eec7351
to
e35c4a2
Compare
with code 1130 from updated EPSG v11.015 with azimuth and scale factor in proj=ortho projection
e35c4a2
to
221b2d5
Compare
As mentioned in #4226, this "new" conversion (added in EPSG v11.015) is an extension of the Orthographic, extending
proj=ortho
withazimuth
andscale
.Notice that the example in Guidance Note 7 part 2, section 3.6.6 uses the parameters of the CRS
NAD83(2011) / San Francisco SFO B18
. However the parameters are not exactly the same values as in the database forEPSG:10622
.docs/source/*.rst
for new APIFun fact: EPSG:10622 is aligned with the main runway (10L/28R) of San Francisco Airport, going (positive) straight to the east in that local system. The end of the runway is aprox. at coordinates (11740, 0) feet.