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 whitewater putin, egress, and hazard to POIs layer #599

Closed
6 tasks
nvkelso opened this issue Mar 13, 2016 · 3 comments
Closed
6 tasks

Add whitewater putin, egress, and hazard to POIs layer #599

nvkelso opened this issue Mar 13, 2016 · 3 comments

Comments

@nvkelso
Copy link
Member

nvkelso commented Mar 13, 2016

pois layer:

  • whitewater=putin, zoom 14
  • whitewater=egress, zoom 14
  • whitewater=hazard, zoom 15
  • whitewater=put_in;egress, zoom 14, but with kind of put_in_egress
  • whitewater=rapid, at zoom 15.

roads layer:

  • whitewater=portage_way is primarily ways (not nodes). Let's try adding those to the roads layer under the general path High Road moniker, but with whitewater:portage_way details. Show them when paths show.

See also:

@nvkelso nvkelso added this to the v0.10.0 milestone Mar 13, 2016
@nvkelso nvkelso added the ready label Mar 31, 2016
@rmarianski
Copy link
Member

There are some other tag combinations, with the most popular being whitewater=rapid. Should those be included too? See taginfo.

@rmarianski rmarianski self-assigned this Mar 31, 2016
@nvkelso
Copy link
Member Author

nvkelso commented Mar 31, 2016

I'm down for also adding rapid, at zoom 15.

portage_way is primarily ways. Let's try adding those to the roads layer under the general path High Road moniker, but with whitewater:portage_way details.

The others seem long tail that we can ignore for now.

I've updated the zoom range recommendation in the primary issue description, too.

@nvkelso
Copy link
Member Author

nvkelso commented Apr 20, 2016

Not seeing example whitewater POIs in Europe :\

@nvkelso nvkelso self-assigned this Apr 20, 2016
@nvkelso nvkelso removed their assignment Apr 30, 2016
@nvkelso nvkelso closed this as completed May 1, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants