Skip to content

Commit 51b96bf

Browse files
Fix base reference frame definitions
- Remove specific reference frame definitions in base stationary and base moving - Add specific reference frame definitions where base objects are used - Mention all messages where the base objects are used in base definition Signed-off-by: Thomas Sedlmayer <[email protected]>
1 parent 49a0ceb commit 51b96bf

9 files changed

+50
-12
lines changed

osi_common.proto

Lines changed: 8 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -494,14 +494,12 @@ message BoundingBox
494494
// \brief The base attributes of a stationary object or entity.
495495
//
496496
// This includes the \c StationaryObject , \c TrafficSign ,
497-
// \c TrafficLight , \c RoadMarking messages.
497+
// \c TrafficLight , \c RoadMarking messages and the corresponding detected
498+
// object messages ( \c DetectedStationaryObject , \c DetectedTrafficSign ,
499+
// \c DetectedTrafficLight , and \c DetectedRoadMarking ).
498500
//
499-
// \image html OSI_BaseStationary.svg
500-
//
501-
// All coordinates and orientations from ground truth objects are relative to
502-
// the global ground truth frame (see image). (All coordinates and orientations
503-
// from detected objects are relative to the host vehicle frame (see:
504-
// \c Vehicle vehicle reference point).)
501+
// The reference frame of \c BaseStationary depends on the context in which it
502+
// is used and is defined in the message where the \c BaseStationary is used.
505503
//
506504
message BaseStationary
507505
{
@@ -575,18 +573,16 @@ message BaseStationary
575573
//
576574
// \brief The base attributes of an object that is moving.
577575
//
578-
// This includes the \c MovingObject messages.
576+
// This includes the \c MovingObject and \c DetectedMovingObject messages.
579577
//
580578
// \image html OSI_BaseMoving.svg
581579
//
582580
// \image html OSI_BaseMoving_Top.svg
583581
//
584582
// E.g. a vehicle is a base moving object.
585583
//
586-
// All coordinates and orientations from ground truth objects are relative to
587-
// the global ground truth frame. All coordinates and orientations
588-
// from detected objects are relative to the host vehicle frame
589-
// (see: \c MovingObject vehicle reference point).
584+
// The reference frame of \c BaseMoving depends on the context in which it is
585+
// used and is defined in the message where the \c BaseMoving is used.
590586
//
591587
message BaseMoving
592588
{

osi_detectedobject.proto

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -114,6 +114,9 @@ message DetectedStationaryObject
114114

115115
// The base parameters of the stationary object.
116116
//
117+
// The parent frame of \c DetectedStationaryObject::base is the virtual
118+
// sensor coordinate system.
119+
//
117120
optional BaseStationary base = 2;
118121

119122
// The root mean squared error of the base parameters of the detected
@@ -205,6 +208,9 @@ message DetectedMovingObject
205208

206209
// The base parameters of the moving object.
207210
//
211+
// The parent frame of \c DetectedMovingObject::base is the virtual sensor
212+
// coordinate system.
213+
//
208214
// \note The bounding box does NOT include mirrors for vehicles.
209215
// \note The height includes the ground_clearance. It always goes from the
210216
// top to the ground.

osi_detectedroadmarking.proto

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -37,6 +37,9 @@ message DetectedRoadMarking
3737

3838
// The base parameters of the road marking.
3939
//
40+
// The parent frame of \c RoadMarking::base is the virtual sensor coordinate
41+
// system.
42+
//
4043
// The orientation of the bounding box \c #base
4144
// \c BaseStationary::orientation is defined as follows:
4245
// The z-axis of the \c BaseStationary::orientation is the vector from the

osi_detectedtrafficlight.proto

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -34,6 +34,9 @@ message DetectedTrafficLight
3434

3535
// The base parameters of the traffic light.
3636
//
37+
// The parent frame of \c TrafficLight::base is the virtual sensor
38+
// coordinate system.
39+
//
3740
// \c BaseStationary::orientation x-axis is view normal of the traffic
3841
// light's icon.
3942
//

osi_detectedtrafficsign.proto

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -52,6 +52,9 @@ message DetectedTrafficSign
5252

5353
// The base parameters of the traffic sign.
5454
//
55+
// The parent frame of \c DetectedMainSign::base is the virtual sensor
56+
// coordinate system.
57+
//
5558
// The orientation of the bounding box \c #base
5659
// ( \c BaseStationary::orientation ) is defined as follows:
5760
//
@@ -186,6 +189,9 @@ message DetectedTrafficSign
186189

187190
// The base parameters of the supplementary traffic sign.
188191
//
192+
// The parent frame of \c DetectedSupplementarySign::base is the virtual
193+
// sensor coordinate system.
194+
//
189195
// The orientation of the bounding box \c SupplementarySign::base
190196
// ( \c BaseStationary::orientation ) is defined as follows:
191197
// The z-axis of the given \c BaseStationary::orientation is the vector

osi_object.proto

Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -11,6 +11,8 @@ package osi3;
1111
// \c MovingObject e.g. pedestrian, animal, or vehicle) nor a traffic related
1212
// object (\c TrafficLight, \c TrafficSign).
1313
//
14+
// The parent frame of a stationary object is the global coordinate system.
15+
//
1416
// \image html OSI_BaseStationary.svg
1517
//
1618
// \c StationaryObject excludes traffic lights, traffic signs and road marking
@@ -28,6 +30,9 @@ message StationaryObject
2830

2931
// The base parameters of the stationary object.
3032
//
33+
// The parent frame of \c StationaryObject::base is the global coordinate
34+
// system.
35+
//
3136
optional BaseStationary base = 2;
3237

3338
// The classification of the stationary object.
@@ -395,6 +400,8 @@ message StationaryObject
395400
// moving object (animal, pedestrian, etc), but not a stationary
396401
// object (\c TrafficLight, \c TrafficSign, or \c StationaryObject).
397402
//
403+
// The parent frame of a moving object is the global coordinate system.
404+
//
398405
// \image html OSI_MovingObject.svg
399406
//
400407
// \image html OSI_HostVehicle.svg
@@ -416,6 +423,9 @@ message MovingObject
416423

417424
// The base parameters of the vehicle.
418425
//
426+
// The parent frame of \c MovingObject::base is the global coordinate
427+
// system.
428+
//
419429
// \note The bounding box does NOT include side mirrors for vehicles.
420430
// \note The height includes the ground_clearance. It always goes from the
421431
// top to the ground.

osi_roadmarking.proto

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -37,6 +37,8 @@ message RoadMarking
3737

3838
// The base parameters of the road marking.
3939
//
40+
// The parent frame of \c RoadMarking::base is the global coordinate system.
41+
//
4042
// The orientation of the bounding box \c #base
4143
// \c BaseStationary::orientation is defined as follows:
4244
// The z-axis of the \c BaseStationary::orientation is the vector from the

osi_trafficlight.proto

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -9,6 +9,9 @@ package osi3;
99
//
1010
// \brief A traffic light.
1111
//
12+
// All coordinates and orientations are relative to the global ground truth
13+
// coordinate system.
14+
//
1215
// \image html OSI_TrafficLight.svg
1316
//
1417
// One traffic light message defines a single 'bulb' and not a box of several
@@ -27,6 +30,9 @@ message TrafficLight
2730

2831
// The base parameters of the traffic light.
2932
//
33+
// The parent frame of \c TrafficLight::base is the global coordinate
34+
// system.
35+
//
3036
// \c BaseStationary::orientation x-axis is view normal of the traffic
3137
// light's icon.
3238
//

osi_trafficsign.proto

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -222,6 +222,9 @@ message TrafficSign
222222
{
223223
// The base parameters of the traffic sign.
224224
//
225+
// The parent frame of \c MainSign::base is the global coordinate
226+
// system.
227+
//
225228
// The orientation of the bounding box \c #base
226229
// ( \c BaseStationary::orientation ) is defined as follows:
227230
// The z-axis of the given \c BaseStationary::orientation is the vector
@@ -5585,6 +5588,9 @@ message TrafficSign
55855588
{
55865589
// The base parameters of the supplementary traffic sign.
55875590
//
5591+
// The parent frame of \c SupplementarySign::base is the global
5592+
// coordinate system.
5593+
//
55885594
// The orientation of the bounding box \c SupplementarySign::base
55895595
// ( \c BaseStationary::orientation ) is defined as follows:
55905596
// The z-axis of the given \c BaseStationary::orientation is the vector

0 commit comments

Comments
 (0)