-
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
Different icon for benches with backrest #3187
Comments
I am strongly against this idea. It is not important info, it is not relevant for orientation, it is too detailed for a general purpose map. |
Sounds good to me. |
Nice |
See https://github.com/gravitystorm/openstreetmap-carto/blob/master/CARTOGRAPHY.md#main-goals
Unfortunately showing everything is not feasible. |
Indeed. Last week I mapped a public space which had two picnic tables. Sadly, they are too close together and so only one is shown by the renderer. I can live with that. I understand. There's not enough room to fit in two symbols that close together. I can wish for higher zoom levels so they can be shown, but I don't expect that to happen (my wish for lots of money didn't come true, either, but This is different. Icons are standard sizes and I think the proposed bench-with-back icon fit into that standard size. If I'm wrong and it was a little larger, it could be redrawn to fit. So if it's feasible to fit a bench-without-back icon it's feasible to fit a bench-with-back icon. The hard part (once a symbol that fits the standard icon size has been created) is modifying the carto code to choose one symbol or the other depending on whether or not it has backrest=yes. I can understand "it's too much work when we have so many other issues to deal with" but not "showing everything is not feasible" when in this case showing it is feasible. Nor can I understand "it's too much detail" when, for some people, it may be an important consideration. Just my opinion. You have the final say. |
FWIW, while i agree this level of detail is not something I'd usually
expect from a general map, I believe this could still have been merged
because it clearly is a feature our mappers are interested in. There are
356.735 of this right now, all of them being either "yes" (83%) or "no"
(17%) (with very few other tags like "partial" (17), "Banco" (5), "Bench"
(4), "broken" (4) and some typos of "yes"). Compared to 876 820
amenity=bench, almost every second bench has this attribute. It could be
one of the examples where our data is different to what commercial map
providers deliver, because we have a bottom up approach and map what the
mappers are interested in.
|
It's interesting to me to hear what it means "too much". As long as we can reasonably easy (design is simple, it's not adding clutter and not eating our resources) show something very popular, we should do it IMO. I believe vector styles will help to choose the lighter or richer style, and even personalize it, and I hope this will make such discussions moot eventually. |
Are you sure that it is not result of a large import? Also, I mapped some backrests with StreetComplete, it does not mean that I think that this data should appear on a general purpose map. |
2018-04-30 15:39 GMT+02:00 Mateusz Konieczny <[email protected]>:
Compared to 876 820 amenity=bench, almost every second bench has this
attribute.
Are you sure that it is not result of a large import?
yes, completely sure. What makes you think we could have half of our
benches imported?
See this graph from taghistory.raifer.tech:
|
The graph is missing. |
maybe because it was an svg
here is a png
![screen shot 2018-05-03 at 02 16 28](https://user-images.githubusercontent.com/581288/39555254-2a74200e-4e78-11e8-82e6-4c5278202c65.png)
|
@matkoniecz As there are people people interested in this feature and we don't have test renderings yet, please re-open this issue. Closing it that early was unfair to another users. |
OK, but reopening won't change too much, the most important thing is having discussion and new arguments or decisions. |
OK. Not "all-new" but some new and some rehashing...
|
Pixel aligned icon proposals: Gist link: https://gist.github.com/Tomasz-W/5703292edb2d8b09aab74291636faaf2 If someone is going to make a test renderings, please remember about #3326 |
I share the concerns of @matkoniecz Further concerns are:
|
I agree that some of the icons aren't intuitive. But is it really better to use a small disc instead? For everything? Surely not. So who decides "this gets an icon but that doesn't"? Because somebody has to draw the line if we decide not to provide icons for everything. Which ones get an icon and which don't and why? I don't use certain types of facility so I see no need for them to have icons: I've never bought a bicycle so the icon for bike shops is a waste of map space as far as I'm concerned. So which person decides "this is useful and should appear on a map but that isn't useful" and can do so objectively? It's possible that two or more instances of a non-obvious icon will be present on the map, one with text rendered making it clear what it is and one without text. Users can at least use that to infer what the unlabelled icon is. Whereas if they both have small purple discs there is no clue as to what the unlabelled one is. Your argument indicates that it's time the map legend included the icons as well as the cartography used for roads. Possibly on a separate button so there are two side panels available, the current one and a map symbols panel. |
While I agree that bicycle shops are not a high priority, the icon is quite intuitive, will not be confused with other things, and the purple colour makes clear immediately that this is a shop. That’s different from the proposed bench icon.
At this place I want to quote our guidelines at CARTOGRAPHY.md:
|
This addition is subtle and it is very similar to the bench icon, so I expect no problems. |
@summerluk I'll deal with your points out of order.
I agree 100%. My opinion is that map keys and FAQs about user interfaces mean the map carto or UI is sub-optimal. UIs should have infrequently-asked questions to cover very rare use cases, the rest should be self-explanatory, or discoverable with tool tips. But we live in an imperfect world. Not all of the map icons are intuitive. Maybe some (perhaps even most) of them could be improved. But a pane with map symbols would be there as a last resort. Because with a poor icon and a pane of map symbols I can figure out what something is, but with nothing more than a small disc I cannot do more than place it into a broad category of shop or amenity or some such.
Of all the things on the map I don't use, and am never likely to use, I chose bike shops because I'm well aware that OSM has a high level of input from the hiking/biking/touring community. We should strive to cast aside our own subjective preferences when judging which facilities are useful enough to deserve an icon. Otherwise I demand bike shops no longer have an icon. :) And yes, the purple bike icon is obvious and it's clear it's a bike shop. How obvious would it be if I had my way and it were demoted to a small purple disc? Unless the name of the shop has "bike" or "bicycle" in the name and the map is uncluttered enough at that location for the name to render, it wouldn't be obvious. At least we've progressed from "I don't use this so there's no need to map it as anything other than a disc." to "I find it difficult to figure out what this icon is so lets have a disc even if it's obvious to other people what the icon means." To me it still seems that the problem is that we need better icons and that until we have better icons a map symbols pane would mitigate the problem. In the longer term vector mapping may mean we have fancy JSON trickery so you hover over an object and a pane pops up giving more details of anonymous discs. But when we have vector mapping we'll be able to zoom in so much we can use larger, clearer icons at the highest zooms. |
Personally I dont see how its not obvious what @Tomasz-W's icon is. Theres really nothing else it can be mistaken for that I can think of. So I dont see whats unintuitive about it. If there is a slight chance someone wont get it automatically though the context around it like woods etc would probably make it clear. |
Which woods? So we would need at least 3 icons, one for yes, one for no, and one for the ca 50% untagged? |
I dont know. Its called an example. Pick a random bench in the woods. The point was it probably wont be confused for a store icon or anything and its the places where they are located usually helps. Like, theres only so many icons that would be in a park along a path for instince. People use the surrounding context in identifying things on a map to some degree even if the icon is perfectly clear. The bench icon clearly looks like a bench though. So I dont know why it even matters. At least to me. Otherwise, what else does it look like or how specifically is it unintuitive? |
@matthijsmelissen, I don't think SomeoneElseOSM is a good example here. His map is more detailed in some respects then ours. For instance he renders street lamps when we don't at the moment. So your really picking and choosing. I also think its a little out there to close this issue based on your own opinion on it when there is clearly support for it and the whole "there is no consensus anymore" discussion is going on. Like @eehpcm has said, where is the evidence to back up your assertions and why is it applicable to this particular icon? The important thing is that it acts as a feedback mechanism for more people add the backrest tag. Which I think it will do. You can't really use the "clutter" or "to many icons" argument here because bench's are already being render. The bottom half of the bench with the back rest is exactly like the normal bench. Your making a major assumption about the intelligence of mappers if you believe they wont make the connection between the two in this case. I'm pretty sure 99% of people are pretty acclimated to how the normal bench icon looks at this point. So I don't see why they couldn't deduce what the new icon. Its not really that different. Otherwise, back it up with some tests or something. Even @polarbearing said he wasn't against it and he's usually pretty bearish on things not being rendered excessively. That should tell you something. |
That’s not true. There have been opinions in favour and there have been opinions against. There is definitively not a clear support for this. Let me add also that among the opinions against this change, if I counted correctly, there are three of the maintainers of this style. |
@sommerluk, notice I said didn't say majority or even equal. There has been a few PRs that I was involved in where the majority where in favor of it but only @polarbearing was against it. Me and the others were perfectly willing to not take action on it until he had been heard out though. Although I know discussion is necessarily killed off by an issue being closed, I still don't see why it is necessary here. Even if three maintainers are against it. There are plenty of open issues going years back that had zero favor but never got closed. There's no reason this one should be unique. Especially considering the discussion was still on going. He could of at least waited until it petered out or the test renders ended up failing massively. Then at least there could have been a reasoned argument for closing it. Where does it say that maintainers have ultimate say over the rest of community anyway? I'll also point out the issue for rendering a cross which was closed at the same time by @matthijsmelissen for the same none reason, when it could have just as easily stayed open also or he could of at least asked what other's thought of closing it. Taking both the issues together at the some time like they where makes me think it was more reactionary then anything. Otherwise, why not indulge me, @eehpcm, and the other people for it by rolling the dice and providing evidence for it shouldn't be rendered. Using the low hanging fruit as it where as reasons makes me think its more cartographic dogma then anything well reasoned though. Which is fine. Personally, id like to see it reopened and at least tested though. If the test turn out bad, Fine. But why kill it off hand before that point? |
Let's count. Basing on comments/ likes:
I didn't count Imagico's comment, because it looks more like an information than a statement in this particular issue. We haven't seen even single one test rendering, and with around 50:50 "votes" @matthijsmelissen closed the issue. I think after this situation he shouldn't have the power to closing them, or at least get some warning as he showed big disrespect for another users. There is no point in our Terms of use (https://github.com/gravitystorm/openstreetmap-carto/blob/master/CONTRIBUTING.md) that maintainers votes are more important than other users opinion, so there were no reason to closing the ticket. |
Decisions should be based on arguments, not on votes. I admit that sometimes it is really hard to judge arguments, but it is not a popularity contest. |
There were reasons given above both for implementing and for rejecting this feature. You may be convinced that it is mistake to reject that idea and all arguments for closing were poor and unsufficient but please do not claim that there were not mentioned. Some were mentioned in a closing comment! |
Do you have citation or formal studies to make an opposite claim? While it would be nice and ideal world we would use high quality studies to guide our decisions this is done very rarely if ever. I am not sure is it a good idea to suddenly demand them from the opposite side, without providing ones supporting your viewpoint. |
I think that it is desirable to make map where being an experienced OSM mapper is not prerequisite to understand it. I made (without even attempt to double-blind, extremely low sample size, confounded as heck etc) some tests and typical person is unable to recognize even seemingly obvious icons. For example many people were confuse by amenity=place_of_worship without icon, amenity=police icon, many shop icons, never noticed that all shop icons are purple so weird purple icons are another shops. People also are generally confused by landuse rendering, bridge styling, tunnel styling, intermittent waterways etc. |
@matkoniecz, yes decisions to a degree should be based on arguments, but none were provided for closing it, at least not one pertaining to this particular issue. It was more a general statement about detail that could literally apply to every open issue here. Although its not popularity contest, it is a community project is it not? So the opinions of the community should be considered and things shouldn't be closed based on a majority opinion or without at least testing it first or having reasoned discussion. That doesn't in any make it a popularity contest. Once again, no reason was given that where specific to this particular issue. It was just a general statement that can apply any issue here. If this particular icon won't be recognizable, then show some evidence for it, but don't use SomeoneElse's map or some general thing about "clutter" as an excuse. Otherwise, we risk every issue and PR dissenting bickering over cartographic dogma. With nothing getting done. I don't think its on @eehpcm to defend his position here. Its on the person that made the general statement about clutter to say how it specifically applies in this particular case. Otherwise, anyone can come along and derail anything by using the same tactic. The responsibility is on the person with the objection to prove why their objection holds water. Not the other way around. Last time I checked there is a feedback mechanism in place for people who have an issue with something on the map to complain about it if need be to. So if its true no one will understand it, why not let it be implemented, let someone complain, then we can revise the icon to be understandable later. Ultimately though its an icon issue though, not a "should this be implemented in the first place" issue. I also think its a weak argument to expect everyone to prove why their imaginary people who do or don't recognize the icon is right. As I said, the icon is exactly like the original bench icon, just with a back rest added to it. Its not a new icon. We aren't reinventing the wheel. Everyone here recognizes that its a bench with a backrest, even the people that are against it, and we are all mappers. Maybe we didn't poll 2000 random people to find if they get it, but when do we? And saying people don't understand tunnel styling so they won't understand a backrest on a bench is a good what aboutism, but they are completely different and have nothing to do with each other. Plus, most people don't understand most things when they are first introduced to them, that's not an argument for or against anything though. The question is will they understand it in this case and I think they will. Since like I said, its almost the exact same icon we are already using with a minor addition. Its not a completely different icon or even really altered. That should be enough evidence. |
You may be convinced that it is mistake to reject that idea and all arguments for closing were poor and unsufficient but please do not claim that there were not mentioned. Some were mentioned in a closing comment! See #3187 (comment) |
TLDR |
As a general rule you can count me as 'boycott' in any attempt to turn design decisions into a popularity contest. The whole idea of voting on design decisions is a transparent attempt to scuttle the argument and evidence based development process. In a venue like this it is always easier to organize and astroturf popular support for a change than to find and present substantial arguments in support of it and to substantially consider and address critique, in particular of course if the change is a bad idea. And to make this perfectly clear - the idea of making design decisions based on popular vote - even if including or being representative for the current mapper community - would be against the documented goals of this style. |
To be more specific: it is trivial to ask people on some forum/channel/mailing list to come here and vote or to create 20 sockpuppet accounts. In case of making decisions like that voting is not a proper way to do it. |
There is not mentioned in CONTRIBUTING.md that there is a vote at all. In last consequence, the maintainers of this style take the decisions.
The group of people participating in this discussion is around a dozen. That’s not really representative for the OSM community as a whole either.
No. If someone proposes something, he has to give arguments in favour of it.
No. @matthijsmelissen is a long-term maintainer and contributor of this style and is doing a great work. Honestly, I think it’s disrespect when in this discussion arguments against this change are ignored: When reading some replies here, I have the impression that the arguments against this change are indeed ignored of at least have not been understood completely and there was no real effort to understand them. Who is maintainer of this style is not object to public vote. Your behaviour with @matthijsmelissen was inappropriate. |
OK, cool down for a moment, please. There were too many "hot", emotionally loaded words and statements here lately. What surprised me was sudden closing of a live discussion and I think this was a direct source of this heat. Closing on GitHub does not really stop discussion, but the change made people nervous (yet some wrote insightful responses I admired!), because it was completely unexpected. I'm not sure why some topics bring more positive or negative emotions than the others, but I hope that we can soon continue talking with less tension. |
The reason of closing it quickly was that there was talk about implementing this issue, and I really want to avoid people spending time on building something that will be rejected anyway. So I think a decision was necessary. What would you have preferred? |
It wasn’t unexpected at all. At the time @matthijsmelissen closed this issue, arguments had been given yet. (And after closing, nothing substantially new had been added.) |
The reason and action were perfectly valid IMO - it's what you think is the right solution in controversial case. Somebody will be not happy either way, but even bad decisions are better than lack of them. My guideline in cases where more people invest their time, is to make some decision at the end of the day (as you did here), but to let people know what's going to happen. This way people have a chance to cool down a bit and have the "last minute" to add something or test if you're aware of some important details. The disappointment is expected anyway, but this way I avoid making people angry, which could lead to unnecessary hard words. I also give a warning when people use hard words already and it works like 99% without need of any action (I'm a moderator on Polish OSM forum). It all boils down to the slower timing and communicating intentions IMO, especially when related to closing, locking etc. It was unexpected for me personally, because Matthijs was absent for a long time and I would expect him to take a stand in discussion and reply once or twice before making any action, so I know he is taking care of the problem. Some other factors that I think were important here and contributed to the unfortunate clash:
|
Yes, good point, I will try to do this in the future. |
I think it's safe to unlock the conversation now, it was enough time to cool down. Please, whatever is your view on the subject, try not to make it personal. |
@Adamant36 I hope you will make some test renderings anyway, then discussion would be about something concrete, not about imaginations of certain users. |
@Tomasz-W, I'm still learning how to work with this kind of stuff in the code, but once I get it figured out I'll do some tests renderings to see how it looks. Maybe it will help change a few people's minds to actually see it on the map. If not though, that's fine. Btw, I emailed you like you requested. |
@Tomasz-W For your information, comments like
read like an insult (what is not welcomed here). |
If we don't have any test renderings of something, we can only imagine how would it look on map. |
My guideline when dealing with too much emotions is to not talk about people at all, only about problems. Even "you're OK" might be read as an offense, because irony might be suspected. I'm also curious how would it look like. |
Due to Taginfo, out of 872k benches, 294k of them are tagged with backrest=yes.
https://taginfo.openstreetmap.org/tags/amenity=bench#overview
As sitting on non-backrest benches is not comfortable, I think there should be 2 different icons for 2 different bench types.
The text was updated successfully, but these errors were encountered: