Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

waterway=fish_pass #2895

Open
printmaps opened this issue Oct 15, 2017 · 23 comments · May be fixed by #4388
Open

waterway=fish_pass #2895

printmaps opened this issue Oct 15, 2017 · 23 comments · May be fixed by #4388
Labels
Milestone

Comments

@printmaps
Copy link

Fish ladders are currently not rendered. I recommend to draw them identically as waterway=stream.

@kocio-pl kocio-pl added the water label Oct 15, 2017
@kocio-pl kocio-pl added this to the New features milestone Oct 15, 2017
@kocio-pl
Copy link
Collaborator

@polarbearing
Copy link
Contributor

taginfo 719 on a way, 17 on a node.

@pnorman
Copy link
Collaborator

pnorman commented Oct 18, 2017

The usage seems insufficient.

@kocio-pl
Copy link
Collaborator

What do you think would be sufficient usage in this case?

@pnorman
Copy link
Collaborator

pnorman commented Oct 18, 2017

There isn't a specific number. And if there were more usage, we'd still need to consider if it's desirable to render it, since usage alone doesn't mean we want to display it.

@kocio-pl
Copy link
Collaborator

It's best to know all such things beforehand to make deciding easier. I don't expect a number or hard limits, even general estimation would be helpful.

@dieterdreist
Copy link

dieterdreist commented Oct 18, 2017 via email

@printmaps
Copy link
Author

Please consider that a fishpass divides the landscape and is the reason for bridges. Example (the small waterway is the fishpass):

bildschirmfoto 2017-10-18 um 11 56 33

@matthijsmelissen
Copy link
Collaborator

I dont think we should increase the amount of tags in the waterway namespace for such specialised purposes. That will make life difficult for every data consumer. Better something like waterway=canal (or ditch?), canal=fish_pass.

@dieterdreist
Copy link

dieterdreist commented Oct 18, 2017 via email

@kocio-pl
Copy link
Collaborator

It's definitely not a canal:

Use waterway=canal for man-made waterways used for transportation or also for the largest waterways created for irrigation purposes.

It has also nothing to do with drainage system:

Use waterway=drain for minor artificial waterways, typically lined with concrete or similar, used to carry storm water or grey-discharge.

Use waterway=ditch for simple narrow artificial waterways used to drain nearby land, to remove storm-water or similar.

I agree that this would be tagging for rendering and we don't know how many of them are tagged this way already. Unfortunately waterway=* also contains other objects than just a water line (like dam or weir), so it's not easy for data consumer to pick just the water, but our database has no explicit "contains water" property.

@kocio-pl
Copy link
Collaborator

I dont think we should increase the amount of tags in the waterway namespace for such specialised purposes. That will make life difficult for every data consumer.

Honestly I don't understand - how would rendering already existing tag on osm-carto make problems for data consumer?

@SomeoneElseOSM
Copy link
Contributor

Honestly I don't understand - how would rendering already existing tag on osm-carto make problems for data consumer?

I'm not sure I understand that either, but http://map.atownsend.org.uk/maps/map/map.html does render fish passes (in the UK and Ireland) so if there's a problem, someone should be able to find an example there.

@ZeiP
Copy link

ZeiP commented Jul 7, 2018

The usage has grown and there are some places where the tagging has not been used because it's not rendered. In some cases the fish pass is separate from the main waterway, so in my opinion it's important to have the fish passes rendered, and I support the suggested rendering as a stream – this is also mostly used in the cases where the tagging has been done based on the rendering.

@jeisenbe
Copy link
Collaborator

jeisenbe commented Sep 10, 2019

There is some debate about this proposed tag on https://wiki.openstreetmap.org/wiki/Talk:Tag:waterway%3Dfish_pass - It would be good to discuss further whether waterway=fish_pass or waterway=canal + canal= or usage= =fish_pass is preferable. Note that waterway=canal is now used with usage=spillway on dams, a similar feature in some ways.

@gpsvisualizer
Copy link

Regardless of how the debate turns out on the OSM wiki, is there any reason NOT to simply render waterway=fish_pass as either a canal or stream?

@jeisenbe
Copy link
Collaborator

jeisenbe commented Jan 7, 2020

If there are a significant number of fish passes which are tagged with waterway=canal + usage=fish_pass then that would be a problem. At the moment this does not appear to be the case: https://taginfo.openstreetmap.org/search?q=usage%3Dfish_pass (16 uses)

If someone wants to submit a new PR showing a rendering for this feature, and tests how it works in a number of places, it would be welcomed. @gpsvisualizer would you be interested in doing this?

@gpsvisualizer
Copy link

I have no idea how to do "pull requests." I don't even understand what Git is, to be honest — and my brain is full enough already.

But I still don't understand why it'd be a problem if both schemes were rendered, regardless of how many tags exist out there. What's the harm?

@SomeoneElseOSM
Copy link
Contributor

I have no idea how to do "pull requests."

Just in case you (or someone else) did want to do this, I wrote a diary entry a while ago that was designed to be a step-by-step guide to what would be required, with no prior knowledge of git or github: https://www.openstreetmap.org/user/SomeoneElse/diary/43041 . I'm sure other guides are available too.

@sun-geo
Copy link

sun-geo commented Aug 25, 2020

Small update, the current count of waterway=fish_pass osm-items crosses the 1k line, see

image
https://ohsome.org/apps/dashboard

The hack for displaying fish passes would be, while having the 1d waterway=fish_pass tag by additional adding a 2d water area e.g. using a tagging like natural=water + water=fish_pass ;-)

See examples here .

@jeisenbe
Copy link
Collaborator

jeisenbe commented Jan 5, 2021

@TheJJ TheJJ linked a pull request Apr 25, 2021 that will close this issue
@TheJJ
Copy link

TheJJ commented Apr 25, 2021

I've naively submitted #4388 to render this just like a stream for now, after noticing that my "de-facto" tagging wouldn't show up :)

Although I'm a total noob, using waterway=canal + usage=fish_pass seems very good for the connecting waterway, just the fish steps (-> not a canal) itself could maybe better be mapped with waterway=fish_pass.
What do you think?

@imagico
Copy link
Collaborator

imagico commented Jun 6, 2022

I looked into the use of the tag a bit today and it seems this is quite clearly the dominant way to tag fish passes/fish ladders. Secondary tags for existing waterway features:

https://taginfo.openstreetmap.org/keys/fish_pass
https://taginfo.openstreetmap.org/tags/usage=fish_pass

have only insignificant use.

It seems the tag is quite consistently used - though only very sporadically outside of central and western Europe. The main variability in use is if the fish pass is mapped as component in the waterway network (i.e. connected to the waterway network so you could do 'routing for fishes') or if it is just drawn for the physical feature itself without connection to other waterways.

Examples:

https://www.openstreetmap.org/way/103152318
https://www.openstreetmap.org/way/763019340

In the former case there is also no clear consensus if the segments connecting the fish pass to the rest of the waterway network (without actually being a fish pass themselves) are tagged waterway=fish_pass or otherwise.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.