-
Notifications
You must be signed in to change notification settings - Fork 819
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
Rendering of amenity=bbq #2996
Comments
Ref #108 but no duplicate |
We simply have no icon for that. Somebody has to design it first or find suitable vector shape on a proper license, which will look good as a 14 px image. |
Potlatch 2 has a pull and drop BBQ icon in its amenity selection? |
I probably would prefer to use a 'bbq' instead of an icon as it is far more descriptive. There are a number of examples where a node has been labelled or named as 'bbq'. |
We have specific requirements for icons: We use text for labels (individual names) when they exist - "bbq" is not a universal symbol. |
I would expect that in most cases this is tagging for renderer and that such names should be deleted. |
I agree but if amenity=bbq doesn't do anything whats a mapper going to do?
…On 28/12/17 21:09, Mateusz Konieczny wrote:
There are a number of examples where a node has been labelled or
named as 'bbq'.
I would expect that in most cases this is tagging for renderer and
that such names should be deleted.
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
<#2996 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AhPipFkiS9k8XCUK141adH6FGAFVF1lgks5tE2jAgaJpZM4RMruZ>.
|
A mapper should add the proper data, but she can also propose/design rendering for suitable map styles, as you just did. Now we need the icon - for example leisure=firepit has been proposed and properly designed, and because of that we could add it soon. |
How do I put an SVG image in this post. I only seem to get a url? |
You can just make a gist for example: |
Could you have a look at this idea for a bbq icon and perhaps change the electric sign to a couple of candles? I have only just loaded inkscape and I have a long learning curve. https://gist.github.com/geesona/9fb46d8467468eeb31d733e0129c7fe2 |
I don't see it as a recognizable shape, but if you think it would be good, you have to test everything at 14 px. |
I have made a few changes and it is now at 14 x 14 I hope. https://gist.github.com/geesona/a5912e34c377d644e088889ecf40d09d |
This one looks better. |
Is it possible to get someone to check the icon for us? |
I'm sorry, I'm currently trying to get few other things to work, so not at the moment. Maybe somebody else is willing to do it? |
One idea for the icon- use the same fire logo as used for the fire fighters, just changing the colour. The fire fighters logo is black, for a bbq it could be blue, as in the tourism icons. This will suggest initially a gas fire, then a cool fire or controlled one. The fire pit icon is not appropriate. |
A blue flame would be ok if it were included in the map key because it would not be immediately identifiable as a bbq. Infact there are a whole lot of icons that are not in the map key? |
I've tune up @geesona draft. |
It might have a sense, however I was also thinking to test this shape: #1519 (comment) What do you think about it? |
All though I like the icon above it does depict a portable bbq with a wheel but all the ones in the parks are very permanent structures and I do like the one by Tomasz-W above. |
For me this shape is just easier to recognize, which might be more important than trying to precisely depict how this object looks like. But if people think it's clear enough, I can go with your design too, because it's quite good. |
I also think that icon of amenity=bbq shouldn't suggest a grill. Gist with my icon project: https://gist.github.com/Tomasz-W/4ff000df11e004abf24d5e77f5b8db62 |
OK, let it be your way than. 😄 Could you make this icon proper (clean SVG instead of the Inkscape bloat, joining the elements, removing white background etc) and maybe prepare a PR with it, if possible? That would be great. |
@kocio-pl Done in the same Gist links for amenity=bbq and historic=wayside_shrine also. |
Anybody wants to prepare a pull request with this icon? It should be rather easy and I can help. |
I will have a go at the pull request but I have no idea yet of what is required or where we are going? |
Great! Steps needed for rendering of bbq:
In order to prepare the PR you need to make a fork, modify the code and test if it really works. That's why testing environment is needed and this is probably the easiest way to do it: https://github.com/gravitystorm/openstreetmap-carto/blob/master/DOCKER.md Is it clear or you need some more guidance? Try to be precise, it will help me to give the proper answers. |
Problem with Docker: E: Package 'docker-ce' has no installation candidate |
OK, I see - did you take 4 steps to set up the repo first?: https://docs.docker.com/install/linux/docker-ce/ubuntu/#set-up-the-repository |
Yes. Everything was following well until the install. Perhaps the server is down and I need to try again tomorrow? I looked at trying the .deb file but was also unable to find that. |
Let's check the state of your repos: cat /etc/apt/sources.list | grep docker My output (I use 17.10 version, hence Next thing - do |
arthur@arthur-Aspire-5750G:~/openstreetmap-carto$ cat /etc/apt/sources.list | grep docker deb-src [arch=amd64] https://download.docker.com/linux/ubuntu xenial stablearthur@arthur-Aspire-5750G: E: Package 'docker-ce' has no installation candidate I do not understand the problem, where is W: and E: ? |
I am using Ubuntu 16.04 |
You're not 😄 , because you have used |
I did it again with sudo: arthur@arthur-Aspire-5750G:/$ sudo apt update E: Package 'docker-ce' has no installation candidate |
It looks like it ignores this repo - maybe the problem is with the key or HTTPS repos? You can also try older method (maybe just or just download and install the package manually: |
I have tried all that with no success. The manual package said I needed i386 not amd64 but I did a search of the download web site but there was no sign of an i386 version. The install docker-engine had a very similar result to the install docker-ce. A frustrating few hours. |
I had a glass of wine to cheer me up. While chasing around I found that there were a lot of old linux images on the machine so I got rid of lots of them, then I noticed that they were 32 bit images! |
So it must be architecture problem... While it's possible to have 32-bit Docker, Kosmtik works only in 64-bit world. |
I now have a 64 bit machine and have loaded docker. Now struggling going through the documentation. I shall be absent on and off between 15 March to 15 April. |
PR added |
I have done a fair amount of mapping contributions and I am wondering why the amenity=bbq is not being rendered.
The text was updated successfully, but these errors were encountered: