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 francois.lacombe)

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.

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 francois.lacombe, 5 years ago

Description: modified (diff)

comment:2 by francois.lacombe, 5 years ago

Description: modified (diff)

comment:3 by Don-vip, 5 years ago

Keywords: pump, pump mechanism → pump mechanism

comment:4 by skyper, 5 years ago

Do you have some nice icons (.svg) ?

by francois.lacombe, 5 years ago

Attachment: eau_pompe.svg added

Pump icon for man_made=pump preset

comment:5 by francois.lacombe, 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 Don-vip, 5 years ago

Milestone: 21.04
Owner: changed from team to Don-vip
Status: newassigned

comment:7 by francois.lacombe, 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 Don-vip, 5 years ago

Cc: Klumbumbus 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 Klumbumbus, 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...

comment:10 by francois.lacombe, 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.

Last edited 5 years ago by francois.lacombe (previous) (diff)

in reply to:  10 comment:11 by Klumbumbus, 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:12 by francois.lacombe, 5 years ago

It's mine, CC BY-SA licence :)

comment:13 by Don-vip, 5 years ago

Ah, didn't see it at all, sorry :D It's nice, thank you!

comment:14 by Don-vip, 5 years ago

@François if you find/make an icon for windpump I'm ok to add it too.

comment:15 by Don-vip, 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 francois.lacombe, 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 gaben, 5 years ago

Cc: gaben added

comment:18 by simon04, 5 years ago

Milestone: 21.0421.05

comment:19 by simon04, 5 years ago

Milestone: 21.0521.06

comment:20 by Don-vip, 5 years ago

Milestone: 21.0621.07

by francois.lacombe, 5 years ago

Attachment: eau_windpump.svg added

Icon for man_made=windpump

by francois.lacombe, 5 years ago

Attachment: oil_well.svg added

Icon for man_made=petroleum_well

comment:21 by francois.lacombe, 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 skyper, 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:23 by Don-vip, 5 years ago

I already started to work on this, I'll do it soon.

comment:24 by skyper, 5 years ago

Fine, I can wait and take care of other tickets then.

comment:25 by francois.lacombe, 5 years ago

Thank you

Let me know, I'll do translation in French for next release.

comment:26 by Don-vip, 5 years ago

Milestone: 21.0721.08

comment:27 by Don-vip, 5 years ago

Owner: changed from Don-vip to team
Status: assignednew

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 Don-vip, 5 years ago

Milestone: 21.0821.09

comment:29 by skyper, 5 years ago

Owner: changed from team to skyper

comment:30 by skyper, 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 francois.lacombe, 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?

Last edited 5 years ago by francois.lacombe (previous) (diff)
Note: See TracTickets for help on using tickets.