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 new quest for parking fee #825

Merged
merged 6 commits into from
Feb 6, 2018
Merged

Conversation

ENT8R
Copy link
Contributor

@ENT8R ENT8R commented Feb 2, 2018

This PR adds a new quest for adding whether there is a fee at a non-private parking.
It fixes #101
I even created a new icon for this quest, but I don't know how to convert the SVG to an Android XML Drawable... I tried tools like http://a-student.github.io/SvgToVectorDrawableConverter.Web/ and http://inloop.github.io/svg2android/ but both didn't work properly... How do you do this?
The quest will be a simple Yes/No quest and looks like this:

@matkoniecz
Copy link
Member

I would ask only for parking with tagged access to avoid "this parking is private" notes.

Access may be tagged by a separate quest.

@matkoniecz
Copy link
Member

matkoniecz commented Feb 2, 2018

Also, it is not clear what should be answered for parking that have fee or are free depending on

  • time (time of year/day of week/hour)
  • status of user (worker, customer, resident, random person)
  • how long car is staying (up to X minutes/hours is free)
  • other factors

@matkoniecz
Copy link
Member

I don't know how to convert the SVG to an Android XML Drawable

Use Android Studio (right click on /app/src/assets/ -> New | Vector asset)?

@ENT8R
Copy link
Contributor Author

ENT8R commented Feb 2, 2018

I would ask only for parking with tagged access to avoid "this parking is private" notes.

I already excluded parkings with access=private. So you want that the query will be like this:
access~ yes|customers|public?

Also, it is not clear what should be answered for parking that have fee or are free depending on

Is there a fee to park here for all persons, no matter how long they stay and what time it is?
(Which would be too long... Or we can add a small description below the question...)

Use Android Studio (right click on /app/src/assets/ -> New | Vector asset)?

results in this:

@westnordost
Copy link
Member

Icon

When saving the single icon to SVG for export, you need to select all and ungroup everything. (Ctrl-A Ctrl+Shift+G iirc). The Android vector drawable importer does not understand svg groups.

Regarding this quest in general, I think we should put some more thought into this remark by @matkoniecz :

Access may be tagged by a separate quest.

Yes, it may be. But note, quests should be designed around questions to ask, not tags to fill in. It's nice, when one coincides with the other. In this case, I am not entirely sure about it. How would the question of that separate quest sound like, and would be the reasonable answer options?
I thought about this a bit today, and at least in German, I am not sure if a paid parking lot would clearly be denoted by people as "public".
Zählt ein Parkplatz mit Parkgebühr als "öffentlich"?

@ENT8R
Copy link
Contributor Author

ENT8R commented Feb 2, 2018

How would the question of that separate quest sound like, and would be the reasonable answer options?

Question: Who can use this parking?
Answers:

  • Private or business people
  • All citizens
  • Only customers

@ENT8R
Copy link
Contributor Author

ENT8R commented Feb 2, 2018

When saving the single icon to SVG for export, you need to select all and ungroup everything. (Ctrl-A Ctrl+Shift+G iirc). The Android vector drawable importer does not understand svg groups.

Great! It works! Thank you!

@westnordost
Copy link
Member

@ENT8R Also du würdest sagen, ein Parkplatz mit Parkgebühr fällt im normalen Sprachgebrauch schon nohc unter "öffentlicher Parkplatz", bzw. das eine hat nichts mit dem anderen zu tun?

@ENT8R
Copy link
Contributor Author

ENT8R commented Feb 2, 2018

I denke, dass die meisten unter einem öffentlichen Parkplatz auch einen Parkplatz mit Parkgebühr verstehen. Gerade Parkplätze mit Parkgebühr sind ja meist für alle gedacht... Parkplätze für Kunden oder Mitarbeiter haben ja so gut wie nie eine Gebühr.

@westnordost
Copy link
Member

Okay

{
@Inject public AddParkingFee(OverpassMapDataDao overpassServer) { super(overpassServer); }

@Override protected String getTagFilters() { return "nodes, ways with amenity=parking and access!=private and !fee"; }
Copy link
Member

@westnordost westnordost Feb 2, 2018

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Perhaps use access!~private|no in case someone tags parking lots' access with "no"?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Or better use access ~ yes|customers|public like @matkoniecz said in #825 (comment) ?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, even better.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ah wait, that would mean that parking lots without any access tagged would not be shown. Hmm, yeah, I guess, that is okay, if there is another quest that asks whether it is public or what else that becomes before this one in priority.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

"if there is another quest" AFAIK we have ticket open for that.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, it's #100. Should I also implement this?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Given that this issue is open it probably would be welcomed...

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You can, but you do not have to to get this PR merged.

@HolgerJeromin
Copy link
Contributor

Especially with hotels we often have customers with fee

@westnordost westnordost merged commit a6a4599 into streetcomplete:master Feb 6, 2018
@ENT8R ENT8R deleted the parking-fee branch February 7, 2018 05:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Quest: Does this parking area[access=customers|public] have a fee?
4 participants