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

Bicycle parking type #923

Merged
merged 1 commit into from
Mar 11, 2018

Conversation

matkoniecz
Copy link
Member

@matkoniecz matkoniecz commented Feb 28, 2018

fixes #330

Note, that is not mergeable - only xxhdpi drawable are added and authors are not documented.

But given that it is possible that images will be changed again I prefer to do both once at end once images are stabilized.

screenshot_2018-02-28-15-35-27-575_de westnordost streetcomplete debug

I am not 100% happy about any image but after next round of image searching, playing with saturation, recropping and other bikeshedding I am sending this as I think that it should be good enough (I am obviously willing to make changes if something is bad).

@matkoniecz
Copy link
Member Author

@westnordost I attempted to play with saturation, contrast etc to improve the stand image - is it good enough? I asking as I am most likely to be biased here.

@matkoniecz matkoniecz changed the title Bicycle parking type Bicycle parking type [waiting for review but not mergeable] Feb 28, 2018
@matkoniecz matkoniecz changed the title Bicycle parking type [waiting for review but not mergeable] Bicycle parking type [waiting for review, mergeable after squashing] Feb 28, 2018
@matkoniecz matkoniecz changed the title Bicycle parking type [waiting for review, mergeable after squashing] Bicycle parking type [waiting for review but not mergeable] Feb 28, 2018
@rugk
Copy link
Contributor

rugk commented Feb 28, 2018

You may add "Fixes ..." in the PR description.

@westnordost
Copy link
Member

Looks good 👍

@matkoniecz matkoniecz force-pushed the bicycle_parking_type branch from 69512ff to 654e2b8 Compare March 1, 2018 05:32
@matkoniecz matkoniecz force-pushed the bicycle_parking_type branch from 654e2b8 to 94660f0 Compare March 1, 2018 06:11
@matkoniecz matkoniecz changed the title Bicycle parking type [waiting for review but not mergeable] Bicycle parking type [waiting for review, mergeable] Mar 1, 2018
@matkoniecz
Copy link
Member Author

I added scaled images (I really need to make a script for that, manually rescaling in Gimp and copy-pasting it to proper folders is a poor method) and documented sources. It is now ready for merging.

@westnordost
Copy link
Member

westnordost commented Mar 1, 2018 via email

@matkoniecz
Copy link
Member Author

I found this one - unfortunately it is not working for me and it is no longer maintained ( winterDroid/android-drawable-importer-intellij-plugin#153 )

@ghost
Copy link

ghost commented Mar 1, 2018

I think we should list more bicycle parking types than that. For example, where I live, bollards are quite common. They look like this, and are very visually distinct:

bollards

Some other common ones listed on the wiki are racks, anchors, and trees.

Also, this type of bicycle parking is common in my area:

wavy stands

They are visually distinct from the others, yet consensus says that they should be tagged as stands. As a user who knows nothing about OpenStreetMap, I wouldn't know to classify them as stands.

Same goes for this kind of parking, which I imagine would be intepreted as wheelbenders, since they only secure the front part of the wheel:

stand

How can we minimize confusion?

@rugk
Copy link
Contributor

rugk commented Mar 1, 2018

Just add these images as another element selection and tag as stands, anyway.

@matkoniecz
Copy link
Member Author

matkoniecz commented Mar 1, 2018

Note #330 (comment)

Please only search for pictures / include the following tags: and leave out rack and bollard.

@matkoniecz
Copy link
Member Author

Also, explanation of difference between stands and wheelbenders may be put into "can't" say menu - like proposed for addresses in #917 (I already planned it as a future enhancement).

@ghost
Copy link

ghost commented Mar 1, 2018

@matkoniecz Sorry, I didn't see the discussion there at #330. Though I would still argue that rack and bollard should be included, though, since they're more common than shed and lockers, and bollards are easily identifiable.

@ENT8R
Copy link
Contributor

ENT8R commented Mar 1, 2018

I really need to make a script for that, manually rescaling in Gimp and copy-pasting it to proper folders is a poor method

I did this for all answers for the new building quest (#774) 😄 Such a script would be a really big help!

@westnordost
Copy link
Member

So, the plugin I use is called "Android Drawable Importer"

@westnordost westnordost added this to the v5 milestone Mar 1, 2018
Copy link
Member

@westnordost westnordost left a comment

Choose a reason for hiding this comment

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

Nothing to complain. Will be merged for v5. :-)

@matkoniecz matkoniecz changed the title Bicycle parking type [waiting for review, mergeable] Bicycle parking type Mar 3, 2018
@matkoniecz
Copy link
Member Author

Nothing to complain. Will be merged for v5. :-)

I am curious - is there some reason why merging in future is better than merging now? Are we in a feature freeze for 4.0?

@matkoniecz
Copy link
Member Author

And thanks for a review!

@westnordost
Copy link
Member

Yes. Feature freeze starts with the first beta of any major release. Any update that I make on this version will only be bugfixes or very minor things. Only when that version is stabilized an no major or minor things pop up anymore, I will continue work on the next major version. This is why it took me so long to review the building type quest.

@ghost
Copy link

ghost commented Mar 6, 2018

@westnordost Sorry if spam, but do you have any response to my comments above? I think I was shadowbanned when I posted them, so they were hidden at first.

@matkoniecz
Copy link
Member Author

I think I was shadowbanned when I posted them, so they were hidden at first.

These replies were shown to me and as far as I know Github has no shadow bans.

@westnordost
Copy link
Member

@xotgucuni did you read through the discussion at #330?
Basically, the definition of racks is really unclear (i.e. search for bicycle rack on image search) and may be an umbrella term for all kinds of bicycle parking types. I umderstand it to be like wheelbenders but with a better design that is more secure and doesn't bend the wheels? The problem here is that there is no "typical" rack then, so adding this will rather add to confusion rather than otherwise. Additionally, if my above assumption is correct, there is really not any information value added as opposed to simply tagging as stands. In short:

  1. bad definition in wiki might have burnt this tag. At least it needs to be clarified.
  2. A typical picture would need to be found
  3. Probably no meaningful difference to stands

Bollards are not in there because basically this is one of the many different possible stand designs. In terms of security, it is the same as stands, and this is what matters. I.e wide_stands are also not included.

@ghost
Copy link

ghost commented Mar 6, 2018

@matkoniecz Getting off-topic, but my account was flagged when I created it, and I had to contact support to resolve this. Before that, I had noticed that my comments were hidden if I was logged out.

@westnordost Thanks, makes sense. I did eventually read through #330, but I just wasn't clear why other types were being excluded.

@westnordost westnordost merged commit 529076f into streetcomplete:master Mar 11, 2018
@matkoniecz matkoniecz deleted the bicycle_parking_type branch March 30, 2019 15:43
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.

Ask for bicycle_parking=*
4 participants