Modify

Opened 5 years ago

Closed 5 years ago

#17339 closed defect (fixed)

false positive "Sudden highway type change"

Reported by: Klumbumbus Owned by: abalosc1
Priority: normal Milestone:
Component: External rule Version:
Keywords: Cc:

Description

wiki:Rules/OsmoseValidations

Validate e.g. https://www.openstreetmap.org/node/29039893 or https://www.openstreetmap.org/node/599192072 which warns about "Sudden highway type change". These are very common tagging situations.

Attachments (0)

Change History (7)

comment:1 by dmanzer95, 5 years ago

Hello,

The reason that this was flagged was because the residential road and secondary road are coming together at their beginning/endpoints (Same reason for the second example just different highway classifications). I notice that the reason for the split in the secondary road in the first example is because of different sidewalk tags, so this would be a false positive. There will definitely be false positives as there are with OSMOSE.

in reply to:  1 comment:2 by Klumbumbus, 5 years ago

Replying to dmanzer95:

There will definitely be false positives

Some false positives are often unavoidable, but imho this rule produces way too much false positives and way more false positives than "right positives" which makes this rule barely useful.

comment:3 by dmanzer95, 5 years ago

after another look I agree. Working to more closely match OSMOSE results

comment:4 by Klumbumbus, 5 years ago

I too thought about possible code improvements and I think a proper solution is not possible with the current mapcss implementation in JOSM, but I could be wrong.

comment:5 by dmanzer95, 5 years ago

I may have a way to narrow down the results but it could get a little messy. I'll test it out and post my findings here. Thanks for bringing this issue up!

comment:6 by dmanzer95, 5 years ago

Right again, mapcss did not support what was required for this idea.

comment:7 by abalosc1, 5 years ago

Resolution: fixed
Status: assignedclosed

The "Sudden highway type change" check 1090 has been removed

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain abalosc1.
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.