Ticket #4582 (new defect)
Overlapping Areas misreported
| Reported by: | pinkduck | Owned by: | team |
|---|---|---|---|
| Priority: | major | Component: | Plugin validator |
| Version: | tested | Keywords: | overlapping,areas |
| Cc: | pinkduck@…, alan.mintz@…, landwirt@… |
Description (last modified by simon04) (diff)
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
Change History
comment:2 Changed 19 months 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:4 Changed 19 months 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:6 Changed 8 months 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 5 months ago by simon04
In [4448/josm]:
comment:8 Changed 5 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).
comment:9 Changed 4 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.
comment:10 follow-up: ↓ 11 Changed 4 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 4 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: ↓ 13 ↓ 15 Changed 4 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)?
comment:13 in reply to: ↑ 12 Changed 4 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: ↓ 17 Changed 4 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"?
comment:15 in reply to: ↑ 12 Changed 4 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 4 months ago by simon04
In [4496/josm]:
comment:17 in reply to: ↑ 14 ; follow-up: ↓ 18 Changed 4 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 4 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 4 months ago by skyper
-
attachment
true_overlapps.osm
added
second try, forgot to save before upload
comment:19 Changed 4 weeks ago by skyper
Ticket #7236 has been marked as a duplicate of this ticket.


