mapsforge / vtm

OpenGL vector map library - running on Android, iOS, Desktop and browser.
GNU Lesser General Public License v3.0
269 stars 89 forks source link

JTS (LocationTech) update #738

Closed sunshine0576 closed 5 years ago

sunshine0576 commented 5 years ago

JTS (LocationTech) current version is 1.16.1,When to update it?

devemux86 commented 5 years ago

JTS 1.16+ seems to require Java 8 and Android 7 Nougat (API 24)?

D8: Default interface methods are only supported starting with Android N (--min-api 24):
org.locationtech.jts.geom.Coordinate org.locationtech.jts.geom.CoordinateSequence.createCoordinate()

See also https://github.com/locationtech/jts/issues/145

CodeK1988 commented 1 year ago

JTS (LocationTech) current version is 1.19.0,When to update it?

devemux86 commented 1 year ago

We may see this in the future as min Android increases.

VTM works fine with current JTS, why need an update?

CodeK1988 commented 1 year ago

because OffsetCurveBuilder getOffsetCurve not working. works fine when distance is set to 100, has bug when set to 1000. ocb.getOffsetCurve(lineDrawable.getGeometry().getCoordinates(), -100 / NEARNESS_FACTOR); working. ocb.getOffsetCurve(lineDrawable.getGeometry().getCoordinates(), -10000 / NEARNESS_FACTOR); have bug. @devemux86

by the way, private static final int NEARNESS_FACTOR = 100000;

devemux86 commented 1 year ago

You can exclude VTM transitive dependencies in application Gradle and use your preferred versions.

CodeK1988 commented 1 year ago

@devemux86 not working, https://github.com/wdtinc/mapbox-vector-tile-java/issues/48 get this. by the way,thank you reply. mate

devemux86 commented 1 year ago

Dependencies are not updated without extensive testing and they must update / work together. More important is not break Android min SDK and keep compatibility with older Android / Java.

VTM uses JTS in vtm-jts for overlays, which is a core feature of the library and must work.

Mapbox vector tiles in vtm-mvt module use 3rd-party mapbox-vector-tile lib (using JTS).