-
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
Add rendering for barrier points #846
Comments
@math1985 Some suggestions based on UTF symbols. The default gate symbol (the UTF symbol for non-equal ≠ U+2260) would be a good start, but you could also take inspiration from similar UTF symbols: U+2260 ≠ (the current barrier=gate symbol) |
Also, is it possible to change the title of this issue to be more generic? It would be great to add a symbol for barrier=kissing_gate to as these are gradually replacing stiles as a more accessable option for pedestrians (you don't have to climb over them) whilst still keeping cattle in the fields and bicycles off the footpaths. You could also render cycle_barrier and cattle_grid. |
Done. |
I don't think the UTF symbols are very clear. Does anyone have a better icon for stiles and kissing gates? Alternatively, does anyone know of any rendering that renders these? |
For stiles, https://github.com/SomeoneElseOSM/openstreetmap-carto-AJT/blob/master/symbols/stile.png works for me. That's not SVG**; it's the same size as the graphic for gate that was in OSM-carto in May last year, and is designed to look "a bit like the gate icon". There's also a cycle barrier and horse stile icon there too (currently experimenting with no transparency on the latter to make it a bit more prominent - the jury's still out on that). I've not used a specific kissing gate icon because (compared to a normal gate) it doesn't restrict different traffic to that restricted by a normal gate (this'd be different if kissing gates occurred with any frequency on wheelchair routes of course, but I don't believe that they do). It's less likely to be useful here, but for info I also went trhough barrier usage in the UK to try and map some of the more esoteric values to more common ones - see https://github.com/SomeoneElseOSM/SomeoneElse-style/blob/master/style.lua for that. ** SVG is not an issue for me personally because I never print anything out; I'm aware that this is a significant use case for some people though. |
@kocio-pl @nebulon42 Do you have any suggestions for an icon for barrier=stile and barrier=kissing_gate? This is how they look like in real life: stile, kissing gate. Something small and simple, similar to barrier=gate, would be ideal. |
Given very small size, complicated (3D) shapes and the fact, that our main gate icon is highly symbolic, I doubt we would be able to do it properly. Such symbolic icon as stile from @althio may be a way and something like "< -" (view from the top) for kissing gate. BTW: I was translating the gate types for JOSM and I even did not know that some of them exist at all and what it really is... |
Kissing gate may be hard to depict in a small frame, maybe start with standard gate representation and move to a more detailed and bigger representation at higher zoom levels? @kocio-pl I also thought about a top view, but I think we should avoid that as most other icons use another perspective. Just a side note to @SomeoneElseOSM comment about SVG: For me SVG is not actually about printing, that is a nice side effect. SVG is so important for me, because it makes the source of the graphic available and thus allows for direct modifications and improvements without having to re-draw shapes all the time. |
we should consider barrier=turnstile |
Maybe render barrier=kissing_gate exactly like barrier=gate? These are typically smaller but I would expect them in areas with lower feature density so rendering them from later zoom level may be not necessary. |
Rendering of barrier=stile, barrier=kissing_gate and barrier=cattle_grid in particular would be very useful for countryside users and would encourage mapping of these features, which can aid outdoor navigation. Another possibility for the stile might be something like the pi symbol π. |
Although the pi symbol is very like the bench symbol... |
I would like to see these rendered too. Is anybody interested in creating an svg image? |
I'm no expert but I will probably be able to have a go at creating/adapting some SVGs if a consensus is reached... So some options for stiles: U+233F ⌿ Options for kissing gates: U+2260 ≠ (the current barrier=gate symbol) Options for cattle grids: |
Re Kissing Gates, I used to (in a different style) just display them as gates but now instead use https://github.com/SomeoneElseOSM/openstreetmap-carto-AJT/blob/master/symbols/kissinggate.png . I tried a number of other possibilities, but the only thing that really worked at such as small size was something fairly "clean" - just a stylised K. It's not an SVG, but feel free to borrow / adapt / whatever. |
I guess we need someone to code it to move this problem out of "issue zone" - would you like to try? If you need some help, feel free to ask. |
I'm new to all this, but would like to help implement these changes. Using cycle_barrier as an example, I believe the necessary change is to add the following code here, starting on line 1043.
The ≓ symbol would then be uploaded as cyclebarrier.svg in symbols folder, here Am I on the right lines? |
Though the shapes can be inspired by anything, including fonts, we need real SVG icons, like this one. Currently we need shapes that could be recognized and this is not the clear one. |
Two options are in the attached zip file |
Alternatively, a simple circle symbol, like for bollards. Perhaps slightly larger? Code would, I think, be:
|
Discussion summary: UTF symbols inspiration:
Icons proposed/ needed: |
I like full height turnstile, the normal turnstile is hard to read, maybe a
modification of the full height turnstile could make it easier to
understand (something like removing all horizontal lines but the second
from below, and cut the vertical pole at the third horizontal line from
below).
both stiles are also hard to recognize.
kissing gate works for me
motorcycle barrier is unclear (seems to be a narrow part of the road, but
it remains unclear whether you can pass, looks like this:
http://www.fuerboeck.at/typo3temp/pics/05c447c697.jpg )
cycle barrier and cattlegrid might work, but only if aligned with the way
that passes the barrier.
|
Ehh... I forgot to pixel align these icons. I've designed them long months ago, and at that time I didn't even know about pixel aligning thing. Now I see that some of them are blurred. I'll upload updated versions of them today, but I think we can reject some of them at this stage - eg. 'cattle grid 1' and 'stile 1' are too big and too prominent IMO. |
I was thinking some of them didn't look pixel aligned. I'll test the new ones when you add them. Also, I agree about the icon sizes. They were especially to big at z17. |
We need some test renderings of big european cities to check which zoom level would be proper to starting rendering these features. |
Other than cycle and motor cycle barriers you will not find these objects in cities. |
@Adamant36 I think it was a kind of 'false start' with opening #3450 at this stage. I've made some icons designs in different shapes and sizes (see #846 (comment)) to choose which one would be the best. We need test renderings for each one on different backgrounds and surroundings to rate it. |
@Tomasz-W, there's nothing saying it can't still be discussed and that things can't be added/subtracted from the PR if need be. Personally, I think cattlegrid/stile is good with the smaller icons and at z17. As the other ones are two small and they aren't city features and wont clutter the map at that zoom level. So, I don't see any reason why those two can't be done in a separate and get merged as is. Whereas, I'm perfectly willing to take out the other ones. Except they don't have multiple icons to decide from and only the zoom level. Which as I said, I think should be worked out after they are on the map for a while and people can provide real world examples of where they are an issue, because Its fine and dandy if I can find two examples motorcycle barrier being cluttered somewhere, but it might just be an outlier. Plus, the discussion of zoom levels started before what icons we where going with was even decided on. Which is putting the cart before the horse. I'm not going to test zoom levels on an icon we haven't even figure out if we are using or not. I am fine removing cycle barriers/motorcycle barriers/turnstiles from the PR if need be and having it just be cattle grid/stile. There's no reason to hold up those two icons when they are good to go though. Plus, as I have said before. I think zoom levels versus being rendered in the first place are separate issues. Especially for issues with multiple items that all need their testing. Really, each barrier type could be its own issue/PR. Especially if I'm the one doing the coding for it and that's how I prefer it. Its not exactly a simple process. There's a lot to keep track of with a normal issue. Let alone one involving this many things. Also, your the one that said PRs are good sometimes to help things move along and that issues could be split into multiple PRs in the first place if need be. So, I don't see what the problem is. |
I uploaded eg. 3 different versions of motorcycle barrier icon, but you took only one for PR without any test renderings provided in issue, so I felt like it was forgotten. |
Oh, my bad. I must of missed them I was looking through the different icons. I there was only the older one and then a new pixel aligned version. Which I included in the PR. Now that I know, I'll add the [wip] tag to the PR and do more tests on it. |
Task list:
Icons files in #846 (comment) |
@Tomasz-W, there's only 393 motorcycle barriers and 845 full height turnstiles mapped. Which is pretty low. So I think we should skip rendering them for now until more are added. Especially motorcycle barriers. What do you think? |
Re motorcycle barriers, the tagging is not "wrong", and deliberately not rendering will just encourage people to tag for the renderer. Why not just render the same as cycle barrier? |
@Adamant36 I think that even with a small usage of these tags it would be good to have them on a map, otherwise it shows a straight wall or footway without any barrier, so map users are lied like there is no barrier there, so I would add them also to make rendering complete. |
For me it makes sense to add some less popular objects if they are clearly documented and make some system together with those which are popular. Systematic approach helps to keep the inevitable chaos within sane limits. |
Especially in cases where some quite similar ones are rendered, to avoid encouraging tagging for renderer (like relatively rare shingle in #1217) |
@Tomasz-W, I think I'm going to do a PR for the remaining barriers with z17 since id like to get it done and thats the default for the all the rest. Then we can modify it later on in the future with a new issue if need be. Wait, I take that back. I guess I have to test the three different types of motorcycle barriers etc first. My bad. I'm still going with z17 though. |
@Adamant36 Yes, task list is here: #846 (comment) and icons to test are here: https://gist.github.com/Tomasz-W/cea0236a9f80f5592a98a8593bc9b989 There is also reuse bollard rendering for turnstiles to test, because I'm still not sure if it won't make big mess in centres of big european cieties, where there is a lot of them underground. |
Alright. That might be a good call. I plan to test motorcycle and cycle barriers first. Since there's three motorcycle barriers to choose from. Then go from there after we decide on those. |
I slightly prefer cycling v1 as well. |
@Adamant36 Thanks for this nice gift in Chrismas morning! ;) My picks are:
|
@Tomasz-W, Your welcome. Merry Christmas ;) |
|
Only important to showing access for pedestrians. It would be kind of off if they were shown bigger then normal gates though. So I really dont know. But reusing the normal gate icon wouldnt neccesarily make the allowed foot access clear. Aside from that, I doubt cows are checking OSM to see were gates they cant get passed are. So its probably superficial to show it. Although we do it with cattle grids. |
great you are picking this up, it would be useful to also see the direction when they are oneway
|
barrier=stile is not rendered. It should be where ever gate is being shown. Knowing where stiles are on country footpaths is useful when walking.
See also https://trac.openstreetmap.org/ticket/3546.
The text was updated successfully, but these errors were encountered: