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

Fix axis order for urn:ogc:def:crs:EPSG::4269 in crs-definition (#1962) #816

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

@tfr42 tfr42 added the bug error issue and bug (fix) label Mar 5, 2017
@tfr42 tfr42 added this to the 3.4 milestone Mar 5, 2017
@tfr42 tfr42 modified the milestones: 3.4, 3.4.1 Apr 6, 2018
@tfr42 tfr42 added ready and removed in progress labels Apr 8, 2018
@tfr42 tfr42 modified the milestones: 3.4.1, 3.4.2 Jun 18, 2018
Copy link
Member

@tfr42 tfr42 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The deegree way of dealing with axis-awareness is to split the CRS definition into a not axis-aware part (backward compatible) using short ID and a axis-aware definition with the urn-id. Please check https://github.com/deegree/deegree3/wiki/Axis-order-handling.
See for example "epsg:31468" and http://www.opengis.net/gml/srs/epsg.xml#31468 where the crs definition is split.

@tfr42 tfr42 modified the milestones: 3.4.2, 3.4.3 Jul 7, 2018
@tfr42 tfr42 added contributions welcome asking for contribution (time and money sponsor) and removed ready labels Jul 20, 2018
@vog
Copy link
Contributor Author

vog commented Jul 27, 2018

Unfortunately, my time is very constrained, so I won't be able to volunteer into the desired refactoring of the XML files.

If no other volunteer appears, I strongly recommend to include the bugfix now and doing the refactoring later.

@tfr42 tfr42 modified the milestones: 3.4.3, Backlog Sep 28, 2018
@tfr42 tfr42 added the needs rebase PR is not up to date and needs rebase label May 16, 2022
@tfr42 tfr42 added stuck stuck and removed needs rebase PR is not up to date and needs rebase labels Nov 30, 2023
@tfr42
Copy link
Member

tfr42 commented Nov 30, 2023

@vog Thank you for your contribution. Unfortunately, we need more background information before we can merge this PR. Can you provide more information about the use case?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug error issue and bug (fix) contributions welcome asking for contribution (time and money sponsor) stuck stuck
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants