iD icon indicating copy to clipboard operation
iD copied to clipboard

Add waterway=flowline to waterway tags representing directional water flows

Open waldyrious opened this issue 1 year ago • 6 comments

Currently, lines with waterway=flowline are represented as generic waterway features, and don't get the arrows pointing in the water flow direction along their length (as is the case with rivers, streams, canals, etc.). This change adds the tag so the representation is consistent with the other flowing waterway tags.

waldyrious avatar Jun 19 '24 10:06 waldyrious

It might be worth making them slightly visually different, just so it's easier to tell if a waterway thru a water body has been converted yet. Maybe a slightly different shade of blue?

JesseWeinstein avatar Jun 19 '24 11:06 JesseWeinstein

For easy reference: https://wiki.openstreetmap.org/wiki/Tag:waterway%3Dflowline , ~800 usage globally.

tordans avatar Jun 19 '24 15:06 tordans

@waldyrious what about https://wiki.openstreetmap.org/wiki/Tag:waterway%3Dlink with 3k usage. The wiki sounds like this is something that indicates a direction as well?

tordans avatar Jun 19 '24 15:06 tordans

The wiki sounds like this is something that indicates a direction as well?

Does it? I read the opposite:

Unlike most waterway=* values, the tag does not model a discrete physical feature and does not imply a flow direction.

(emphasis mine)

waldyrious avatar Jun 19 '24 21:06 waldyrious

It might be worth making them slightly visually different, just so it's easier to tell if a waterway thru a water body has been converted yet. Maybe a slightly different shade of blue?

Sounds good to me, but I didn't want to do too dramatic a change. Happy to add the change if maintainers agree it makes sense.

waldyrious avatar Jun 19 '24 21:06 waldyrious

Does it?

You are right. I think I was thinking about it because of the routing part of the wiki but that has nothing to do with the direction.

tordans avatar Jun 20 '24 05:06 tordans

@tordans @JesseWeinstein any suggestions on who I should ping (here or elsewhere) for additional feedback on the suggestion of using a different visual style?

waldyrious avatar Jul 12 '24 20:07 waldyrious

@waldyrious ATM Martin is the only one who can merge. We are working on adding community contributors that can help with easy PRs like this one. Once that is set up, I will be able to merge as well…

tordans avatar Jul 13 '24 05:07 tordans

That's great to hear! But I was asking specifically about making additional changes to the rendering of waterway=flowline ways, as @JesseWeinstein proposed above, so that I can know whether there's more work for me to do in this PR as opposed to just wait for the merge. Is this something that Martin alone can decide? Do you or others have thoughts on the matter?

waldyrious avatar Jul 16 '24 16:07 waldyrious

Thanks for merging and for the suggestions! I like both of them, actually :sweat_smile: @JesseWeinstein do you have any preference between the two rendering options?

waldyrious avatar Jul 17 '24 10:07 waldyrious

I still like the different shade of blue better. Glad to get this PR merged though, in any case! Thank you both.

JesseWeinstein avatar Jul 17 '24 22:07 JesseWeinstein

ok, these features a rendered in a slightly different shade of blue now.

tyrasd avatar Jul 26 '24 09:07 tyrasd

Ah, it looks really nice: https://ideditor.netlify.app/#background=MAPNIK&disable_features=boundaries&map=16.62/43.24503/-79.21668

The next thing is to add a preset, so flowlines don't just show up as "Waterway feature". If no-one else has done this, I'll try and pull together a PR soon.

(and done so: https://github.com/openstreetmap/id-tagging-schema/pull/1300 )

JesseWeinstein avatar Jul 26 '24 12:07 JesseWeinstein

I opened a formal proposal, here: https://wiki.openstreetmap.org/wiki/Proposal:Flowlines

JesseWeinstein avatar Aug 14 '24 00:08 JesseWeinstein