Modify

Opened 12 years ago

Closed 10 years ago

Last modified 10 years ago

#7959 closed defect (fixed)

[patch] join areas exception

Reported by: Larry0ua Owned by: team
Priority: major Milestone: 14.04
Component: Core Version: latest
Keywords: Cc: Balaitous

Description

While merging two polygons (BBOX 25.8947754,48.3082036,25.917263,48.3139691, IDs: way 86633791, way 86633396) exception occurs. Please do not upload merged polygon as it may be useful for further testing.

Repository Root: http://josm.openstreetmap.de/svn
Build-Date: 2012-08-12 01:31:10
Last Changed Author: Don-vip
Revision: 5433
Repository UUID: 0c6e7542-c601-0410-84e7-c038aed88b3b
URL: http://josm.openstreetmap.de/svn/trunk
Last Changed Date: 2012-08-12 02:49:21 +0200 (Sun, 12 Aug 2012)
Last Changed Rev: 5433

Identification: JOSM/1.5 (5433 en)
Memory Usage: 62 MB / 247 MB (16 MB allocated, but free)
Java version: 1.7.0_05, Oracle Corporation, Java HotSpot(TM) Client VM
Operating system: Windows NT (unknown)
Dataset consistency test: No problems found

Plugin: FastDraw (28412)
Plugin: OpeningHoursEditor (28477)
Plugin: PicLayer (27883)
Plugin: buildings_tools (28529)
Plugin: reltoolbox (28412)
Plugin: reverter (28535)
Plugin: utilsplugin2 (28523)
Plugin: wikipedia (28506)

java.lang.RuntimeException: Join areas internal error.
	at org.openstreetmap.josm.actions.JoinAreasAction.findBoundaryPolygons(JoinAreasAction.java:1015)
	at org.openstreetmap.josm.actions.JoinAreasAction.joinAreas(JoinAreasAction.java:470)
	at org.openstreetmap.josm.actions.JoinAreasAction.actionPerformed(JoinAreasAction.java:364)
	at javax.swing.SwingUtilities.notifyAction(SwingUtilities.java:1661)
	at javax.swing.JComponent.processKeyBinding(JComponent.java:2879)
	at javax.swing.KeyboardManager.fireBinding(KeyboardManager.java:306)
	at javax.swing.KeyboardManager.fireKeyboardAction(KeyboardManager.java:263)
	at javax.swing.JComponent.processKeyBindingsForAllComponents(JComponent.java:2971)
	at javax.swing.JComponent.processKeyBindings(JComponent.java:2963)
	at javax.swing.JComponent.processKeyEvent(JComponent.java:2842)
	at java.awt.Component.processEvent(Component.java:6282)
	at java.awt.Container.processEvent(Container.java:2229)
	at java.awt.Component.dispatchEventImpl(Component.java:4861)
	at java.awt.Container.dispatchEventImpl(Container.java:2287)
	at java.awt.Component.dispatchEvent(Component.java:4687)
	at java.awt.KeyboardFocusManager.redispatchEvent(KeyboardFocusManager.java:1893)
	at java.awt.DefaultKeyboardFocusManager.dispatchKeyEvent(DefaultKeyboardFocusManager.java:752)
	at java.awt.DefaultKeyboardFocusManager.preDispatchKeyEvent(DefaultKeyboardFocusManager.java:1017)
	at java.awt.DefaultKeyboardFocusManager.typeAheadAssertions(DefaultKeyboardFocusManager.java:889)
	at java.awt.DefaultKeyboardFocusManager.dispatchEvent(DefaultKeyboardFocusManager.java:717)
	at java.awt.Component.dispatchEventImpl(Component.java:4731)
	at java.awt.Container.dispatchEventImpl(Container.java:2287)
	at java.awt.Window.dispatchEventImpl(Window.java:2719)
	at java.awt.Component.dispatchEvent(Component.java:4687)
	at java.awt.EventQueue.dispatchEventImpl(EventQueue.java:703)
	at java.awt.EventQueue.access$000(EventQueue.java:102)
	at java.awt.EventQueue$3.run(EventQueue.java:662)
	at java.awt.EventQueue$3.run(EventQueue.java:660)
	at java.security.AccessController.doPrivileged(Native Method)
	at java.security.ProtectionDomain$1.doIntersectionPrivilege(ProtectionDomain.java:76)
	at java.security.ProtectionDomain$1.doIntersectionPrivilege(ProtectionDomain.java:87)
	at java.awt.EventQueue$4.run(EventQueue.java:676)
	at java.awt.EventQueue$4.run(EventQueue.java:674)
	at java.security.AccessController.doPrivileged(Native Method)
	at java.security.ProtectionDomain$1.doIntersectionPrivilege(ProtectionDomain.java:76)
	at java.awt.EventQueue.dispatchEvent(EventQueue.java:673)
	at java.awt.EventDispatchThread.pumpOneEventForFilters(EventDispatchThread.java:244)
	at java.awt.EventDispatchThread.pumpEventsForFilter(EventDispatchThread.java:163)
	at java.awt.EventDispatchThread.pumpEventsForHierarchy(EventDispatchThread.java:151)
	at java.awt.EventDispatchThread.pumpEvents(EventDispatchThread.java:147)
	at java.awt.EventDispatchThread.pumpEvents(EventDispatchThread.java:139)
	at java.awt.EventDispatchThread.run(EventDispatchThread.java:97)

Attachments (3)

bad_polygons.osm (73.0 KB ) - added by Larry0ua 12 years ago.
file reproducing the bug
bad_polygons-simplified.osm (1.7 KB ) - added by AlfonZ 12 years ago.
Simplified file
ticket7959_2014-04-13_17:06:56_+0200.patch (14.6 KB ) - added by Balaitous 10 years ago.
Need to be test

Download all attachments as: .zip

Change History (19)

comment:1 by Larry0ua, 12 years ago

this error looks like in #5747, but as that is old ticket I preferred opening new one.

comment:2 by stoecker, 12 years ago

Can you extract the polygons and attach an osm file, which only contains the minimum data to reproduce this error?

by Larry0ua, 12 years ago

Attachment: bad_polygons.osm added

file reproducing the bug

by AlfonZ, 12 years ago

Attachment: bad_polygons-simplified.osm added

Simplified file

comment:3 by AlfonZ, 12 years ago

I have attached simplified file. It seems the precise position of four nodes (I have named them 1-4) is crucial for reproducibility.

comment:4 by Larry0ua, 12 years ago

so the problem is with code related to intersecting vertical lines. Unfortunately we found two polygons with several points in one vertical line...

comment:5 by Don-vip, 12 years ago

Ticket #5966 has been marked as a duplicate of this ticket.

comment:6 by Don-vip, 12 years ago

Ticket #8044 has been marked as a duplicate of this ticket.

comment:7 by Don-vip, 10 years ago

Cc: Balaitous added

@Balaitous: I think you should like this one :)

comment:8 by Balaitous, 10 years ago

Ticket #9911 has been marked as a duplicate of this ticket.

comment:9 by Balaitous, 10 years ago

I will look at this

by Balaitous, 10 years ago

Need to be test

comment:10 by Balaitous, 10 years ago

Not easy, but work with test case Simplified file .
#9911 is not the same case. The test case associate is a broken dataset (but good improvement for the algorithm !). Now it doesn't raise an error.

Version 0, edited 10 years ago by Balaitous (next)

comment:11 by Balaitous, 10 years ago

Summary: join areas exception[patch] join areas exception

comment:12 by Balaitous, 10 years ago

Resolution: fixed
Status: newclosed

In 6976/josm:

fix #7959: join areas exception

comment:13 by Don-vip, 10 years ago

Milestone: 14.04

comment:14 by anonymous, 10 years ago

Problem is still present in v. 7000

comment:15 by Don-vip, 10 years ago

if it's still present, please attach a sample file showing the problem.

comment:16 by akks, 10 years ago

see also #9951 (incorrect merging results)

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.