Opened 5 years ago
Last modified 4 years ago
#20530 closed enhancement
Add man_made pump preset — at Version 31
| Reported by: | francois.lacombe | Owned by: | skyper |
|---|---|---|---|
| Priority: | normal | Milestone: | 21.10 |
| Component: | Internal preset | Version: | |
| Keywords: | pumps, pump mechanism, drivers | Cc: | Klumbumbus |
Description (last modified by )
Dear maintainers,
Earlier this year, pumping proposal has been reviewed and approved on osm wiki.
https://wiki.openstreetmap.org/wiki/Proposed_features/Pumping_proposal
It reinforces tagging associated to man_made=pump and some presets could be useful to encourage mappers to use it accordingly.
Is this possible to add a preset "Pump" for man_made_pump please?
https://wiki.openstreetmap.org/wiki/Tag:man_made%3Dpump
Some combinations could be done with man_made=pump and could also be added to man_made=windpump, man_made=water_well or man_made=petroleum_well.
- pump_mechanism=* see values https://wiki.openstreetmap.org/wiki/Key:pump_mechanism
- mechanical_driver=* see values https://wiki.openstreetmap.org/wiki/Key:mechanical_driver
- mechanical_coupling=* see values https://wiki.openstreetmap.org/wiki/Key:mechanical_coupling
- handle=wheel,cross,lever,crank,button,no https://wiki.openstreetmap.org/wiki/Key:handle
- pressure=*
- flow_rate=*
- operator=*
- manufacturer=*
handle is already in use with pipeline=valve
Very simple validation rules apply
- when mechanical_driver=manual used, handle=* is expected.
- pump_mechanism=* requires man_made=pump, man_made=windpump, man_made=water_well or man_made=petroleum_well
Thanks in advance
Change History (34)
comment:1 by , 5 years ago
| Description: | modified (diff) |
|---|
comment:2 by , 5 years ago
| Description: | modified (diff) |
|---|
comment:3 by , 5 years ago
| Keywords: | pump, pump mechanism → pump mechanism |
|---|
comment:4 by , 5 years ago
comment:5 by , 5 years ago
Hi Skyper
Here is a pump icon that could be associated to man_made=pump
It is provided with a blue background that could be changed accordingly with substance=* value, just like street cabinets with street_cabinet=*
All the best
comment:6 by , 5 years ago
| Milestone: | → 21.04 |
|---|---|
| Owner: | changed from to |
| Status: | new → assigned |
comment:7 by , 5 years ago
Keep in mind #20531 is linked to this ticket and can be processed in the same time.
Best regards
comment:8 by , 5 years ago
| Cc: | added |
|---|
We need a pump icon and I can't find a suitable one.
https://www.flaticon.com/free-icon/water-pump_940748 and https://www.flaticon.com/free-icon/water-pump_523818 are really good but I'm not sure about the license.
Any volunteer to draw a new one? :D
comment:9 by , 5 years ago
In r17028 I made one for man_made=pumping_station. As this was not so long ago we could change it and use it for man_made=pump instead. But that would require to find a new icon for man_made=pumping_station...
follow-up: 11 comment:10 by , 5 years ago
Don-Vip, Klmbumbus, didn't you guys see the beautiful one I've provided upside? :)
https://josm.openstreetmap.de/attachment/ticket/20530/eau_pompe.svg
Well, mine is a centrifugal one while Don-Vip proposed a piston (fountain) shape one, both are suitable but I'd use the fountain for man_made=water_well.
We can discuss this here.
comment:11 by , 5 years ago
Replying to francois.lacombe:
Don-Vip, Klmbumbus, didn't you guys see the beautiful one I've provided upside? :)
https://josm.openstreetmap.de/attachment/ticket/20530/eau_pompe.svg
True, thats nice. Did you create it yourself or what's the source?
comment:15 by , 5 years ago
@François same for petroleum_well. It has been ignored in r9050 because it was unclear at the time but it's time we include it now.
comment:16 by , 5 years ago
Sorry guys, I won't have enough time to work on this shortly
I agree this would be great but will require further improvement
comment:17 by , 5 years ago
| Cc: | added |
|---|
comment:18 by , 5 years ago
| Milestone: | 21.04 → 21.05 |
|---|
comment:19 by , 5 years ago
| Milestone: | 21.05 → 21.06 |
|---|
comment:20 by , 5 years ago
| Milestone: | 21.06 → 21.07 |
|---|
comment:21 by , 5 years ago
Dear all
Find attached 2 missing icons, for windpump and petroluem_well.
Hope this will do the job, all the best
comment:22 by , 5 years ago
@Don-vip:
Do you take care of this or would you like to get support by a patch? Ticket is assign to yourself.
comment:26 by , 5 years ago
| Milestone: | 21.07 → 21.08 |
|---|
comment:27 by , 5 years ago
| Owner: | changed from to |
|---|---|
| Status: | assigned → new |
I can't find enough time to finish this :( If anyone wants to do it, go. If not, I'll take it back after my holidays, in September.
comment:28 by , 5 years ago
| Milestone: | 21.08 → 21.09 |
|---|
comment:29 by , 5 years ago
| Owner: | changed from to |
|---|
comment:30 by , 5 years ago
Mmh, I did not fully understand which additional tags are suited for water_well, petroleum_well and windpump.
mechanical_drive does not work with windpump, I guess, and handle is only useful with mechanical_drive=manual (validator warning). Anything else to consider?
For the background color of pump and pumping_station I would use the same colors used for generator:source=* with power=generator.
comment:31 by , 5 years ago
| Description: | modified (diff) |
|---|---|
| Keywords: | pumps drivers added |
Hi Skyper, thank you for the followup
Indeed only windpump has restriction on mechanical_driver.
Wells can got all pump tagging.
As handles may be installed in case of failing driver, windpumps particularly can have handles despite the driver isn't manual.
See https://wiki.openstreetmap.org/wiki/Tag:man_made%3Dwindpump
I've corrected the ticket because it wasn't correct to validate handle with manual driver only. It's more handle is expected when mechanical_driver=manual is used (already the case with pipeline=valve + actuator=manual)
I understand you propose to adapt icons background, according to what tag exactly?



Do you have some nice icons (.svg) ?