ROIGeometry improvements

mbedward commented 11 years ago

Original author: andrea.a...@gmail.com (October 01, 2011 08:38:15)

This patch improves ROIGeometry and VectorBinarize in several ways:

improves thread safety (JAI tiles can be computed in parallel)
gets rid of JTS shape/geometry wrappers and backports LiteShape. We found JTS wrappers arbitrarily alter coordinates, rounding them to integer, and do not implement some operations. LiteShape and friends was created by me, I authorize the license transfer to jaitools
allows to specify the RenderingHints for ROIGeometry, which will be used when turning it into an image via VectoBinarize (this allows to make sure all the system is using the same user specified tile caches)
when doing set oriented operations on geometries it cleans up the results, getting rid of the spurious points and lines that might result when doing intersections/subtractions

The patch also contains a fix for PolygonSmootherTest, so that it can pass also on a machine with italian locale (we use comma instead of dot as the decimal separator)

Original issue: http://code.google.com/p/jaitools/issues/detail?id=200

mbedward commented 11 years ago

From michael.bedward@gmail.com on October 05, 2011 12:07:08 Many thanks for this patch Andrea and my apologies for not getting to it before now.

I've just applied it to my local trunk sources and all went well. I'll apply it to the 1.2.x sources tomorrow with a view to releasing a 1.2.1 update.

mbedward commented 11 years ago

From michael.bedward@gmail.com on October 11, 2011 11:21:02 As per IRC, we'll just commit this to trunk now, aiming for a 1.3 release later this month. I've moved the LiteShape and associated iterator classes into an imageutils.shape package and used the iterator package for the image iterators. If that creates any probs let me know.

Thanks again. Great to have these changes in jaitools.

mbedward / jaitools

ROIGeometry improvements #198