mbedward / jaitools

Raster image processing for Java developers
17 stars 15 forks source link

ROIGeometry fails to replace ROIShape in a number of methods - #232 #233

Open aaime opened 8 years ago

aaime commented 8 years ago

Some tickets depending on this one: https://osgeo-org.atlassian.net/browse/GEOS-7340 https://osgeo-org.atlassian.net/browse/GEOT-5325 https://osgeo-org.atlassian.net/browse/GEOT-5002

unomar commented 8 years ago

Any word when this will be accepted and merged? I'm now starting to see the issues described in GEOS-7340 on my Linux server with native JAI 1.1.3 installed. Caused by: java.lang.UnsupportedOperationException: Not implemented at org.jaitools.imageutils.ROIGeometry.getAsRectangleList(ROIGeometry.java:488)

aaime commented 8 years ago

Hi, unfortunately I prepared this work in my spare time and never had a chance to test it properly... that would be a few days of work alone...

mbedward commented 8 years ago

Andrea - feel free to merge this if you want, you should have commit access.

I'll contact you separately about moving or retiring the contents of this repo.

aaime commented 8 years ago

Hi @mbedward, thanks! My hesitation right now is primarily about lack of decent testing, like I have some unit tests, but I don't have an environment that would use the methods, so I have no clue if they are any good under common circumstances (nor, presently, the time to follow up on someone that might test them)...

metasim commented 8 years ago

Having problems with GEOS-7340 too, so hoping for this to get merged.