Michaël has the data through private mail and is working on. SourceForge mail 
did not accept zipped JML as an attachment but actually the case rather 
interesting so I try to create a new issue and attach JML that way for future 
reference.

-Jukka-

-----Alkuperäinen viesti-----
Lähettäjä: edgar.sol...@web.de [mailto:edgar.sol...@web.de] 
Lähetetty: 3. huhtikuuta 2017 12:35
Vastaanottaja: jump-pilot-devel@lists.sourceforge.net
Aihe: Re: [JPP-Devel] Error from Make valid with OpeJUMP 1.10

Jukka, Mike,

obviously two issues here. Jukka's original one

>>> java.lang.IllegalArgumentException: This method does not support 
>>> GeometryCollection arguments
>>>       at 
>>> com.vividsolutions.jts.geom.Geometry.checkNotGeometryCollection(Geometry.java:1781)
>>>       at 
>>> com.vividsolutions.jts.geom.Geometry.symDifference(Geometry.java:1453)
>>>       at 
>>> com.vividsolutions.jump.geom.MakeValidOp.nodePolygon(MakeValidOp.jav
>>> a:404)

and the new issue of noding differently.

my guess ist that the original dataset for some reason contains or is 
interpreted as GeomCollections. Jukka, can you provide a *.jml file for the 
dataset and check if throws the same error when reloaded into the latest OJ 
snapshot?

..ede

On 02.04.2017 23:25, Michaël Michaud wrote:
> Hi Jukka,
> 
> With your sample, I get no error, but the polygon is not fixed correctly.
> 
> I wonder why we do not get te same result though
> 
>   
> 
> 
> Le 02/04/2017 à 22:21, Rahkonen Jukka (MML) a écrit :
>> Hi,
>>
>> Same error with the latest NB. Should be possible to re-produce with this 
>> polygon:
>>
>> POLYGON (( 270704.9999964252 4297754.99988556, 270734.99999642686 
>> 4297754.999885563, 270734.9999964269 4297724.99988556, 
>> 270764.9999964286 4297724.999885563, 270764.9999964286 
>> 4297694.999885565, 270794.99999643036 4297694.999885567, 
>> 270794.9999964305 4297604.999885567, 270824.999996432 
>> 4297604.999885569, 270824.9999964321 4297574.99988557, 
>> 270794.9999964305 4297574.999885569, 270794.9999964305 
>> 4297514.99988557, 270824.9999964322 4297514.999885572, 
>> 270824.9999964321 4297574.99988557, 270854.9999964338 
>> 4297574.999885572, 270854.99999643385 4297514.999885574, 
>> 270884.9999964356 4297514.999885576, 270884.9999964356 
>> 4297484.999885576, 270854.99999643397 4297484.999885574, 
>> 270854.99999643397 4297424.999885575, 270824.9999964322 
>> 4297424.999885573, 270824.99999643234 4297454.999885572, 
>> 270734.99999642733 4297454.9998855665, 270734.9999964272 
>> 4297484.9998855665, 270704.9999964256 4297484.999885564, 
>> 270704.9999964255 4297544.999885563, 270674.9999964237 
>> 4297544.999885561, 270674.99999642384 4297604.999885561, 
>> 270644.9999964221 4297604.99988556, 270644.99999642203 
>> 4297634.999885559, 270614.99999642035 4297634.999885556, 
>> 270614.9999964203 4297664.999885556, 270584.99999641866 
>> 4297664.9998855535, 270584.9999964186 4297724.999885553, 
>> 270674.9999964236 4297724.999885557, 270674.9999964236 
>> 4297694.99988556, 270704.99999642535 4297694.999885561, 
>> 270704.9999964252 4297754.99988556 ), ( 270674.9999964237 
>> 4297664.99988556, 270674.99999642366 4297634.99988556, 
>> 270704.99999642535 4297634.999885562, 270704.99999642535 
>> 4297664.999885562, 270674.9999964237 4297664.99988556 ))
>>
>> -Jukka-
>>
>> ________________________________________
>> Lähettäjä: Michaël Michaud <m.michael.mich...@orange.fr>
>> Lähetetty: 2. huhtikuuta 2017 23:07
>> Vastaanottaja: jump-pilot-devel@lists.sourceforge.net
>> Aihe: Re: [JPP-Devel] Error from Make valid with OpeJUMP 1.10
>>
>> Hi Jukka,
>>
>> I have made a correction since 1.10 which was about simple invalid 
>> polygon having a linear spike.
>> Can you try with last NB and/or send me your polygon (file or WKT)
>>
>> Thanks,
>>
>> Michaël
>>
>>
>> Le 02/04/2017 à 21:46, Rahkonen Jukka (MML) a écrit :
>>> Hi,
>>>
>>> I was trying to make a polygon layer  with some self-intersecting polygons 
>>> valid with OpenJUMP 1.10 but I am getting this error:
>>>
>>>
>>> java.lang.IllegalArgumentException: This method does not support 
>>> GeometryCollection arguments
>>>       at 
>>> com.vividsolutions.jts.geom.Geometry.checkNotGeometryCollection(Geometry.java:1781)
>>>       at 
>>> com.vividsolutions.jts.geom.Geometry.symDifference(Geometry.java:1453)
>>>       at 
>>> com.vividsolutions.jump.geom.MakeValidOp.nodePolygon(MakeValidOp.java:404)
>>>       at 
>>> com.vividsolutions.jump.geom.MakeValidOp.makePolygonValid(MakeValidOp.java:325)
>>>       at 
>>> com.vividsolutions.jump.geom.MakeValidOp.makeValid(MakeValidOp.java:123)
>>>       at 
>>> org.openjump.core.ui.plugin.tools.MakeValidPlugIn.createNewLayer(MakeValidPlugIn.java:242)
>>>       at 
>>> org.openjump.core.ui.plugin.tools.MakeValidPlugIn.run(MakeValidPlugIn.java:141)
>>>       at 
>>> com.vividsolutions.jump.workbench.ui.task.TaskMonitorManager$TaskWrapper.run(TaskMonitorManager.java:152)
>>>       at java.lang.Thread.run(Unknown Source)
>>>
>>> All features on that layer are simple one part polygons without even holes.
>>>
>>> -Jukka Rahkonen-
>>> --------------------------------------------------------------------
>>> ---------- Check out the vibrant tech community on one of the 
>>> world's most engaging tech sites, Slashdot.org! 
>>> http://sdm.link/slashdot 
>>> _______________________________________________
>>> Jump-pilot-devel mailing list
>>> Jump-pilot-devel@lists.sourceforge.net
>>> https://lists.sourceforge.net/lists/listinfo/jump-pilot-devel
>>>
>> ---------------------------------------------------------------------
>> --------- Check out the vibrant tech community on one of the world's 
>> most engaging tech sites, Slashdot.org! http://sdm.link/slashdot 
>> _______________________________________________
>> Jump-pilot-devel mailing list
>> Jump-pilot-devel@lists.sourceforge.net
>> https://lists.sourceforge.net/lists/listinfo/jump-pilot-devel
>>
>> ---------------------------------------------------------------------
>> --------- Check out the vibrant tech community on one of the world's 
>> most engaging tech sites, Slashdot.org! http://sdm.link/slashdot 
>> _______________________________________________
>> Jump-pilot-devel mailing list
>> Jump-pilot-devel@lists.sourceforge.net
>> https://lists.sourceforge.net/lists/listinfo/jump-pilot-devel
>>
> 
> 
> 
> ----------------------------------------------------------------------
> -------- Check out the vibrant tech community on one of the world's 
> most engaging tech sites, Slashdot.org! http://sdm.link/slashdot
> 
> 
> 
> _______________________________________________
> Jump-pilot-devel mailing list
> Jump-pilot-devel@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/jump-pilot-devel
> 
------------------------------------------------------------------------------
Check out the vibrant tech community on one of the world's most engaging tech 
sites, Slashdot.org! http://sdm.link/slashdot 
_______________________________________________
Jump-pilot-devel mailing list
Jump-pilot-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/jump-pilot-devel

------------------------------------------------------------------------------
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot
_______________________________________________
Jump-pilot-devel mailing list
Jump-pilot-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/jump-pilot-devel

Reply via email to