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

add rendering for amenity=vending_machine #1561

Closed
yarl opened this issue May 20, 2015 · 46 comments
Closed

add rendering for amenity=vending_machine #1561

yarl opened this issue May 20, 2015 · 46 comments

Comments

@yarl
Copy link

yarl commented May 20, 2015

It's very nice, that we have waste baskets on map. I think, that vending machines also should be added, since they are also helpful.

  • icon: SJJB proposal (definitely too big, but good starting point)
  • label: name=*, then, operator=*, then ref=*
@kocio-pl
Copy link
Collaborator

There's a lot of them (like 58k), so I have no doubt. However I think that maybe showing what can you buy there - if the related shop icon is available - may be more useful than generic vending machine icon?

@matkoniecz matkoniecz added this to the New features milestone May 20, 2015
@yarl
Copy link
Author

yarl commented May 20, 2015

Sure, that would be great, but I try to be realistic 😄

For addition: top values for vending=*:
Screen

@HolgerJeromin
Copy link
Contributor

Is a generic vending machine really useful?
On shops the name gives the missing information, but that doesn't work for vending machines.

@matkoniecz
Copy link
Contributor

I think that in this case value of rendering it is rather low (though waste baskets are rendered) and rendering it in understandable way would be problematic.

@pnorman
Copy link
Collaborator

pnorman commented Sep 19, 2015

I'm inclined to agree

@HolgerJeromin
Copy link
Contributor

At least white list only some vending values is IMO needed. But that has to wait for a db reload.

@kocio-pl
Copy link
Collaborator

For me this is dead easy, because we already have clear analogies mentioned in #1745. Progression of size/importance would go like this:

  • bank (z17) > ATM (z18)
  • post office (z17) > post box (z18)
  • prominent shop (z16) > typical shop (z17) > vending machine (z18)

(there are also few other examples of such progression)

As for the icons - I prefer reusing shop icons, because they're selling the same things just without a man behind a counter. It would be nice if we find acceptable general icon, but it's not a priority for me now.

I still haven't figured out however how to tag and render "paczkomat" (machine for sending/receiving parcels), which is very similar and gaining traction (at least in my city). I also don't have a clear vision of CDM (cash deposit machine) - should it look exactly like ATM and how should it be tagged?

[EDIT:] CDM tagging should not be an issue - amenity=atm + cash_in=yes is exactly for this. Of course we may still want do render it in a different way, but I'm not interested in it now.

[EDIT 2:] "paczkomat" is packstation and is a subtype of a vending machine indeed. Tagging scheme is not clean (vending=parcel_pickup;parcel_mail_in), but the number of uses is quite high (~1.5k).

@pnorman pnorman closed this as completed Sep 20, 2015
@pnorman
Copy link
Collaborator

pnorman commented Sep 20, 2015

Closing, for reasons above

@Tomasz-W
Copy link

Packstations icon proposal:

vending parcel

@kocio-pl
Copy link
Collaborator

kocio-pl commented Mar 22, 2018

I think packstation tagging is broken, since it has nothing to do with vending and uses a name of a particular brand - here is a draft of my solution (a bit old, but I plan to finish it):

https://wiki.openstreetmap.org/wiki/Proposed_features/amenity%3Dparcel_box

Which also makes tagging of similar objects more coherent and universal:

https://wiki.openstreetmap.org/wiki/Proposed_features/amenity%3Dlaundry_box

The icon is interesting. Maybe using [ as with tickets vending would make it more clear. On the other hand it breaks the rule of not using perspective, so maybe gift shop icon could be used instead as a base.

@Tomasz-W
Copy link

I though about use this icon for: vending=parcel_pickup;parcel_mail_in , vending=parcel_pickup , vending=parcel_mail_in . In total they have 3,4k uses.
I didn't use "[" because there is no "[" in the real shape, as it is in parking/ public transport ticket machines.

@Tomasz-W
Copy link

@matthijsmelissen Can you re-open this issue due to #108 (comment) ?
It passed almost 3 years since oryginal discussion and usage of amenity=vending_machine has grown from 58k to 105k. I think we should consider rendering of most popular and most useful types like:

@matthijsmelissen
Copy link
Collaborator

Done.

@Tomasz-W
Copy link

Tomasz-W commented Jul 2, 2018

vending=parcel_pickup;parcel_mail_in , vending=parcel_pickup , vending=parcel_mail_in :

vending parcel

Gist link: https://gist.github.com/Tomasz-W/18032cc646055bf17b866cb4eb48fa48

@kocio-pl
Copy link
Collaborator

kocio-pl commented Aug 3, 2018

Vending machine icon in direktvermarkter project (see https://farmshops.eu ) on MIT license:

https://github.com/CodeforKarlsruhe/direktvermarkter/blob/master/img/a-pin2.png

@Tomasz-W
Copy link

Tomasz-W commented Aug 5, 2018

This shape rather wouldn't work. In 14x14 size it would look like some freezer, safe box or bulleted list, but surely not a vending machine.

@Adamant36
Copy link
Contributor

@Tomasz-W, oh yeah. Duh. My bad.

@Adamant36
Copy link
Contributor

@Tomasz-W, So, I get that you don't think an icon would work for the default vending machine, it appears though after a lot of testing that because of the way the code/rendering priority works, if there are icons for a sub type like this it will render those icons on the default tag if it doesn't have anything to render for its self. Its a similar thing as to why the code for recycling had to be changed to "does not equal." I don't think there is a way to get around it without at least having a dot or something. I could be wrong though. Aside from that, It would be good if the default was rendered with something anyway, just so the names will render if nothing else or so people will know they are there and need a vending=* tag.

@kocio-pl, do you know how we might get around this issue? Its a little out of my realm of knowledge. Personally, id prefer plain vending machines be rendered somehow, instead of just using a code trick to get around it, even if its just a dot, but Tomasz-W thinks an icon would be to generic. So..

@dieterdreist
Copy link

dieterdreist commented Oct 8, 2018 via email

@kocio-pl
Copy link
Collaborator

kocio-pl commented Oct 8, 2018

There are 2 basic approaches, both with their own limitations:

  • rendering all of them as generic automate, but we don't know what they sell
  • rendering them similar/the same as shops (tickets/newspapers/drinks...), but most of them will never be shown

Of course there can be some mix of both or maybe some other ideas.

@dieterdreist
Copy link

dieterdreist commented Oct 8, 2018 via email

@kocio-pl
Copy link
Collaborator

kocio-pl commented Oct 8, 2018

This is a case when size rule would be helpful - ticket shop on z17+, ticket machine on z19+, so you can easily guess what it really is beside knowing what you can get there.

@kocio-pl
Copy link
Collaborator

I believe vending=parcel_pickup;parcel_mail_in is bad tagging scheme:

Because these machines are not vending machines, the alternative tag amenity=parcel_lockers has been proposed.

https://wiki.openstreetmap.org/wiki/Proposed_features/Parcel_lockers

I also tried to propose scheme but did not finish it:

https://wiki.openstreetmap.org/wiki/Proposed_features/amenity%3Dparcel_box

@jragusa
Copy link
Contributor

jragusa commented Dec 18, 2018

I'm testing icon of @Tomasz-W and I would like to know if we need to also include name tag or just the icon ?

@kocio-pl
Copy link
Collaborator

I guess with a name. However without proper tagging it does not make sense for me.

@Tomasz-W
Copy link

@jragusa @kocio-pl I've checked some european cities in Overpass Turbo and vending machine's name tag in most of cases is like "Packstation" for packstations, "Tickets" for ticket machines or machine operator company name wrongly put in name tag, so I wouldn't render names here as it could mess a map with unnecessary unuseful information.

@kocio-pl
Copy link
Collaborator

Let's make some order, I got lost a bit... Which tagging scheme do you like to render and using which icons?

@Tomasz-W
Copy link

@kocio-pl See #1561 (comment)

@jragusa
Copy link
Contributor

jragusa commented Dec 20, 2018

vending=public_transport_tickets
https://www.openstreetmap.org/node/3735097197
before
public_transport_before
after
public_transport_after

vending=parking_tickets
https://www.openstreetmap.org/#map=19/46.19375/6.23334
before
parking_ticket_before
after
parking_ticket_after

vending=excrement_bags
https://www.openstreetmap.org/node/4461667570
before
excrement_before
after
excrement_after

@Adamant36
Copy link
Contributor

Maybe the transport ticket could have a T in it to signify its for public transportation like the parking ticket has a P. Otherwise, it seems neutral. Also it might help to crop the images a little more, because it took me a minute to figure out where the icon was in the after image.

@Tomasz-W
Copy link

As "P" letter is intuitive for parkings, I think "T" wouldn't be an intuitive letter for public transport. Notice also that this machines with public transport tickets are usually near stations or tram/ bus stops, so it makes them more intuitive to understand.
@jragusa I think they should be in man-made gray as amenity-brown makes them looking more important than they actually are. Btw. @Adamant36 any news about #3326?

@jragusa
Copy link
Contributor

jragusa commented Dec 21, 2018

marker-fill: @man-made-icon;
public_transport_after_manmade

parking_ticket_after_manmade

excrement_after_manmade

@Tomasz-W
Copy link

Tomasz-W commented Dec 21, 2018

I'm always not sure about proper colour for the features like that: at the one hand they are small and simple, but at the other they may be very important for cityzens and also some of them are more 'interactive' somehow, so both amenity-brown and man-made-grey have advantages and disadvantages for vending machines. Another thing is that when I look on test renderings above I think if we would decide on rendering vending machines in man-made-grey we should change also amenity=atm icon to grey to keep map logical. I would like to read the opinions of others.

@kocio-pl
Copy link
Collaborator

kocio-pl commented Dec 21, 2018

In general I think of brown as something you can interact with (a service), while grey is just for watching (also a landmark), so I would render them brown.

@jragusa
Copy link
Contributor

jragusa commented Dec 21, 2018

Based on #3326 and #3402, I would leave amenity=vending_machine in amenity-brown, otherwise we also need to move amenity=atm in man-made colour.

I also would propose to render operator as label for vending=public_transport_tickets. 71.86 % of them have an operator tag and it's useful to distinguish them when there is more than one transport network. In the below example, there is both railway and public transports and the machine is dedicated to railway transport.

However, I noticed that operator is only render when a name tag is also added. For example this one does not have any label in contrast to this one. Is that intentional ? EDIT : already reported in #3419.

In the same example, I have to add a name tag to render the label
public_transport_after_brownlabel

@Adamant36
Copy link
Contributor

@Tomasz-W, I just did a PR for it. I didn't include anything related to recycling so it won't get derailed like last time, but everything else is included.

@Tomasz-W
Copy link

Tomasz-W commented Jan 4, 2019

@kocio-pl What about packstations? I know you've proposed
https://wiki.openstreetmap.org/wiki/Proposed_features/amenity%3Dparcel_box
but it seems to be stuck (and another thing is that propably "packstation" would be much more better name for them, compare google images of both names). Can you start a discussion about potential amenity=packstation tag (instaed of amenity=vending_machine for them) on Mailing list?

@kocio-pl
Copy link
Collaborator

kocio-pl commented Jan 4, 2019

I'm not ready to start any discussion on the list. Anyone else?

The name however should be different, since "Packstation" is used by DHL, so the best thing should be to go with already existing proposals - my proposition or this one (both are OK for me, maybe lockers is better than just box, since there are many of them):

https://wiki.openstreetmap.org/wiki/Proposed_features/Parcel_lockers

@Tomasz-W
Copy link

Tomasz-W commented Jan 4, 2019

Ok, thanks. I would go with "parcel_lockers". I added my comment on the talk page . I hope this proposal won't get stuck as yours one.

@kocio-pl
Copy link
Collaborator

kocio-pl commented Jan 4, 2019

You might contact the author, because if nobody will talk on the Tagging list, I feel there's little chance that this would be ever used wider:

https://wiki.openstreetmap.org/wiki/User:SelfishSeahorse

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

No branches or pull requests

10 participants