Modify

Opened 15 years ago

Closed 7 years ago

Last modified 7 years ago

#3346 closed defect (fixed)

Validator - fixing duplicate nodes spams with many dialogs

Reported by: bilbo Owned by: team
Priority: normal Milestone: 17.02
Component: Core validator Version: latest
Keywords: Cc: bilbo

Description (last modified by Don-vip)

When I load attached file (it is from OSM where someone accidentally uploaded everything
multiple times) and try to fix this using validator, I run into trouble:

When I select "Duplicate nodes" (there is almost 1000 of them) and press "fix", Validator starts fixing it - unfortunately, many of the duplicate nodes lie outside of downloaded area, so I am flooded with neverending prompts, saying that nodes are outside downloaded area and propting me if I really want to delete them (no, I want to fix only those inside, but I have to click through hundreds of prompts).

This should be solved somehow better with validator (perhaps with one prompt "some nodes are outside downloaded area" -> "fix all of them" / "fix only those inside" / "cancel")

Attachments (1)

bad_Kralupy.osm.bz2 (59.9 KB ) - added by anonymous 15 years ago.
OSM file with bad data that I tried to fix using Validator

Download all attachments as: .zip

Change History (15)

by anonymous, 15 years ago

Attachment: bad_Kralupy.osm.bz2 added

OSM file with bad data that I tried to fix using Validator

comment:1 by bilbo, 15 years ago

Cc: bilbo added

in reply to:  description comment:2 by anonymous, 14 years ago

Replying to anonymous:

When I load attached file (it is from OSM where someone accidentally uploaded everything
multiple times) and try to fix this using validator, I run into trouble:

When I select "Duplicate nodes" (there is almost 1000 of them) and press "fix", Validator starts fixing it - unfortunately, many of the duplicate nodes lie outside of downloaded area, so I am flooded with neverending prompts, saying that nodes are outside downloaded area and propting me if I really want to delete them (no, I want to fix only those inside, but I have to click through hundreds of prompts).

This should be solved somehow better with validator (perhaps with one prompt "some nodes are outside downloaded area" -> "fix all of them" / "fix only those inside" / "cancel")

In 2561 (current tested) you can disable this dialogue:
settings (F12)
advanced settings (last entry in list)
message.delete_outside_nodes= (empty)
message.delete_outside_nodes.value=1

These settings do not delete the outside_nodes as the value "1" implies!

Last edited 11 years ago by Don-vip (previous) (diff)

comment:3 by HamishB <hamish_b@…>, 14 years ago

for me using the latest testing release (r3208) I still see the errors listed in the validator results box, but when I hit fix it just fixes the downloaded ones it is sure about. no annoying dialog boxes. haven't touched the advanced properties.

so is this one fixed?

Hamish

comment:4 by HamishB <hamish_b@…>, 14 years ago

(it might be nice to get a "x errors not fixed as components are partially out of region" or so message, at fist I couldn't figure out why "Fix" didn't fix them. until I zoomed to one and guessed why)

comment:5 by Don-vip, 11 years ago

Description: modified (diff)
Reporter: changed from anonymous to bilbo

comment:6 by GerdP, 8 years ago

I think this was partly fixed with r10924, see also #13494.

comment:7 by Don-vip, 7 years ago

Milestone: 17.02

comment:8 by Don-vip, 7 years ago

Resolution: fixed
Status: newclosed

In 11627/josm:

fix #3346 - improve drastically the performance of fixing duplicate nodes by:

  • caching data sources area computation
  • moving layer invalidation from UndoRedoHandler.addNoRedraw to UndoRedoHandler.add
  • avoiding any EDT call when building tag conflict dialog if it's not meant to be displayed

comment:9 by Don-vip, 7 years ago

In 11628/josm:

see #3346 - remove debug info committed accidentally

comment:10 by Don-vip, 7 years ago

In 11629/josm:

see #3346 - findbugs - IS2_INCONSISTENT_SYNC

comment:11 by Don-vip, 7 years ago

In 11630/josm:

see #3346 - update unit test

in reply to:  8 comment:12 by GerdP, 7 years ago

Replying to Don-vip:

In 11627/josm:

fix #3346 - improve drastically the performance of fixing duplicate nodes by:

  • caching data sources area computation
  • moving layer invalidation from UndoRedoHandler.addNoRedraw to UndoRedoHandler.add
  • avoiding any EDT call when building tag conflict dialog if it's not meant to be displayed

I wonder if this works when e.g. a plugin modifies the public field Dataset.dataSources. Do you plan to change that to
private?

comment:13 by Don-vip, 7 years ago

After updating the plugins, yes. At least PBF and O5M plugins are concerned.

comment:14 by Don-vip, 7 years ago

In 11632/josm:

see #3346 - make DataSet.dataSources private

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.