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

[WIP] Add Boat Rental Quest #5741

Draft
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

kmpoppe
Copy link
Collaborator

@kmpoppe kmpoppe commented Jul 14, 2024

Resolves #5426.

Known issues - feedback required:

  • Order in QuestModule
  • Is this a RARE achievement (only 5.3k boat_rentals tagged today)
  • Answer SVGs range from 0 to 100 in my view as to their fit (I love the floaty duckboat for "pedalboard" ;-)) Wouldn't CC0 images from wikicommons be the better idea? Lacking any kind of artistic skills, "fun icons", as asked for by westnordost in the comments of the Issue is nothing I can provide, any help would be appreciated!

Resolves streetcomplete#5426.

Known issues - feedback required:
* Order in QuestModule
* Is this a RARE achievement (only 5.3k boat_rentals tagged today)
* Answer SVGs range from 0 to 100 in my view as to their fit (I love the floaty duckboat for "pedalboard" ;-)) Wouldn't CC0 images from wikidata the better idea?
@mnalis
Copy link
Member

mnalis commented Jul 14, 2024

  • I'd rather use rowboat_rental=yes/no instead of dinghy_rental=yes/no to avoid ambiguity (Dinghy may use motor instead of oars); especially as both tags (while documented) are not really used currently (single-digit uses worldwide).
  • I think you meant (both in description and tag name) pedalboat instead of pedalboard (I was looking at https://en.wikipedia.org/wiki/Pedalboard in confusion just now 😃 )
  • I don't think it is RARE (but I might be affected by having dozens of boat rental places in few miles around me ATM). They are often not mapped though (esp. when they are seasonal only), and might use alternative tagging instead (e.g. shop=rental, rental=boat, shop=jetski, boat:rental=*, etc.)

Corrected typo Pedalboard -> Pedalboat
@westnordost
Copy link
Member

westnordost commented Jul 14, 2024

Hm, I am sad that you didn't take the pictures that I proposed. I see no reason whatsoever why one should prefer black-and-white icons over the more colorful icons. IMO the style of those also doesn't quite match. Some pictures show waves, some not, some have a more 3D look, others not.

@kmpoppe
Copy link
Collaborator Author

kmpoppe commented Jul 14, 2024

You listed 3 or 4 icons that could be used without any changes, for all others there's work needed that I am not capable of doing artistically to the standards I hold myself to.

This is why this is a draft PR.

@eginhard
Copy link
Contributor

Catamarans would be another common boat type.

@westnordost
Copy link
Member

westnordost commented Jul 15, 2024

Sounds like a hull type rather than a boat type? I mean, many/most pedal boats are catamarans in regards to the hull shape.

And this thing, whatever you'd call that (yes, it's bicycles on top)

is also a catamaran. (... well, I guess it is a very specific type of pedal boat... but unique enough that it should be tagged with another value)

But I guess the usual type of catamaran you get on boat rentals (that you would also call a catamaran) would be sailing boats, right? So, if this is common, maybe better have a value like sailing_catamaran.

@westnordost
Copy link
Member

There's also an activity called "tubing" where you'd rent the tube of a truck tire. Not sure how common it is (compared to rafting/kayaking), I've only seen it in Vang Vieng (Laos):

https://duckduckgo.com/?q=vang+vieng+tubing&t=ffab&iax=images&ia=images

@kmpoppe kmpoppe added the help wanted help by contributors is appreciated; might be a good first contribution for first-timers label Sep 24, 2024
@kmpoppe
Copy link
Collaborator Author

kmpoppe commented Sep 24, 2024

Added help wanted, people being able to contribute good graphics are welcome to do so.

@westnordost westnordost self-assigned this Sep 24, 2024
@westnordost
Copy link
Member

I'll look into icons (maybe next week). Is it otherwise done? E.g. I think certain values should be considered ambiguous and thus if that value is present, the quest should be asked again (such as dinghy).

@kmpoppe
Copy link
Collaborator Author

kmpoppe commented Sep 24, 2024

I'll look through the comments when I get home and check how the answer could be made better!

@westnordost
Copy link
Member

westnordost commented Oct 3, 2024

I added some icons on master. Most are from https://www.svgrepo.com/collection/water-sports-flat-icons/ or mashups from these.

Missing are:

  • yacht - should be a smallish yacht with sails
  • (raft - could be similar to SUP icon)

@westnordost
Copy link
Member

westnordost commented Oct 3, 2024

So, summarizing the TODOs:

  • add yacht
  • add any of which I already added the icons
  • treat dinghy_rental and some others (see taginfo) as ambiguous and hence as "need to ask this quest again"
  • order rentals by commonness (rowboat should go up, many had been tagged with dinghy in the past)
  • use new boat graphics
  • use new quest icon (boat.svg)

@westnordost westnordost removed their assignment Oct 21, 2024
@westnordost westnordost removed the help wanted help by contributors is appreciated; might be a good first contribution for first-timers label Oct 25, 2024
@westnordost
Copy link
Member

Are you still planning to work on this? It would be a shame if I prepared all those icons for nothing.

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 what sort of boats can be rented
4 participants