-
Notifications
You must be signed in to change notification settings - Fork 62
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
Including lane_edge_reference
on every lane in a road event is confusing
#66
Comments
Along with standardizing the starting lane number in #67, instead of moving This would solve the original issue and simplify the spec, as it removes an enumerated type as well as a field. Unless the option to specify the direction of count greatly lowers the effort for feed producers (please chime in), I think standardizing the counting edge reference and removing the field is the better approach to resolve this issue. |
This makes sense. Keeping this as an option lowers the barrier to entry since both left and right standards are in use. |
@CraigMoore-Sea by "this makes sense", are you referring to moving the |
Yes, moving the lane_edge_reference to the road event. |
Comments received during the V3 Specification Update Subgroup meeting on 4/29/2020:
|
Resolved in #94 |
Including
lane_edge_reference
on every lane in a road event is confusingBackground
The
lane_edge_reference
property defines the start side (either left or right) for identifying the lanes within a road event bylane_number
index. For example, if thelane_edge_reference
is left, thelane_number
0 refers to the leftmost lane.Issue
Including the
lane_edge_reference
is a simple and effective technique for clarifying the often confusing indexing of lanes.lane_edge_reference
only needs to be defined once for each road event, rather than for every lane in the road event as is currently done. Defining it for every lane:Currently (possible scenario)
Note the above is pseudocode
Solution
Moving the
lane_edge_reference
as a conditional field (required if using "lanes") in theroad_events
table would be a simple solution.Example
Note the above is pseudocode
The text was updated successfully, but these errors were encountered: