Modify

Opened 6 years ago

Closed 6 years ago

#16958 closed enhancement (worksforme)

On "zoom to problem" in "Crossing boundaries" validator report zoom to one of crossing points, not bounding box of boundaries

Reported by: mkoniecz Owned by: team
Priority: normal Milestone:
Component: Core validator Version:
Keywords: template_report Cc:

Description

What steps will reproduce the problem?

  1. run validator
  2. find two huge crossing boundaries
  3. select "zoom to problem"

What is the expected result?

I get zoomed to problem (crossing point of two boundaries)

What happens instead?

I get zoomed to boundary extend ( https://www.openstreetmap.org/way/231994686 https://www.openstreetmap.org/way/232003133 )

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

It would be also nice to mark location of the currently selected validator issue (pulsing green point? arrow? green target sign pointing at it?)

URL:https://josm.openstreetmap.de/svn/trunk
Repository:UUID: 0c6e7542-c601-0410-84e7-c038aed88b3b
Last:Changed Date: 2018-10-30 00:47:23 +0100 (Tue, 30 Oct 2018)
Build-Date:2018-10-30 02:32:55
Revision:14385
Relative:URL: ^/trunk

Identification: JOSM/1.5 (14385 en) Linux Ubuntu 16.04.5 LTS
Memory Usage: 773 MB / 869 MB (324 MB allocated, but free)
Java version: 1.8.0_191-b12, Oracle Corporation, Java HotSpot(TM) 64-Bit Server VM
Screen: :0.0 1920x1080
Maximum Screen Size: 1920x1080
Dataset consistency test: No problems found

Plugins:
+ OpeningHoursEditor (34535)
+ buildings_tools (34572)
+ continuosDownload (82)
+ imagery_offset_db (34641)
+ measurement (34529)
+ reverter (34552)
+ todo (30306)

Last errors/warnings:
- W: java.io.IOException: Attribution is not loaded yet
- W: java.io.IOException: Attribution is not loaded yet
- W: java.io.IOException: Attribution is not loaded yet
- W: java.io.IOException: Attribution is not loaded yet
- W: java.io.IOException: Attribution is not loaded yet
- W: java.io.IOException: Attribution is not loaded yet
- W: java.io.IOException: Attribution is not loaded yet
- W: java.io.IOException: Attribution is not loaded yet
- W: java.io.IOException: Attribution is not loaded yet
- W: java.io.IOException: Attribution is not loaded yet

Attachments (0)

Change History (4)

comment:1 by GerdP, 6 years ago

Hmm, the zoom to shows the area where they overlap. The two boundaries in your example mostly overlap, so I don't see what else JOSM could do. Maybe you think about a new function like "zoom to where they differ" ?

comment:2 by mkoniecz, 6 years ago

So this validator rule reports both crossing and overlapping boundaries, not only crossing ones?

comment:3 by GerdP, 6 years ago

The validator produces two warnings, one with crossing boundaries, one with overlapping ways.
The 2nd one zooms to a smaller area. Maybe this helps you?

comment:4 by mkoniecz, 6 years ago

Resolution: worksforme
Status: newclosed

It helped in this case, thanks! I will report it again if I encounter a better example.

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.