Modify

Opened 13 years ago

Closed 13 years ago

#8138 closed defect (duplicate)

IllegalPathStateException when running validator

Reported by: wmgcnfg@… Owned by: team
Priority: normal Milestone:
Component: Core validator Version: tested
Keywords: template_report Cc:

Description (last modified by skyper)

What steps will reproduce the problem?

  1. Load some areas with the remote plugin containing ways with one node only (http://tools.geofabrik.de/osmi/ => Geometry => Way nodes => Single node in way). I think it happens more often if the areas are spread over a very large area (e.g. europe)
  2. Run validator

What is the expected result?
The validator should show errors (lots of ways with a single node) and warnings

What happens instead?
The attached exception occurs

Please provide any additional information below. Attach a screenshot if
possible.

Repository Root: http://josm.openstreetmap.de/svn
Build-Date: 2012-10-07 01:31:07
Last Changed Author: stoecker
Revision: 5531
Repository UUID: 0c6e7542-c601-0410-84e7-c038aed88b3b
URL: http://josm.openstreetmap.de/svn/trunk
Last Changed Date: 2012-10-06 16:58:02 +0200 (Sat, 06 Oct 2012)
Last Changed Rev: 5531

Identification: JOSM/1.5 (5531 de)
Memory Usage: 418 MB / 878 MB (324 MB allocated, but free)
Java version: 1.7.0_09, Oracle Corporation, Java HotSpot(TM) 64-Bit Server VM
Operating system: Windows 7

Dataset consistency test:
[WARN - NO NODES] Way {Way id=144259067 version=2 VT nodes=[{Node id=1496455608 version=2 V lat=45.7010112,lon=12.0168333}]} has only one node
[WARN - NO NODES] Way {Way id=183160656 version=1 V nodes=[{Node id=1935222170 version=1 V lat=45.9840623,lon=12.3297446}]} has only one node
[WARN - NO NODES] Way {Way id=180139824 version=1 VT nodes=[{Node id=1905712004 version=1 V lat=45.7011172,lon=11.9282912}]} has only one node
[WARN - NO NODES] Way {Way id=171713174 version=1 V nodes=[{Node id=1594447263 version=2 V lat=45.3465017,lon=12.3166041}]} has only one node
[WARN - NO NODES] Way {Way id=181736750 version=2 V nodes=[{Node id=1921700900 version=2 V lat=46.3653009,lon=11.8938847}]} has only one node
[WARN - NO NODES] Way {Way id=175754603 version=1 VT nodes=[{Node id=1594845995 version=1 V lat=45.7280059,lon=11.8863548}]} has only one node
[WARN - NO NODES] Way {Way id=164919773 version=1 V nodes=[{Node id=1765144894 version=1 V lat=46.1806735,lon=11.668759}]} has only one node
[WARN - NO NODES] Way {Way id=176937656 version=1 VT nodes=[{Node id=1874172589 version=1 V lat=45.7488164,lon=11.8802759}]} has only one node
[WARN - NO NODES] Way {Way id=180180099 version=1 V nodes=[{Node id=803790103 version=1 V lat=46.2692435,lon=12.0351694}]} has only one node
[WARN - NO NODES] Way {Way id=176246341 version=1 V nodes=[{Node id=31318854 version=4 V lat=45.4306667,lon=12.3568627}]} has only one node
[WARN - NO NODES] Way {Way id=180139806 version=1 VT nodes=[{Node id=1905712004 version=1 V lat=45.7011172,lon=11.9282912}]} has only one node
[WARN - NO NODES] Way {Way id=94629706 version=4 V nodes=[{Node id=1068080462 version=2 V lat=45.4442923,lon=12.3240168}]} has only one node
[WARN - NO NODES] Way {Way id=180139839 version=1 VT nodes=[{Node id=1905712535 version=1 V lat=45.7060984,lon=11.9323618}]} has only one node
[WARN - NO NODES] Way {Way id=129210815 version=2 VT nodes=[{Node id=278538146 version=5 V lat=45.661369,lon=12.2842438}]} has only one node
[WARN - NO NODES] Way {Way id=181446444 version=1 V nodes=[{Node id=1521925852 version=1 V lat=46.1427133,lon=12.2196215}]} has only one node
[WARN - NO NODES] Way {Way id=180180097 version=1 V nodes=[{Node id=1906111642 version=1 V lat=46.2689468,lon=12.0357804}]} has only one node
[WARN - NO NODES] Way {Way id=175761058 version=1 V nodes=[{Node id=1863142430 version=1 V lat=45.4357118,lon=12.2061676}]} has only one node
[WARN - NO NODES] Way {Way id=184641969 version=1 V nodes=[{Node id=1951422924 version=1 V lat=46.0303645,lon=11.2912993}]} has only one node
[WARN - NO NODES] Way {Way id=139003624 version=3 VT nodes=[{Node id=1523813228 version=1 V lat=45.5616148,lon=11.5334978}]} has only one node
[WARN - NO NODES] Way {Way id=144259071 version=2 VT nodes=[{Node id=537186917 version=2 V lat=45.7011341,lon=12.0176979}]} has only one node
[WARN - NO NODES] Way {Way id=150483706 version=2 VT nodes=[{Node id=1633905201 version=2 V lat=45.8244072,lon=13.3340365}]} has only one node
[WARN - NO NODES] Way {Way id=148839652 version=2 VT nodes=[{Node id=1619032375 version=1 V lat=45.7944236,lon=12.1671306}]} has only one node
[WARN - NO NODES] Way {Way id=136424659 version=3 VT nodes=[{Node id=1496455608 version=2 V lat=45.7010112,lon=12.0168333}]} has only one node
[WARN - NO NODES] Way {Way id=184067011 version=2 VT< nodes=[{Node id=1944955123 version=1 V lat=45.5258899,lon=12.7162084}]} has only one node
[WARN - NO NODES] Way {Way id=169508660 version=1 V nodes=[{Node id=1806691292 version=1 V lat=45.6588908,lon=12.2597563}]} has only one node
[WARN - NO NODES] Way {Way id=180139827 version=1 VT nodes=[{Node id=1905712004 version=1 V lat=45.7011172,lon=11.9282912}]} has only one node
[WARN - NO NODES] Way {Way id=170648180 version=1 V nodes=[{Node id=1818071630 version=1 V lat=45.7835274,lon=12.492858}]} has only one node
[WARN - NO NODES] Way {Way id=180139818 version=1 VT nodes=[{Node id=1905712004 version=1 V lat=45.7011172,lon=11.9282912}]} has only one node
[WARN - NO NODES] Way {Way id=88226849 version=2 VT nodes=[{Node id=1025067642 version=2 V lat=45.4244963,lon=12.3259819}]} has only one node
[WARN - NO NODES] Way {Way id=166898840 version=1 V nodes=[{Node id=1783159447 version=3 VT lat=45.6507342,lon=11.640937}]} has only one node
[WARN - NO NODES] Way {Way id=175754605 version=1 VT nodes=[{Node id=1594845995 version=1 V lat=45.7280059,lon=11.8863548}]} has only one node
[WARN - NO NODES] Way {Way id=180139801 version=1 VT nodes=[{Node id=1905712004 version=1 V lat=45.7011172,lon=11.9282912}]} has only one node
[WARN - NO NODES] Way {Way id=91063208 version=7 VT nodes=[{Node id=469845638 version=4 V lat=46.0157228,lon=13.1071025}]} has only one node
[WARN - NO NODES] Way {Way id=180139812 version=1 VT nodes=[{Node id=1905712004 version=1 V lat=45.7011172,lon=11.9282912}]} has only one node
[WARN - NO NODES] Way {Way id=180139838 version=1 VT nodes=[{Node id=1905712535 version=1 V lat=45.7060984,lon=11.9323618}]} has only one node
[WARN - NO NODES] Way {Way id=168412396 version=1 V nodes=[{Node id=1796841869 version=1 V lat=45.5693854,lon=11.7294448}]} has only one node
[WARN - NO NODES] Way {Way id=180139825 version=1 VT nodes=[{Node id=1905712004 version=1 V lat=45.7011172,lon=11.9282912}]} has only one node
[WARN - NO NODES] Way {Way id=174757752 version=1 V nodes=[{Node id=1854234302 version=1 V lat=45.4423625,lon=12.3417429}]} has only one node
[WARN - NO NODES] Way {Way id=168404937 version=1 V nodes=[{Node id=339836523 version=1 V lat=46.1296564,lon=13.2269051}]} has only one node
[WARN - NO NODES] Way {Way id=18831137 version=11 V nodes=[{Node id=1821987831 version=1 V lat=45.4335573,lon=12.3450032}]} has only one node
[WARN - NO NODES] Way {Way id=170232287 version=1 V nodes=[{Node id=1813866970 version=1 V lat=45.4617336,lon=12.1171685}]} has only one node
[WARN - NO NODES] Way {Way id=176937657 version=1 VT nodes=[{Node id=1874172589 version=1 V lat=45.7488164,lon=11.8802759}]} has only one node
[WARN - NO NODES] Way {Way id=182593398 version=1 V nodes=[{Node id=1929484559 version=1 V lat=45.756715,lon=12.8038572}]} has only one node
[WARN - NO NODES] Way {Way id=177735971 version=1 VT> nodes=[{Node id=1881665032 version=1 V lat=46.0865521,lon=12.2882185}]} has only one node
[WARN - NO NODES] Way {Way id=170483373 version=1 V nodes=[{Node id=1816234028 version=1 V lat=45.8830989,lon=11.0134209}]} has only one node
[WARN - NO NODES] Way {Way id=183360438 version=1 V nodes=[{Node id=1937314112 version=1 V lat=45.8776142,lon=12.53834}]} has only one node
[WARN - NO NODES] Way {Way id=184710750 version=1 VT nodes=[{Node id=1952168219 version=1 V lat=45.8180069,lon=11.8591136}]} has only one node
[WARN - NO NODES] Way {Way id=175754606 version=1 VT nodes=[{Node id=1594845995 version=1 V lat=45.7280059,lon=11.8863548}]} has only one node
[WARN - NO NODES] Way {Way id=175856826 version=2 VT nodes=[{Node id=1311769443 version=3 V lat=45.4445422,lon=12.1425051}]} has only one node
[WARN - NO NODES] Way {Way id=177275808 version=1 V nodes=[{Node id=1877330639 version=1 V lat=46.4543237,lon=12.024022}]} has only one node
[WARN - NO NODES] Way {Way id=184131607 version=1 V nodes=[{Node id=1945796537 version=1 V lat=45.5281942,lon=12.697133}]} has only one node
[WARN - NO NODES] Way {Way id=180139826 version=1 VT nodes=[{Node id=1905712004 version=1 V lat=45.7011172,lon=11.9282912}]} has only one node
[WARN - NO NODES] Way {Way id=167901986 version=1 V nodes=[{Node id=1792384791 version=1 V lat=45.7604542,lon=11.4311653}]} has only one node
[WARN - NO NODES] Way {Way id=167901982 version=1 VT nodes=[{Node id=387410762 version=5 VT lat=45.7627621,lon=11.4359428}]} has only one node
[WARN - NO NODES] Way {Way id=164759537 version=1 V nodes=[{Node id=1763628994 version=1 V lat=45.9741911,lon=11.9841013}]} has only one node
[WARN - NO NODES] Way {Way id=172446593 version=1 V nodes=[{Node id=1833519383 version=1 V lat=45.971539,lon=12.3105015}]} has only one node
[WARN - NO NODES] Way {Way id=175754604 version=1 VT nodes=[{Node id=1594845995 version=1 V lat=45.7280059,lon=11.8863548}]} has only one node
[WARN - NO NODES] Way {Way id=148838608 version=2 VT nodes=[{Node id=1619031362 version=2 V lat=45.7938867,lon=12.1673801}]} has only one node
[WARN - NO NODES] Way {Way id=180139800 version=1 VT nodes=[{Node id=1905712004 version=1 V lat=45.7011172,lon=11.9282912}]} has only one node
[WARN - NO NODES] Way {Way id=180139815 version=1 VT nodes=[{Node id=1905712004 version=1 V lat=45.7011172,lon=11.9282912}]} has only one node
[WARN - NO NODES] Way {Way id=167069527 version=2 VT nodes=[{Node id=1784765138 version=1 V lat=45.5871002,lon=11.6235748}]} has only one node
[WARN - NO NODES] Way {Way id=96687831 version=3 VT nodes=[{Node id=699994081 version=3 V lat=46.206848,lon=13.0309607}]} has only one node
[WARN - NO NODES] Way {Way id=134892162 version=2 VT nodes=[{Node id=1482224009 version=2 V lat=45.6320965,lon=11.7566403}]} has only one node


Plugin: FixAddresses (28656)
Plugin: OpeningHoursEditor (28656)
Plugin: PicLayer (28660)
Plugin: editgpx (28689)
Plugin: multipoly-convert (28656)
Plugin: openstreetbugs (28656)
Plugin: openvisible (28656)
Plugin: pbf (28656)
Plugin: poly (28755)
Plugin: reltoolbox (28720)
Plugin: turnrestrictions (28656)
Plugin: undelete (28541)

java.awt.geom.IllegalPathStateException: missing initial moveto in path definition
	at java.awt.geom.Path2D$Double.needRoom(Unknown Source)
	at java.awt.geom.Path2D.closePath(Unknown Source)
	at org.openstreetmap.josm.tools.Geometry.getArea(Geometry.java:385)
	at org.openstreetmap.josm.tools.Geometry.polygonIntersection(Geometry.java:398)
	at org.openstreetmap.josm.data.validation.tests.BuildingInBuilding.isWayInsideMultiPolygon(BuildingInBuilding.java:100)
	at org.openstreetmap.josm.data.validation.tests.BuildingInBuilding$1.evaluateWay(BuildingInBuilding.java:147)
	at org.openstreetmap.josm.data.validation.tests.BuildingInBuilding$1.evaluate(BuildingInBuilding.java:173)
	at org.openstreetmap.josm.data.validation.tests.BuildingInBuilding$1.evaluate(BuildingInBuilding.java:121)
	at org.openstreetmap.josm.tools.SubclassFilteredCollection$FilterIterator.findNext(SubclassFilteredCollection.java:36)
	at org.openstreetmap.josm.tools.SubclassFilteredCollection$FilterIterator.hasNext(SubclassFilteredCollection.java:44)
	at org.openstreetmap.josm.tools.SubclassFilteredCollection.isEmpty(SubclassFilteredCollection.java:87)
	at org.openstreetmap.josm.data.validation.tests.BuildingInBuilding.endTest(BuildingInBuilding.java:180)
	at org.openstreetmap.josm.actions.ValidateAction$ValidationTask.realRun(ValidateAction.java:171)
	at org.openstreetmap.josm.gui.PleaseWaitRunnable.doRealRun(PleaseWaitRunnable.java:82)
	at org.openstreetmap.josm.gui.PleaseWaitRunnable.run(PleaseWaitRunnable.java:145)
	at java.util.concurrent.Executors$RunnableAdapter.call(Unknown Source)
	at java.util.concurrent.FutureTask$Sync.innerRun(Unknown Source)
	at java.util.concurrent.FutureTask.run(Unknown Source)
	at java.util.concurrent.ThreadPoolExecutor.runWorker(Unknown Source)
	at java.util.concurrent.ThreadPoolExecutor$Worker.run(Unknown Source)
	at java.lang.Thread.run(Unknown Source)

Attachments (0)

Change History (2)

comment:1 by skyper, 13 years ago

Description: modified (diff)

Mmh, all these "single node ways" seem to be created by potlatch2 (https://trac.openstreetmap.org/ticket/4378). Primary goal would be to fix the critical bug there !

But, yes you are right that validator should show these errors instead of throwing an exception.

Can not reproduce with openjdk-6 or 7 where validator shows errors and even offers an automatical repair. I did not test with too many cases though.

Repository Root: http://josm.openstreetmap.de/svn
Build-Date: 2012-10-17 01:31:11
Last Changed Author: stoecker
Revision: 5537
Repository UUID: 0c6e7542-c601-0410-84e7-c038aed88b3b
URL: http://josm.openstreetmap.de/svn/trunk
Last Changed Date: 2012-10-16 23:39:27 +0200 (Tue, 16 Oct 2012)
Last Changed Rev: 5537

Identification: JOSM/1.5 (5537 de)
Memory Usage: 314 MB / 592 MB (86 MB allocated, but free)
Java version: 1.6.0_24, Sun Microsystems Inc., OpenJDK 64-Bit Server VM
Operating system: Linux

and

Repository Root: http://josm.openstreetmap.de/svn
Build-Date: 2012-10-17 01:31:11
Last Changed Author: stoecker
Revision: 5537
Repository UUID: 0c6e7542-c601-0410-84e7-c038aed88b3b
URL: http://josm.openstreetmap.de/svn/trunk
Last Changed Date: 2012-10-16 23:39:27 +0200 (Tue, 16 Oct 2012)
Last Changed Rev: 5537

Identification: JOSM/1.5 (5537 de)
Memory Usage: 294 MB / 592 MB (109 MB allocated, but free)
Java version: 1.7.0_03, Oracle Corporation, OpenJDK 64-Bit Server VM
Operating system: Linux
Dataset consistency test: No problems found

EDT: status after fixing with validator

Last edited 13 years ago by skyper (previous) (diff)

comment:2 by skyper, 13 years ago

Resolution: duplicate
Status: newclosed

Closed as duplicate of #8118.
Is not related with the data inconsistency but rather a problem with the new building tests

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain team.
as The resolution will be set.
The resolution will be deleted. Next status will be 'reopened'.

Add Comment


E-mail address and name can be saved in the Preferences .
 
Note: See TracTickets for help on using tickets.