Skip to content

Add new lane boundary classification type (fence, tension cable system) #658

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.

Sign up for GitHub

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

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

Add new lane boundary classification type (fence, tension cable system) #658

Changes from 4 commits
Commits
Show all changes
6 commits
Select commit Hold shift + click to select a range
7fa0c18
Adding elements to DetectionClassification: NEAREST, OBSTACLE, STRONGEST
FlorianMueller87 Aug 5, 2022
e38fdc2
Added three new elements to DetectionClassification: NEAREST, OBSTACL…
FlorianMueller87 Aug 5, 2022
7088b2c
Added two new elements to LaneBoundary-Classification-Type: FENCE and…
FlorianMueller87 Aug 5, 2022
8fe2f2a
Merge branch 'master' into feature/add_elements_to_LaneBoundary_Class…
jdsika Oct 17, 2022
65b330b
Update osi_lane.proto
jdsika Dec 7, 2022
03e4da2
Merge branch 'master' into feature/add_elements_to_LaneBoundary_Class…
jdsika Dec 7, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
8 changes: 8 additions & 0 deletions osi_lane.proto
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1068,6 +1068,14 @@ message LaneBoundary
// A sound barrier.
//
TYPE_SOUND_BARRIER = 15;

// The lane boundary is a fence.
//
TYPE_FENCE = 16;
Copy link
Contributor

@jdsika jdsika Oct 17, 2022

Choose a reason for hiding this comment

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

A FENCE could be described with STRUCTURE?

Copy link
Contributor Author

@FlorianMueller87 FlorianMueller87 Nov 1, 2022

Choose a reason for hiding this comment

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

At the moment we have mapped STRUCTUR to WALL.


// The lane boundary is a tension cable system.
//
TYPE_TENSION_CABLE_SYSTEM = 17;
Copy link
Contributor

@jdsika jdsika Oct 17, 2022

Choose a reason for hiding this comment

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

This would be the same as a GUARD_RAIL but constructed differently?

Copy link
Contributor Author

@FlorianMueller87 FlorianMueller87 Nov 1, 2022

Choose a reason for hiding this comment

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

It looks something like this: https://gibraltarglobal.com/

}

// The semantic color of the lane boundary in case of a lane markings.
Expand Down