Skip to content

Adding Express Lane flag to Traffic#1831

Merged
nvkelso merged 4 commits intomasterfrom
unknown repository
Feb 15, 2019
Merged

Adding Express Lane flag to Traffic#1831
nvkelso merged 4 commits intomasterfrom
unknown repository

Conversation

@ghost
Copy link
Copy Markdown

@ghost ghost commented Feb 8, 2019

  • Adding optional flag to identify whether Traffic Flow or Incident information is related to a Express Lane linear.

- Adding optional flag to identify whether Traffic Flow or Incident information is related to a HOV (High Occupancy Vehicle) linear. 
(https://en.wikipedia.org/wiki/High-occupancy_vehicle_lane)
@ghost ghost changed the title Adding HOV flag to Traffic Adding Express Lane flag to Traffic Feb 8, 2019
Comment thread docs/layers.md Outdated
* `is_bridge`: set to `true` when the linear is a bridge
* `is_tunnel`: set to `true` when the linear is a tunnel
* `is_link`: set to true when the linear is a slip-road
* `is_express_lane`: set to `true` when the linear is an HOV (High-Occupancy Vechicle) lane.
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Why is this necessary, can you explain your rational with an example, please?

We currently don't include hov tags on the roads layer features, so seems slightly odd to include it here in the traffic_flow and traffic_incidents layers. While there has been discussion to add lane tags into the tiles, and then hov could be added as part of that, I'm still not sure why it's necessary – except for HD mapping, which Tilezen currently isn't really geared for.

If we were to do this...

The OSM-ism we'd follow is hov instead of express, please say is_hov instead.

See also:

Visually would you sort the HOV traffic over the non-HOV traffic flow lines? Would you want a way to turn off the non-HOV traffic lines? This starts to get tricky fast.

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Hi Nathaniel,
I got confused (I'm British/Irish) and misunderstood what an Express Lane is. So updating the patch with a new commit for the explanations.

However this still won't satisfy this comment:

While there has been discussion to add lane tags into the tiles, and then hov could be added as part of that, I'm still not sure why it's necessary – except for HD mapping, which Tilezen currently isn't really geared for.

Indeed this is lane-specific information, but basically we'd like to be able to have this tag so that the client is aware that the Traffic Flow/Incident information is affecting the Express Lane on a particular road.

Comment thread docs/layers.md Outdated
* `is_tunnel`: set to `true` when the linear is a tunnel
* `is_link`: set to true when the linear is a slip-road
* `drives_on_left`: set to `true` when the country drives on the left, e.g. In the U.K
* `is_express_lane`: set to `true` when the linear is an HOV (High-Occupancy Vechicle) lane.
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

See above, would be is_hov.

But please explain why HOV tag is needed here at all. Plus we're talking about points here, right? So the comment would also need to update.

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Please see my comment above :)

Conor Ettinoffe and others added 3 commits February 12, 2019 11:41
Copy link
Copy Markdown
Member

@nvkelso nvkelso left a comment

Choose a reason for hiding this comment

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

I updated this to the OSM-ism is_hov_lane, reworded the docs, and LGTM for merging.

@nvkelso nvkelso added this to the v1.7.0 milestone Feb 15, 2019
@nvkelso nvkelso merged commit 52da611 into tilezen:master Feb 15, 2019
@nvkelso
Copy link
Copy Markdown
Member

nvkelso commented Feb 19, 2019

@conor-ettinoffe-here I remember seeing followup about this over the weekend, but I can't find now. Can you summarize with a comment here, please?

@ghost
Copy link
Copy Markdown
Author

ghost commented Feb 20, 2019

@conor-ettinoffe-here I remember seeing followup about this over the weekend, but I can't find now. Can you summarize with a comment here, please?

Hi Nathaniel, its okay no longer a point :) I basically was saying I meant Express Lane and not HOV - however according to OSM HOV is a synonym of express lane:

[W] High-occupancy vehicle lanes (sometimes also called carpool lanes, commuter lanes, express lanes, transit lanes). The number of such lanes could be tagged using lanes:hov=*.

So all is clear now. Thanks!

@nvkelso
Copy link
Copy Markdown
Member

nvkelso commented Feb 20, 2019

Okay, sounds good! I agree it's a little confusing, but all is well that ends well :)

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.

1 participant