Splitting steps corrupts step_count #10534
rusty-snake
started this conversation in
General
Replies: 1 comment 1 reply
-
per #8069 (comment) and d43fc62 step_count is handled differently has it changed |
Beta Was this translation helpful? Give feedback.
1 reply
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Uh oh!
There was an error while loading. Please reload this page.
-
This is a follow-up on #8069, so ping @matkoniecz @quincylvania @DerDings @westnordost
The current behaviour of iD when splitting ways is to create two ways with identical tags. So if you have a 10m footway with
surface=asphaltbut actually the first half issurface=pacing_stone, you place a node, split the way and change thesurface. The same applies for all other kinds likeaccess*,smoothness,cycleway,tracktype, ... Except forstep_countwhich gets very silently modified. If you split ahighway=stepsto change a steps related tag such aswidth, this is good (i.e. no double count). If you however go to change one of the two ways to ahighway=footwaybecause there was just a longhighway=stepsmapped where half of the distance is actually only the footway to it, you end up withhighway=footwaywithstep_count=*. If it has steps, it shouldn't be a footway but if it does not have steps, the tag does not make sense.highway=stepswith an incorrect step count. Sometimes evenstep_count=0.So especially in the case of changing a
highway=stepsto ahighway=footway|pathit would be good to nuke thestep_count, mark it for re-survey or somehow notify the user that they did not got an identical copy of the tags.Beta Was this translation helpful? Give feedback.
All reactions