Modify

Opened 3 years ago

Last modified 17 months ago

#4582 new defect

Overlapping Areas misreported

Reported by: pinkduck Owned by: team
Priority: major Component: Core validator
Version: tested Keywords: overlapping, areas
Cc: pinkduck@…, alan.mintz@…, landwirt@…

Description (last modified by simon04)

With two closed ways representing two mutually exclusive areas, if both are separate there is no issue. If they share a single node at a vertex there is no validation report. However, if they share two nodes of a common edge then an 'Overlapping areas' validation 'Other'-type notification is given by the JOSM Validator plug-in.

When these two areas only overlap at their polygon borders I don't think there should be a validation notification raised. While there is a one-dimensional overlapping line of the virtual separation of a 2D surface, it tends towards zero given an infinitesimal approximation.

If there is any true overlap of the areas defined by the polygonal borders then this 'Other' notification is acceptable. This could be calculated using an algorithm similar to that described at: http://mathforum.org/library/drmath/view/63171.html. However, with the adaptation that parallel test edges should be excluded from determining the overlap. An obvious exception being when all nodes of each area overlap.

Attachments (3)

4582_no_overlap.osm (1010 bytes) - added by simon04 20 months ago.
4582_still_problem.osm (16.2 KB) - added by skyper 20 months ago.
example osm file
true_overlapps.osm (13.2 KB) - added by skyper 20 months ago.
second try, forgot to save before upload

Download all attachments as: .zip

Change History (23)

comment:1 Changed 3 years ago by pinkduck

  • Cc pinkduck@… added
  • Version set to latest

comment:2 Changed 3 years ago by Am909

  • Cc alan.mintz@… added
  • Priority changed from normal to major
  • Version changed from latest to tested

Agreed. In the US, administrative boundaries are quite commonly, correctly, glued together. The warning should only occur if they actually cross - not if they simply share edges.

Another example is landuse polygons. It is common, and correct, for housing tracts to be glued together along their interfaces, and this should not result in an error.

Too many false positives, and warnings are ignored.

comment:3 Changed 3 years ago by e2jk

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

comment:4 Changed 3 years ago by e2jk

From Ticket #4768:

@skyper:
If two areas with landuse=* area bound together using the same nodes as boundery validator should not inform about overlapping areas. The same for buildings and ways as areas. For example a parking-area with highway=service and a building.

But it should inform or even warn if these area really overlap which it does not !!

@e2jk:
This also happens for waterways that are connected (but not really overlapping)

comment:5 Changed 3 years ago by e2jk

  • Type changed from enhancement to defect

comment:6 Changed 2 years ago by skyper

This check is not much useful right now, as it does not work for really overlapping areas and only leads to use relations for all areas which have at least one border sharinging with another area which is the default !

comment:7 Changed 20 months ago by simon04

In [4448/josm]:

see #4582 - validator test for overlapping areas

Changed 20 months ago by simon04

comment:8 Changed 20 months ago by simon04

Validator test added – please test and report :-).

Geometry.polygonIntersection has a problem with a certain type of overlap (cf. attachment:4582_no_overlap.osm).

Changed 20 months ago by skyper

example osm file

comment:9 Changed 20 months ago by skyper

Still have misreported overlaps which are only sharing borders. Have a look at 4582_still_problem.osm.

Problems are with buildings and boundary.

r4475

Last edited 20 months ago by skyper (previous) (diff)

comment:10 follow-up: Changed 20 months ago by simon04

Right, I only removed 1 out of 4 such errors. Currently, one will get "Overlapping {highways,railways,ways} (with area)".

Apart from the wrong wording (should be something like "{highway,railway,way} is sharing segment with area"), the main question is whether this informational warning is worthless at all. I tend to drop this.

Please comment :-).

comment:11 in reply to: ↑ 10 Changed 20 months ago by skyper

Replying to simon04:

Right, I only removed 1 out of 4 such errors. Currently, one will get "Overlapping {highways,railways,ways} (with area)".

Major improvement would be if Josm finds overlapping areas which it still does not ! See comment:8

Apart from the wrong wording (should be something like "{highway,railway,way} is sharing segment with area"), the main question is whether this informational warning is worthless at all. I tend to drop this.

Please comment :-).

Please, don't drop it. Most of the times, I consider these warnings as errors or at least areas to investigate.

comment:12 follow-ups: Changed 20 months ago by simon04

Major improvement would be if Josm finds overlapping areas which it still does not ! See comment:8

In your example, which overlap should be reported, i.e., between which objects?

Please, don't drop it. Most of the times, I consider these warnings as errors or at least areas to investigate.

Thus, this informational warning should remain but with a different wording (see suggestion above)?

Last edited 20 months ago by simon04 (previous) (diff)

comment:13 in reply to: ↑ 12 Changed 20 months ago by skyper

Replying to simon04:

Major improvement would be if Josm finds overlapping areas which it still does not ! See comment:8

Your example shows that a overlap between a way and an area isn't reported.

Sorry, I think I got it now:

The way is a boundary and part of a multip. relation. Maybe leave out ways in general and only look for some (highway,waterway...)

Please, don't drop it. Most of the times, I consider these warnings as errors or at least areas to investigate.

Thus, this informational warning should remain but with a different wording (see suggestion above)?

+1

comment:14 follow-up: Changed 20 months ago by simon04

For consistency, the remaining test errors of this kind, i.e., "Overlapping {highways,railways,ways}", should also renamed to "{highways,railways,ways} share segment"?

Last edited 20 months ago by simon04 (previous) (diff)

comment:15 in reply to: ↑ 12 Changed 20 months ago by skyper

Replying to simon04:

Did miss that:

In your example, which overlap should be reported, i.e., between which objects?

There exists no overlap, only if you count the multip.-relation as area. All other are only touching!

My example is not very good, maybe I can add a better one.

  • The test should not include ways with tag boundary.
  • does the check work with multipolynom relations ?

comment:16 Changed 20 months ago by simon04

In [4496/josm]:

see #4582 - clarify warning messages of overlapping way validator test

comment:17 in reply to: ↑ 14 ; follow-up: Changed 20 months ago by simon04

  • Description modified (diff)

Replying to simon04:

For consistency, the remaining test errors of this kind, i.e., "Overlapping {highways,railways,ways}", should also renamed to "{highways,railways,ways} share segment"?

This question is to be decided before closing this ticket.

comment:18 in reply to: ↑ 17 Changed 20 months ago by skyper

Replying to simon04:

Replying to simon04:

For consistency, the remaining test errors of this kind, i.e., "Overlapping {highways,railways,ways}", should also renamed to "{highways,railways,ways} share segment"?

This question is to be decided before closing this ticket.

It does not even find all overlapping buildings. I have creating a test files, which needs to be improved but it is a start.

Changed 20 months ago by skyper

second try, forgot to save before upload

comment:19 Changed 17 months ago by skyper

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

comment:20 Changed 17 months ago by skyper

  • Cc landwirt@… added

Add Comment

Modify Ticket

Change Properties
<Author field>
Action
as new .
as The resolution will be set. Next status will be 'closed'.
to The owner will be changed from team. Next status will be 'new'.
Next status will be 'needinfo'.The owner will change to pinkduck
as duplicate The resolution will be set to duplicate. Next status will be 'closed'.The specified ticket will be cross-referenced with this ticket
The owner will be changed from team to anonymous. Next status will be 'assigned'.
Author


E-mail address and user name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.