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

Update to EPSG 9.6.2 #1462

Merged
merged 3 commits into from May 8, 2019

Conversation

Projects
None yet
2 participants
@rouault
Copy link
Member

commented May 8, 2019

No description provided.

rouault added some commits May 8, 2019

@rouault rouault added the backport 6.1 label May 8, 2019

@kbevers

This comment has been minimized.

Copy link
Member

commented May 8, 2019

Are you confident that this will not introduce new problems downstream?

@rouault

This comment has been minimized.

Copy link
Member Author

commented May 8, 2019

Are you confident that this will not introduce new problems downstream?

yes, at 99%. The updates are very minor: 2 grid names changed for Australia (none of which we actually ship in proj-datumgrid-oceania), and a fix for the oddity I noticed around Costa Rica Helmert transformations.
The biggest change is in scripts/build_db.py which is only used when refreshing the database manually like I did (that change should have been done previously, and is independent, but needed, to do the update)
So, nothing critical and we could as well restrict that to master if you prefer to exercise your right in saying "no" :-)

@kbevers

This comment has been minimized.

Copy link
Member

commented May 8, 2019

So, nothing critical and we could as well restrict that to master if you prefer to exercise your right in saying "no" :-)

It did indeed cross my mind :) I think we will be okay to include this on.

@kbevers kbevers merged commit a33a236 into OSGeo:master May 8, 2019

4 checks passed

FreeBSD Task Summary
Details
Travis CI - Pull Request Build Passed
Details
continuous-integration/appveyor/pr AppVeyor build succeeded
Details
coverage/coveralls Coverage decreased (-0.003%) to 85.861%
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.