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

Upgrade to geomatics-geotk version 1.17-SNAPSHOT #77

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

Conversation

desruisseaux
Copy link

@desruisseaux desruisseaux commented Feb 27, 2024

This pull request depends on opengeospatial/geomatics-geotk#15 which must be merged first.

Two tests had to be modified:

  • Surface-PolygonPatch-AxisOrder.xml: wrong CRS identifier.
  • LineString-invalidCoords.xml: coordinates were geographic instead of projected, and were not really outside the CRS domain.

There is TestNG failure unrelated to Geotk:

Caused by: java.lang.reflect.InaccessibleObjectException: Unable to make protected final java.lang.Class java.lang.ClassLoader.defineClass(java.lang.String,byte[],int,int,java.security.ProtectionDomain) throws java.lang.ClassFormatError accessible: module java.base does not "opens java.lang" to unnamed module @551bca18
	at org.opengis.cite.iso19136.data.VerifySchematronTests.setUpClass(VerifySchematronTests.java:36)

Two tests had to be modified:

* Surface-PolygonPatch-AxisOrder.xml: wrong CRS identifier.
* LineString-invalidCoords.xml: coordinates were geographic instead of projected, and were not really outside the CRS domain.
@dstenger
Copy link
Contributor

dstenger commented Mar 1, 2024

Thanks for the pull request!
We will check the failure.

@dstenger dstenger added the dependencies Pull requests that update a dependency file label Mar 1, 2024
@dstenger dstenger assigned dstenger and unassigned bpross-52n May 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file
Projects
Status: To verify
Development

Successfully merging this pull request may close these issues.

3 participants