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

Remove is_globally_unique rule from messages, where it does not make sense #812

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open
Show file tree
Hide file tree
Changes from all commits
Commits
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
4 changes: 0 additions & 4 deletions osi_lane.proto
Original file line number Diff line number Diff line change
Expand Up @@ -38,10 +38,7 @@ message Lane
// The ID of the lane.
// Example: l4 (see reference picture HighwayExit).
//
// \note Note ID is global unique.
//
// \rules
// is_globally_unique
// is_set
// \endrules
//
Expand Down Expand Up @@ -713,7 +710,6 @@ message LaneBoundary
// The ID of the lane boundary.
//
// \rules
// is_globally_unique
// \endrules
//
optional Identifier id = 1;
Expand Down
7 changes: 0 additions & 7 deletions osi_logicallane.proto
Original file line number Diff line number Diff line change
Expand Up @@ -57,7 +57,6 @@ message LogicalLaneBoundary
// The ID of the lane boundary.
//
// \rules
// is_globally_unique
// is_set
// \endrules
//
Expand Down Expand Up @@ -330,12 +329,6 @@ message LogicalLane
{
// The ID of the logical lane.
//
// \note Note ID is global unique.
//
// \rules
// is_globally_unique
// \endrules
//
optional Identifier id = 1;

// The type of the logical lane.
Expand Down
3 changes: 0 additions & 3 deletions osi_referenceline.proto
Original file line number Diff line number Diff line change
Expand Up @@ -28,10 +28,7 @@ message ReferenceLine
{
// The ID of the reference line.
//
// \note Note ID is global unique.
//
// \rules
// is_globally_unique
// is_set
// \endrules
//
Expand Down
1 change: 0 additions & 1 deletion osi_sensorview.proto
Original file line number Diff line number Diff line change
Expand Up @@ -65,7 +65,6 @@ message SensorView
// which are used in the detected features.
//
// \rules
// is_globally_unique
// is_set
ClemensLinnhoff marked this conversation as resolved.
Show resolved Hide resolved
// \endrules
//
Expand Down
2 changes: 1 addition & 1 deletion osi_streamingupdate.proto
Original file line number Diff line number Diff line change
Expand Up @@ -82,7 +82,7 @@ message StreamingUpdate
// Entities that will no longer be updated, because they are considered
// obsolete by the sender.
//
ClemensLinnhoff marked this conversation as resolved.
Show resolved Hide resolved
// \note IDs are globally unique.
// \note Referenced IDs are globally unique.
//
repeated Identifier obsolete_id = 9;
}