-
-
Notifications
You must be signed in to change notification settings - Fork 366
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 fuel service information #2822
Conversation
I think disable for @dbdean any opinion? |
Note that Also, is it meaning even really clear? |
I'm taking this in consideration, that's why it's a draft. EDIT: I found a discussion about this from 2015.
The term is on the Cambridge dictionary, so I think it is. Maybe a better term could be used for the tag, but for the quest I think that's what it should ask. |
Well, we do have one in Armidale that pump fuel for you. My wife loves it, but it's to weird for me so I avoid using them :) I don't recall seeing one before in any other place I've lived though. |
Just a short comment on the icon: |
Whilst I am aware that one or two US states make self serve illegal, this should be disabled for the United Kingdom and probably most of mainland Europe where unstaffed pay at pump is very much the norm. |
What was outcome of full_service discussion? From cursory look it appears that meaning of that term is quite unclear and/or duplicate of https://wiki.openstreetmap.org/wiki/Tag:amenity%3Dfuel#Service mentions it right now but without any hint about meaning. |
+1 It should be disabled too for non-continental French territories (such as Réunion, Guyane, Guadeloupe, Martinique, Mayotte) where, contrary to mainland France, all pumps are staffed. |
There were a lot of great ideas and opinions, but there was not a precise outcome, the full service tag is both praised and hated as I can understand, because there are only few countries where this could be useful.
|
Sorry for not getting back at you earlier. I'd say, keep it simple first. Option 2 sounds good:
|
No problem, don't worry, I understand that everyone here has their own life too and StreetComplete is a very big project to deal with, so don't have to apologize 🙂. To understand which countries could benefit from the quest we could ask on the forum if you agree, I think that you being German can confirm it's not useful for you, on the other hand I can say that for me here in Italy would be useful (I've seen some gas stations which were closed at night, since they are full service only, also in some major cities, usually very small ones). "Can you refuel using self service at GAS STATION?" (please help me with the wording) image with the stickman pumping their own gas image with the stickman pumping their own gas and a worker image with a big red circle (the one used for no pedestrians allowed) on the stickman pumping their own gas By the way, congratulations for your State of the Map talk, you did a great job! 🥳 |
Many rural petrol stations in the UK are Full Service only. You're not allowed to "pump your own gas". It's also important that the wording does not confuse users with the difference between Self-service and Pay@Pump, which is becoming very popular in the UK. |
Thanks for the information @timothywashere, sorry if I noticed your comment just now! By the way I was thinking about removing the full service question altogether and just leaving the self service one, that I think has more relevant use cases, as usually it's obvious that where there's no self service, there must be full service (your statement is an example). I would really like to show it everywhere, as what you said is basically my use case for Italy. I'd like to find a way to show the quest only to relevant gas stations all over the world except the few cases where it's restricted by law (example in USA). I don't know if there's a way to find elements near the gas station or if that would be useful, but that would be great. I proposed this quest when there was the lockdown and maps like Ca reste ouvert only showed gas station with the self service tag as open. I still think that having this information could be useful for some purposes in the future, but implementing the quest doesn't seem as simple as I initially thought it to be... If @westnordost and @matkoniecz agree we could start merging the quest with only some selected countries to give it a push and get more feedback from users that would need it in their countries (the quest selection menu of the app would become an ad space basically). (EDIT: forgot to say but of course I'll first need to make the changes before merging haha, I'm not asking to merge it as it is, just asking for feedback about it becoming basically a very limited quest) |
Just self service yes / no should be an easy yes/no question and would be very easy to implement. The "complicated" part would be to research in which countries it should be enabled. I.e. where do both exist. It is okay to only start with 1-few countries, it is not necessary to research for the whole world. |
Oh, wait, why would covid-19 lockdown be indicative of that? It seems like they/you consider My reading of self_service=yes is that it is about whether the service itself is provided by other humans, or do you do it yourself. I.e. in case of fuel pump, do you put the nozzle into your car reservoir yourself, or if employee does it for you. But not if the human employee is involved with the payment for the goods/services. So I interpret it the same as it is with other tags listed in So I think I think automated=yes is the closest tag to indicate whether you'll have to interact with other humans to finish the job or not. If we want to indicate fully automated fuel station without any humans onboard, it would probably need to be marked (note: In this review I do not even analyze TL;DR: please make clear a distinction whether the quest is about if the users needs to perform service themselves (ie. |
Yes, that was exactly what I meant with difficult, not the code per se, that fortunately, I must give credit for you and all contributors who worked on it, is very simple to write for basic quests even for a beginner like me 😁
I don't know why but it was like that, if I recall correctly there was a law here in Italy which demanded self service gas station (I'm not sure, might be wrong, more than a year passed), so maybe there was a similar one in France.
Yes, usually I meant self service without cashier, but you're right I have sometimes found self service where you paid to a cashier but pumped your own gas... You just made me think about it, I had never considered this 🤦♂️
Mh... Maybe we need more than a quest. I just assumed
You did well, I think that dealing with non standard tags might be dangerous at the moment... It's not very clear its utility and meaning so I think it's better to avoid it for now (to be honest I'm starting to doubt about its utility myself). Thanks by the way for bringing to light the matter! |
I think if a station has self service, and accepts credit cards, it can be *mostly* assumed that you do not have to interact with the cashier. At least in the US.
|
In Germany, all stations have self service. You still need to go into the shop and pay for the gasoline. |
} | ||
} | ||
|
||
override val enabledInCountries = NoCountriesExcept("IT") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Because of #2822 (comment), maybe it should be enabled in the UK?
override val enabledInCountries = NoCountriesExcept("IT") | |
override val enabledInCountries = NoCountriesExcept("IT", "UK") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Mh, I did not understand if it's really useful or not as it is now, @timothywashere what do you think?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Mh, I did not understand if it's really useful or not as it is now, @timothywashere what do you think?
I think it's useful and would like to see it.
Despite the fact that I used the phrase earlier, "Pump your own gas" isn't really appropriate in British English, as an Americanism it would offend a lot of people. I'm not sure what is a succinct correct phrase, "Dispense your own fuel?" I suppose.
If anyone objects to this quest in the UK an option would be to only shown the quest for small or unknown brands. You wouldn't get self-dispense at BP/Shell/Esso/Tesco/Morrisons/Asda/Sainsburys. But I am happy to answer this quest for all petrol stations so don't add that functionality unless someone else complains.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Also one pedantic query. Should it be "Can you.." or "Must you.." . Can implies a choice that isn't there (In the UK anyway).
For self service stations it's usually required that you operate the pump yourself, unless you are disabled and may be able to ask for help.
If it's not self service, then you are forbidden from operating the pump.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@timothywashere Thank you for the answer, not being English I'm unaware of what's correct and what isn't and can't appreciate the nuances between two wordings, if you can suggest a question for the tag I'd be very glad.
I added UK by the way, thank you!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If anyone objects to this quest in the UK an option would be to only shown the quest for small or unknown brands. You wouldn't get self-dispense at BP/Shell/Esso/Tesco/Morrisons/Asda/Sainsburys.
I don't know how this could be handled, maybe just remove all fuels with brand tags altogether?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you for your persistence! The wording sounds a bit off, otherwise it looks good.
app/src/main/res/values/strings.xml
Outdated
@@ -604,6 +604,8 @@ Otherwise, you can download another keyboard in the app store. Popular keyboards | |||
<string name="quest_construction_steps_title">Are these steps completed?</string> | |||
<string name="quest_construction_generic_title">Is this way completed?</string> | |||
<string name="quest_construction_building_title">Is this building completed?</string> | |||
<string name="quest_fuelSelfService_title">Can you pump your own gas at this fuel station?</string> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sounds like you can bring your own gas to pump here.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
<string name="quest_fuelSelfService_title">Can you pump your own gas at this fuel station?</string> | |
<string name="quest_fuelSelfService_title">Can you pump gas on your own at this fuel station?</string> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Mh I don't know how to write it because English isn't my mother tongue 😕 I trust @riQQ 😅
This is a preview (I also changed the icon):
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I propose "Can you pump gas yourself at this fuel station?".
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@timothywashere suggested something along the lines of "Dispense your own fuel?"
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
"Do you dispense your own fuel at .... ?"
"Do you operate the pump yourself at ...?"
In terms of absolute clarity I think maybe:
"Are the pumps at ... operated by the customer?"
But maybe I'm straying too far from the OSM tags and it should be
"Can the customer operate the pumps at ....?"
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, undoubtably "Pump your own gas" is correct in American English.
Hahah I hope this is a good thing 😂 |
The icon is better now! |
Thanks! The problem with the mask is still there unfortunately. I can't find any tutorial on how to make a mask like yours where the icon doesn't show the masked elements in Firefox 😔 |
That's ok, I'll take care of it after merge
On 17 January 2022 10:24:49 CET, naposm ***@***.***> wrote:
Thanks! The problem with the mask is still there unfortunately. I can't
find any tutorial on how to make a mask like yours where the icon
doesn't show the masked elements in Firefox 😔>
>
-- >
Reply to this email directly or view it on GitHub:>
#2822 (comment)>
You are receiving this because you were mentioned.>
>
Message ID:
***@***.***>
…--
Diese Nachricht wurde von meinem Android-Gerät mit K-9 Mail gesendet.
|
app/src/main/java/de/westnordost/streetcomplete/quests/fuel_service/AddFuelSelfService.kt
Outdated
Show resolved
Hide resolved
app/src/main/java/de/westnordost/streetcomplete/quests/fuel_service/AddFuelSelfService.kt
Show resolved
Hide resolved
I think everything was OK with the mask, just that the viewer you used doesn't properly handle masks. For this reason and because masks are only supported at a certain API level in Android (and otherwise they automatically get rasterized by the build process), I ususally "bake" the masks in. I did this now too with your icon. The way I do it in Inkscape: select both the circular background and the thing that should be masked by that circular background, select Path->Intersection (Ctrl+*). Copy the result and undo the last operation. Then, paste: Replace the original with the intersection result. Not sure if there is any mor estraightforward way to do it. |
That was a surprise, thank you very much Tobias for merging! And thanks to FloEdelmann, timothywashere, mnalis, riQQ and anyone who helped out here and in the linked issues (too many people to thank)! (small note @westnordost : I did not update P.S. Thank you for letting me know the Inkscape trick, I think it will be very useful to me, I looked it everywhere, I hate having that mask problem, so thank you very much! |
I already did it myself. FYI this will go into v40. Tomorrow or so I'll release v39.0 |
That's nice, thanks! |
This is the pull request for #1827. I'll just add the updated screenshots here.
SCREESHOTS
Self service quest:Full service quest
closes #1827