Apply suggestions from code review

Signed-off-by: Georg Seifert <Georg.Seifert@carissma.eu>

Co-authored-by: Kmeid <kmeid.saad@ansys.com>

Author: 2 people

osi_common.proto

@@ -217,8 +217,8 @@ message Identifier
 
 // \brief References to external objects
 //
-// The external reference is used to identify objects defined in non-OSI
-// descriptions. This could be other OpenX-Standards as well as user defined
+// The external reference is an optional recommendation to refer to objects defined outside of OSI.
+// This could be other OpenX standards, 3rd-party standards or user-defined objects.
 // simulation environments.
 //
 message ExternalReference
@@ -233,14 +233,14 @@ message ExternalReference
 
     // The type of the external references
     //
-    // Can be used to describe the original source, e.g. OpenDRIVE 1.6
+    // Must be used to describe the type of the original source.
     //
     optional string type = 2;    
 
     // The external identifier reference value.
     //
     // For a common description of the external identifier, where a wide range
-    // of identification types could be represented, the repeded string is chosen.
+    // of identification types could be represented, the repeated string is chosen.
     //
     // E.g. referencing a unique lane in OpenDRIVE 
     // (RoadId --> String, S-Value of LaneSection --> Double, LaneId --> Int)
@@ -550,4 +550,3 @@ message WavelengthData
     //
     optional double samples_number = 3;
 }
-

osi_environment.proto

@@ -90,14 +90,14 @@ message EnvironmentalConditions
 
     // External Reference to the Environmental Condition Sources
     //
-    // \note For OpenDRIVE and OpenSECNARIO there is no no direct counterpart.
+    // \note For OpenDRIVE and OpenSECNARIO there is no direct counterpart.
     //
     // \note For non-ASAM Standards, it is implementation-specific how
     //       source_reference is resolved.
     //
     // \note The value has to be repeated, as it cannot guarantee, that one
     //       object is derived from only one origin source, like from one 
-    //       scernario file and and from the sensors.
+    //       scernario file and from the sensors.
     //
     repeated ExternalReference source_reference = 9;
 

osi_object.proto

@@ -54,7 +54,7 @@ message StationaryObject
     // * identifier[1] = id of t_road_objects_object or t_road_objects_bridge
     //
     // For OpenSCENARIO 1.0 the entities of the type MiscObject, which describes
-    // partly stationary objecs should be set as following:
+    // partly stationary objects should be set as following:
     // * reference = URI to the OpenSCENARIO File
     // * type = "OpenSCENARIO 1.0"
     // * identifier[0] = Entity-Type ("MiscObject")
@@ -69,7 +69,7 @@ message StationaryObject
     //
     // \note The value has to be repeated, as it cannot guarantee, that one
     //       object is derived from only one origin source, like from one 
-    //       scernario file and and from the sensors.
+    //       scenario file and and from the sensors.
     //
     repeated ExternalReference source_reference = 5;
 

osi_occupant.proto

@@ -25,7 +25,7 @@ message Occupant
 
     // External Reference to the Occupant Source
     //
-    // \note For OpenDRIVE and OpenSECNARIO there is no no direct counterpart.
+    // \note For OpenDRIVE and OpenSCENARIO there is no no direct counterpart.
     //
     // \note For non-ASAM Standards, it is implementation-specific how
     //       source_reference is resolved.