Modify ↓

Opened 10 years ago

Closed 10 years ago

Last modified 10 years ago

#9951 closed defect (fixed)

"Join Overlapping Areas" gives strange multipolygons

Reported by: akks Owned by: team
Priority: normal Milestone: 14.05
Component: Core Version: tested
Keywords: join areas Cc: Balaitous

Description (last modified by akks)

People on forum noticed that after fixing exceptions in r6976, Shift-J started to create some strange multipolygonal shapes.

Here are the sample osm files and before-after pictures (before; after shift-J; after shift-J in older JOSM)

Attachments (4)

before.png​ (7.7 KB ) - added by akks 10 years ago.
After.png​ (9.1 KB ) - added by akks 10 years ago.
After_josm6969.png​ (7.1 KB ) - added by akks 10 years ago.
test.osm​ (3.0 KB ) - added by akks 10 years ago.

Download all attachments as: .zip

Change History (16)

by akks, 10 years ago

Attachment: before.png​ added

by akks, 10 years ago

Attachment: After.png​ added

by akks, 10 years ago

Attachment: After_josm6969.png​ added

by akks, 10 years ago

Attachment: test.osm​ added

comment:1 by akks, 10 years ago

Description: modified (diff)

comment:2 by Balaitous, 10 years ago

I think I have found the bug, I will fix it quickly.

comment:3 by drnoble, 10 years ago

Just a quick note to say I am also experiencing this since upgrading to release 7000 on Mac OS X 10.9.2

thanks in advance for any progress in fixing this one

comment:4 by Don-vip, 10 years ago

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

comment:5 by anonymous, 10 years ago

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

comment:6 by Balaitous, 10 years ago

Resolution: → fixed
Status: new → closed

In 7052/josm:

fix #9951 - "Join Overlapping Areas" gives strange multipolygons

comment:7 by Balaitous, 10 years ago

This algorithm is complex. I hope that this time it is good!

comment:8 by bastiK, 10 years ago

Yes, was a long process to get there, see #5179 for some difficult cases.

comment:9 by Don-vip, 10 years ago

it could be a good idea to add non-regression unit tests for this feature, as it looks complicated.

comment:10 by Balaitous, 10 years ago

Yes, I have tried after #7959, but I haven't be able to create necessary mock objects.

comment:11 by Don-vip, 10 years ago

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

comment:12 by Don-vip, 10 years ago

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

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 .
Attachments ↑ Description ↑
 
Note: See TracTickets for help on using tickets.