Modify

Opened 14 years ago

Closed 4 years ago

#7120 closed enhancement (duplicate)

Check: Railway end node near other railway

Reported by: newsScott@… Owned by: team
Priority: normal Milestone:
Component: Core validator Version:
Keywords: unconnected_way railway Cc: GerdP

Description

There allready exists a "Way end node near other highway" check. The same would be very helpful to improve the quality of railway data.

Some false positives may be avoidable if the code would also check for the tag "railway:buffer_stop" in the end node.

Attachments (0)

Change History (10)

comment:1 by anonymous, 14 years ago

i realized that there is allready some support of this. But i think the current implementation needs some changes:

in org/openstreetmap/josm/data/validation/tests/UnconnectedWays.java:

The default distance (mindist) of 10 meters is not useful for checking railways. I my experiments, something like 1-2 meters is best. So should there be a mindist_highway and mindist_railway?

The code finds some false positives: many platforms are still tagged "railway=platform" instead of "public_transport=platform". In member function "visit", ways tagged with railway=platform should not be taken into account:

something like

if (w.hasKey("highway")
w.hasKey("railway") && !w.get("railway").equals("platform")) {

set = endnodes_highway;

}

comment:2 by simon04, 14 years ago

Component: CoreCore validator

comment:3 by simon04, 14 years ago

Keywords: unconnected_way added

comment:4 by Don-vip, 7 years ago

Keywords: railway added

comment:5 by skyper, 5 years ago

The test was reworked. Does it work better, now?

Last edited 5 years ago by skyper (previous) (diff)

comment:6 by skyper, 4 years ago

Cc: GerdP added

Anything left, besides #21038.

comment:7 by GerdP, 4 years ago

Resolution: fixed
Status: newclosed

I think #18272 fixes this. Please open a new ticket if further issues are found

comment:8 by Don-vip, 4 years ago

Resolution: fixedduplicate

Marked as duplicate of #18272.

comment:9 by skyper, 4 years ago

Resolution: duplicate
Status: closedreopened

comment:10 by skyper, 4 years ago

Resolution: duplicate
Status: reopenedclosed

Closed as duplicate of #21038.
See r18272.

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.