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

Solved issue deegree CRS #4

Merged
merged 1 commit into from
Jun 13, 2023
Merged

Conversation

manuelaromero-guadaltel
Copy link
Contributor

To solve the validating issues with geometries, the files gmlgeox/src/main/resources/srsconfig/deegree/d3/config/crs-definitions.xml and testdriver-legacy/src/main/resources/srsconfig/deegree/d3/config/crs-definitions.xml has been updated and modified to match the EPSG definitions in the coordinate reference systems approved for use in INSPIRE.

Signed-off-by: manuelaromero-guadaltel <manuelaromero@guadaltel.es>
@manuelaromero-guadaltel
Copy link
Contributor Author

We have updated the definitions of CRS in this release to be consistent with the EPSG guidelines. While the update is not yet complete, we have made corrections to the most commonly used CRS. The reason for this change is that the CRS-definitions.xml file was out of date which led to some issues. To fix these issues, we obtained the latest version from deegree, which fixes these CRS issues that were identified before our meeting with Deegree, established in deegree/deegree3#1480. This solution was suggested during the meeting. However, during testing we found that some of the definitions of CRS did not comply with EPSG guidelines. For this reason, we have made changes in these cases.
The purpose of this file is to check the axis order of the geometries and to determine if it represents a right-handed or left-handed geometry system. Incorrect CRS declarations in this file can lead to erroneous results in the validation of the geometries' orientation.

@jonherrmann jonherrmann merged commit 259858e into etf-validator:next Jun 13, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants