OpenStreetBrowser
/
main
2
0
Fork 7
Code Issues 3 Pull Requests Releases Wiki Activity
Labels Milestones
New Issue

Support comma-separated list for routes #7

Closed
opened 5 years ago by Hufkratzer · 2 comments
Hufkratzer commented 5 years ago

Some routes are suited for differnent activities and tagged with "route=<comma-separaded-list>" (e.g. "route=mtb;hiking;horse"). But these routes are currently not shown here. Example: Bicentennial National Trail in Australia:
waymarkedtrails <-> OSB

Maybe you can just change the queries from "relation[type=route][route=<activity>]" to "relation[type=route][route~<activity>]" (for hinking, mtb and maybe other types). Maybe that will be slower then, but if some routes are missing this isn't too good either.

Some routes are suited for differnent activities and tagged with "route=\<comma-separaded-list\>" (e.g. "route=mtb;hiking;horse"). But these routes are currently not shown here. Example: Bicentennial National Trail in Australia: <a href="https://hiking.waymarkedtrails.org/#routelist?map=13!-35.272!149.1229" >waymarkedtrails</a> <-> <a href="https://www.openstreetbrowser.org/#map=14/-35.2809/149.0877&categories=hiking_routes,mtb-routes&repo=default" >OSB</a> Maybe you can just change the queries from "relation[type=route][route=\<activity\>]" to "relation[type=route][route\~\<activity\>]" (for hinking, mtb and maybe other types). Maybe that will be slower then, but if some routes are missing this isn't too good either.
plepe commented 5 years ago
Owner

Sure, no problem. Should not be slower than normal queries (I guess, Overpass API internally always uses regexp). Done in 973f5b8b61.

Sure, no problem. Should not be slower than normal queries (I guess, Overpass API internally always uses regexp). Done in 973f5b8b61eb5ce7b0f024759dc380ee14c4feb9.
plepe closed this issue 5 years ago
Hufkratzer commented 5 years ago
Poster

Thanks for fixing.

Any idea why I did't get your reply by email?

Thanks for fixing. Any idea why I did't get your reply by email?
plepe referenced this issue from a commit 2 years ago
Merge pull request #7 from plepe/master
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
aboriginal_lands
bug-43
buildings
climbing
cycle_infrastructure
filter
fitness_trail
fossil
heritage
leisure-restruct
master
maxspeed
office
osm-qa
parking-lanes
pattern
pedestrian
places
pt
railway-electrification
shop-bulk_purchase
Labels
Apply labels
Clear labels
Easy first contribution
No Label Easy first contribution
Milestone
Set milestone
Clear milestone
No items
No Milestone
Assignees
Assign users
Clear assignees
No Assignees
2 Participants
Notifications
Due Date

No due date set.

Dependencies

This issue currently doesn't have any dependencies.

Write Preview
Loading…
Cancel
Save
There is no content yet.
Delete Branch '%!s(MISSING)'

Deleting a branch is permanent. It CANNOT be undone. Continue?

No
Yes