From bc56c0b2f6d12c7de8870a5bb68124d600716a22 Mon Sep 17 00:00:00 2001 From: Habedank Clemens Date: Mon, 14 Jun 2021 13:42:11 +0200 Subject: [PATCH 1/3] Clarified semantics currently supported with TrafficUpdate Signed-off-by: Habedank Clemens --- osi_trafficupdate.proto | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/osi_trafficupdate.proto b/osi_trafficupdate.proto index 5a05344ae..221c29a80 100644 --- a/osi_trafficupdate.proto +++ b/osi_trafficupdate.proto @@ -45,5 +45,9 @@ message TrafficUpdate // or vehicle category. All dynamic fields should be populated where known, // for example, velocity, light states, or future trajectory. // + // \note The field is repeated because it is possible to have a trailer attached to + // a vehicle, see MovingObject::VehicleClassification::has_trailer and + // MovingObject::VehicleClassification::trailer_id. + // repeated MovingObject update = 3; } From a73632d9443f7ea041321a4433a8c104b335f642 Mon Sep 17 00:00:00 2001 From: Habedank Clemens Date: Tue, 15 Jun 2021 10:04:39 +0200 Subject: [PATCH 2/3] Add more explanations to TrafficUpdate semantics Signed-off-by: Habedank Clemens --- osi_trafficupdate.proto | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/osi_trafficupdate.proto b/osi_trafficupdate.proto index 221c29a80..32844c812 100644 --- a/osi_trafficupdate.proto +++ b/osi_trafficupdate.proto @@ -11,7 +11,9 @@ package osi3; // // \brief The traffic update message is provided by traffic // participant models to provide updates to their position, state -// and future trajectory back to the simulation environment. +// and future trajectory back to the simulation environment. The message is designed +// to update data of exactly ONE traffic participant, optionally with an attached +// trailer. // // \note For reasons of convenience and consistency, the // updated information is provided as a MovingObject. Certain fields From 8e79ad7ffa77a53af5d87fdd69bc1164107bf486 Mon Sep 17 00:00:00 2001 From: "Pierre R. Mai" Date: Wed, 4 Aug 2021 09:51:41 +0200 Subject: [PATCH 3/3] Decapitalize one, reflow text for even widths Signed-off-by: Pierre R. Mai --- osi_trafficupdate.proto | 16 ++++++++-------- 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/osi_trafficupdate.proto b/osi_trafficupdate.proto index 32844c812..965937630 100644 --- a/osi_trafficupdate.proto +++ b/osi_trafficupdate.proto @@ -9,15 +9,15 @@ import "osi_object.proto"; package osi3; // -// \brief The traffic update message is provided by traffic -// participant models to provide updates to their position, state -// and future trajectory back to the simulation environment. The message is designed -// to update data of exactly ONE traffic participant, optionally with an attached -// trailer. +// \brief The traffic update message is provided by traffic participant +// models to provide updates to their position, state and future +// trajectory back to the simulation environment. The message is +// designed to update data of exactly one traffic participant, +// optionally with an attached trailer. // -// \note For reasons of convenience and consistency, the -// updated information is provided as a MovingObject. Certain fields -// of this sub-message are not required to be set and will be ignored by the +// \note For reasons of convenience and consistency, the updated +// information is provided as a MovingObject. Certain fields of this +// sub-message are not required to be set and will be ignored by the // simulation environment, because they are static information. // Instead of creating a seperate message type for only the non-static // information, re-use existing message.