PolygonExtractionProcess should have more intuitive behaviour when using an ROI 
Geometry
----------------------------------------------------------------------------------------

                 Key: GEOT-3861
                 URL: https://jira.codehaus.org/browse/GEOT-3861
             Project: GeoTools
          Issue Type: Improvement
          Components: process
    Affects Versions: 8.0-M1
            Reporter: Michael Bedward
            Assignee: Michael Bedward
             Fix For: 8.0-M2


The roi argument of PolygonExtractionProcess.execute is used to restrict the 
bounds of the raster to vector conversion. A client passes a Polygon as the roi 
value which is then used to create an ROIGeometry instance in image-space for 
the jaitools Vectorize operation.

The unit test PolygonExtractionProcess.useCoverageBoundsAsROI shows that the 
result can be counter-intuitive (at least to me). In the test, the bounds of 
the input grid coverage are used to create a Polygon which is then used as the 
roi arg. The expected result is a single polygon with bounds equal to the input 
coverage, but the test fails - I think due to the inclusion behaviour of 
ROIGeometry which excludes edges.

I know that Andrea and Daniele spent a lot of time tuning the ROIGeometry class 
so I assume its present behaviour is desired. So perhaps the best way to get 
more intuitive results from the process is for CoverageUtilities to adjust the 
bounds of the roi Polygon, but while that seems straightforward for rectangles 
I'm nervous about introducing other surprises when irregularly shaped Polygons 
are used.

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

------------------------------------------------------------------------------
All of the data generated in your IT infrastructure is seriously valuable.
Why? It contains a definitive record of application performance, security
threats, fraudulent activity, and more. Splunk takes this data and makes
sense of it. IT sense. And common sense.
http://p.sf.net/sfu/splunk-d2dcopy2
_______________________________________________
Geotools-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/geotools-devel

Reply via email to