Modify

Opened 6 years ago

Closed 6 years ago

Last modified 3 years ago

#16147 closed enhancement (fixed)

Pressurised waterway preset

Reported by: francois.lacombe Owned by: team
Priority: normal Milestone: 18.06
Component: Internal preset Version:
Keywords: waterway, pressurised, tunnel, pipeline Cc:

Description (last modified by Don-vip)

Recently, waterway=pressurised was reviewed on osm wiki, encouraging the use of waterway=* key to map not only free space but pipe flow waterway features also.
https://wiki.openstreetmap.org/wiki/Tag:waterway%3Dpressurised

Waterway presets should be updated and get new entries as follow :
==Penstock==

  • waterway=pressurised
  • man_made=pipeline
  • substance=water
  • location={underground,overground}
  • usage=penstock
  • name=*
  • operator=*
  • diameter=*
  • material=*

==Pipe flow tunnel==

  • waterway=pressurised
  • usage={headrace,tailrace,transmission,irrigation}
  • location=underground
  • tunnel=flooded
  • name=*
  • operator=*

==Canal preset should be updated as follow==

  • tunnel={culvert,flooded}
  • usage={transportation,transmission,irrigation,headrace,tailrace}

Main validation rules :

  • usage=penstock requires man_made=pipeline and waterway=pressurised
  • tunnel=yes and waterway=pressurised requires man_made=pipeline
  • tunnel=flooded is highly improbable with man_made=pipeline
  • waterway=canal shouldn't be used with tunnel=yes
  • waterway=pressurised requires tunnel=flooded or man_made=pipeline

Render I'd suggest :

  • waterway=pressurised + tunnel=* => add grey 1px casing to existing waterway line
  • waterway=pressurised + man_made=pipeline => add "pipeline color" 1px casing to existing waterway line

I know it's huge and you may have a lot of undergoing business, it would be great to handle this
Let me know if I can help or provide more precise things

Attachments (0)

Change History (12)

comment:1 by Don-vip, 6 years ago

Milestone: 18.06

comment:2 by Don-vip, 6 years ago

In 13895/josm:

see #16147 - Presets: add penstock, update canal

comment:3 by francois.lacombe, 6 years ago

Hi

Great commit for now with the best icon I could immagine.

Thanks :)

comment:4 by Don-vip, 6 years ago

In 13896/josm:

see #16147 - add usage context as "transmission" is already used for power substations

comment:5 by Don-vip, 6 years ago

You're welcome :) Can you please review French translations tomorrow?

comment:6 by francois.lacombe, 6 years ago

I would if I could find where to go to get preset translations.

Note that transmission substations are tagged with substation=transmission and not usage=transmission
Anyway I don't know if it has any impact yet

comment:7 by anonymous, 6 years ago

One more thing :

I've just noticed there is an existing usage key validation rule called 'missing attribute - usage=* without railway=*'
It should be extended to waterways if possible

comment:8 by Don-vip, 6 years ago

Description: modified (diff)

comment:9 by Don-vip, 6 years ago

Resolution: fixed
Status: newclosed

In 13972/josm:

fix #16147 - add rendering and validator rules for pressurised waterways

comment:10 by Klumbumbus, 6 years ago

In 13975/josm:

fix #16423, see #16147 - fix mapcss rule

comment:11 by Klumbumbus, 6 years ago

In 14041/josm:

see #16147 - better validator warning text (print value of second tag)

comment:12 by Klumbumbus, 3 years ago

In 17790/josm:

fix #18815, see #16147 - Don't warn about waterway=canal together with tunnel=yes and don't warn about bridge/tunnel=* on type=bridge/tunnel relations

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.