Opened 15 years ago
Closed 11 years ago
#4582 closed defect (fixed)
Overlapping Areas misreported
Reported by: | pinkduck | Owned by: | team |
---|---|---|---|
Priority: | major | Milestone: | |
Component: | Core validator | Version: | tested |
Keywords: | overlapping area way | Cc: | pinkduck, AM909, landwirt@…, moltonel, scai |
Description (last modified by )
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)
Change History (31)
comment:1 by , 15 years ago
Cc: | added |
---|---|
Version: | → latest |
comment:2 by , 15 years ago
Cc: | added |
---|---|
Priority: | normal → major |
Version: | latest → tested |
comment:4 by , 15 years ago
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 by , 15 years ago
Type: | enhancement → defect |
---|
comment:6 by , 14 years ago
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 !
by , 13 years ago
Attachment: | 4582_no_overlap.osm added |
---|
comment:8 by , 13 years ago
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 by , 13 years ago
Still have misreported overlaps which are only sharing borders. Have a look at 4582_still_problem.osm.
Problems are with buildings and boundary.
follow-up: 11 comment:10 by , 13 years ago
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 by , 13 years ago
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.
follow-ups: 13 15 comment:12 by , 13 years ago
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 by , 13 years ago
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
follow-up: 17 comment:14 by , 13 years ago
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 by , 13 years ago
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 ?
follow-up: 18 comment:17 by , 13 years ago
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 by , 13 years ago
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.
comment:20 by , 13 years ago
Cc: | added |
---|
comment:22 by , 11 years ago
Cc: | added |
---|---|
Keywords: | area way added; areas removed |
comment:23 by , 11 years ago
Cc: | added |
---|
comment:27 by , 11 years ago
The fix works for me (svn 6312), at least for the waterway usecase. Thanks.
comment:28 by , 11 years ago
Resolution: | → fixed |
---|---|
Status: | new → closed |
Closing this 4-year old ticket as I believe I have fixed the reported examples. Please open a new one with up-to-date description and samples if you find other false positives.
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.