geosolutions-it / jai-ext

Java Advanced Imaging Open Source Replacement Wannabe
Apache License 2.0
91 stars 37 forks source link

Upgrade to jts-core 1.15 #182

Closed willcohen closed 6 years ago

willcohen commented 6 years ago

While #162 is still unresolved, the jts-example issue mentioned there also currently prevents an upgrade to 1.15 (locationtech/jts#263), since ExtendedCoordinate isn't on maven for 1.15's org.locationtech classes. Does jai-ext need those classes, and if so, should it just bundle a copy?

jnh5y commented 6 years ago

@willcohen As a note about upgrading jai-ext to JTS 1.15, I've got branches for jai-ext, jaitools, GeoTools, GeoWebCache, and GeoServer all hanging out...

I'm interested in sorting out things like this as we go forward with JTS.

aaime commented 6 years ago

@jnh5y you should probably do one (or more) proposals across the various communities to get this started. Commits/merges should be synched to avoid extended periods of the various bits not building.

jodygarnett commented 6 years ago

Coordinating JTS 1.15 upgrade here so we can manage to sync the change across related projects.

simboss commented 6 years ago

Is this still valid for JAI-Ext?

aaime commented 6 years ago

The upgrade of JTS happened already, I'm closing it.

jodygarnett commented 6 years ago

the resolution was not to use extended coordinate sequence, the code in question only used it as a dependency and never made use of the M information (in XYZM)