Skip to content

Commit d36ce88

Browse files
author
Stefan Cyliax
authored
Merge pull request #576 from OpenSimulationInterface/documentation/native-speaker-review
docs: native speaker review
2 parents f3db096 + 15878d0 commit d36ce88

21 files changed

+84
-83
lines changed

doc/architecture/architecture_overview.adoc

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
= Overview of OSI architecture
22

33
OSI contains an object-based environment description that uses the message format of the https://github.com/protocolbuffers/protobuf/wiki[Protocol Buffer] library.
4-
The Protocol Buffer library was developed and is maintained by Google.
4+
Google developed and maintains the Protocol Buffer library.
55
OSI defines top-level messages that are used to exchange data between separate models.
66
Top-level messages define the `GroundTruth` interface, the `SensorData` interface, and – since OSI version 3.0.0 – the interfaces `SensorView`, `SensorViewConfiguration`, and `FeatureData`.
77

@@ -27,15 +27,15 @@ image::{images_open_simulation_interface}/osi-traffic-participant-advanced.png[1
2727

2828
The `HostVehicleData` interface describes the measured internal states of a traffic participant.
2929
OSI currently provides only limited support for data structures that describe measured internal states of traffic participants.
30-
Actuator intentions are currently not covered by OSI and must be handled with a different data description format.
30+
Actuator intentions are currently not covered by OSI and must be handled using a different data description format.
3131

3232
All fields in an interface are set to `optional`.
3333
`required` is not used.
3434
This has been done to allow backward-compatible changes in the field.
35-
Additionally, this is the default behavior in Protocol Buffer version 3 that does no longer have the `required` type.
35+
Additionally, this is the default behavior in Protocol Buffer version 3 that no longer has the `required` type.
3636
Setting all fields to `optional` thus ensures update compatibility.
37-
However, this does not mean that filling the field is optional.
38-
For the purpose of providing a complete interface, all existing fields should be set, unless not setting a field carries a specific meaning as indicated in the accompanying comment.
37+
However, this does not mean that it is optional to fill the field.
38+
For the purpose of providing a complete interface, all existing fields should be set, unless not setting a field carries a specific meaning, as indicated in the accompanying comment.
3939

4040
All field numbers equal to or greater than 10000 are available for user-specific extensions via custom fields.
41-
Therefore, no future evolution of OSI will use field numbers equal to or greater than 10000.
41+
No future evolution of OSI will therefore use field numbers equal to or greater than 10000.

doc/architecture/data_layer.adoc

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,14 +1,14 @@
11
= Data layer
22

3-
The data layer of OSI is defined in the message specifications using the ProtoBuf IDL.
4-
It defines the data that can be transmitted using OSI, including the structure and the semantics of the data.
3+
The OSI data layer is defined in the message specifications using the ProtoBuf IDL.
4+
This defines the data that can be transmitted using OSI, including the structure and the semantics of the data.
55

66
Additionally, it specifies the encoding to be used when OSI data is transmitted.
77
Currently, ProtoBuf encoding is used, but other encodings are possible with the ProtoBuf IDL.
88
FlatBuffer encoding has been implemented as an experimental feature.
99

1010
The data layer does not directly define components and transmission routes.
11-
These are defined in the packaging layer of OSI.
11+
These are defined in the OSI packaging layer.
1212
There may be different packaging layer implementations using the shared data layer definitions.
1313
The data that is exchanged remains compatible regardless of the packaging layer implementation.
1414
The use of a shared data layer ensures easy bridging between different packaging layer implementations.

doc/architecture/feature_data.adoc

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,4 +2,4 @@
22

33
Feature-data messages contain detected features in the reference frame of a sensor.
44
Feature-data messages are generated from ground-truth messages.
5-
They serve, for example, as input to sensor models simulating object detection or feature fusion models.
5+
They serve, for example, as an input to sensor models simulating object detection or feature fusion models.

doc/architecture/formatting_scripts.adoc

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
= Trace-file formatting scripts
22

3-
The OSI repository contains Python scripts for converting trace files from one format to the other.
3+
The OSI repository contains Python scripts for converting trace files from one format to another.
44
The formatting scripts are stored in `open-simulation-interface/format/`
55

66
**txt2osi.py**
@@ -13,16 +13,16 @@ String containing the path to the file with serialized data.
1313

1414
`--type`, `-t`::
1515
Optional string describing the message type used to serialize data.
16-
Allowed values are `'SensorView'`, `'GroundTruth'`, or `'SensorData'`.
16+
`'SensorView'`, `'GroundTruth'`, or `'SensorData'` are permitted values.
1717
The default value is `'SensorView'`.
1818

1919
`--output`, `-o`::
2020
Optional string containing the name of the output file.
2121
The default value is `'converted.osi'`.
2222

2323
`--compress`, `-c`::
24-
Optional boolean controlling whether to compress the output to a lzma file.
25-
Allowed values are `True`, or `False`.
24+
Optional Boolean controlling whether to compress the output to an lzma file.
25+
`True`, or `False` are permitted values.
2626
The default value is `False`.
2727

2828
**osi2read.py**
@@ -35,7 +35,7 @@ String containing the path to the file with serialized data.
3535

3636
`--type`, `-t`::
3737
Optional string describing the message type used to serialize data.
38-
Allowed values are `'SensorView'`, `'GroundTruth'`, or `'SensorData'`.
38+
`'SensorView'`, `'GroundTruth'`, or `'SensorData'` are permitted values.
3939
The default value is `'SensorView'`.
4040

4141
`--output`, `-o`::
@@ -44,7 +44,7 @@ The default value is `'converted.txth'`.
4444

4545
`--format`, `-f`::
4646
Optional string containing the format type of the trace file.
47-
Allowed values are `'separated'`, or `None`.
47+
`'separated'`, or `None` are permitted values.
4848
The default value is `None`.
4949

5050
**Related topics**

doc/architecture/ground_truth.adoc

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
= Ground truth
22

33
Ground-truth messages describe the simulated environment containing all simulated objects in the global coordinate system at consecutive time instances.
4-
It is based on data available to the simulation environment.
5-
Typically, ground-truth messages are contained in sensor view messages.
4+
They are based on data available to the simulation environment.
5+
Ground-truth messages are typically contained in sensor view messages.

doc/architecture/packaging_layer.adoc

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,8 @@
11
= Packaging layer
22

3-
The packaging layer of OSI specifies how components that use the OSI data layer, for example, sensor models, are packaged for exchange.
3+
The OSI packaging layer specifies how components that use the OSI data layer, for example, sensor models, are packaged for exchange.
44

5-
It specifies model types and their mandatory and optional OSI inputs, OSI outputs, and parameter interfaces.
5+
This specifies model types and their mandatory and optional OSI inputs, OSI outputs, and parameter interfaces.
66
A model type may be, for example, a sensor model or a traffic participant model.
77
The packaging layer also specifies component technology standards.
88
This makes it possible to encapsulate model types in easily exchangeable component packages that can be used across platforms and implementations.

doc/architecture/sensor_data.adoc

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
= Sensor data
22

33
Sensor-data messages imitate the output of real sensors.
4-
It can be generated from ground-truth messages, sensor-view messages, feature-data messages or from sensor-data messages.
5-
Except feature data, all information regarding the environment is given with respect to the virtual sensor coordinate system.
4+
They can be generated from ground-truth messages, sensor-view messages, feature-data messages, or sensor-data messages.
5+
With the exception of feature data, all information regarding the environment is given with respect to the virtual sensor coordinate system.
66
Feature data is given with respect to the physical sensor coordinate system.
77
Sensor data can be used as input for an automated driving function, a sensor model simulating limited perception, or a sensor fusion model.

doc/architecture/sensor_view.adoc

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -5,5 +5,5 @@ Sensor-view messages are derived from ground-truth messages.
55
All information regarding the environment is given with respect to the virtual sensor coordinate system, with two exceptions:
66

77
* Physical technology-specific data, given with respect to the physical sensor coordinate system specified in the corresponding physical sensor's mounting position.
8-
Example of technology-specific data: https://opensimulationinterface.github.io/open-simulation-interface/structosi3_1_1CameraSensorView.html#ac58456a34babf78792ea2608eb963f36[`image_data` of `osi3::CameraSensorView`]
9-
* Ground truth, given in the global coordinate system.
8+
One example of technology-specific data is: https://opensimulationinterface.github.io/open-simulation-interface/structosi3_1_1CameraSensorView.html#ac58456a34babf78792ea2608eb963f36[`image_data` of `osi3::CameraSensorView`]
9+
* Ground truth given in the global coordinate system.

doc/architecture/sensor_view_configuration.adoc

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -1,27 +1,27 @@
11
= Sensor view configuration
22

3-
The sensor view is flexibly defined to provide different kinds of sensor models with appropriate input.
3+
The sensor view is flexibly defined to provide different kinds of sensor models with an appropriate input.
44
The sensor view configuration defines the configuration of a particular sensor view.
55

66
The sensor-view-configuration message is used in the initialization phase of a simulation to negotiate the sensor view configuration for a particular sensor view input.
77
It is also included as a sub-message in sensor view messages to indicate that the sensor view configuration is valid for a particular sensor view message.
88

99
Sensor-view-configuration data has two main applications:
1010

11-
- It enables the environment simulation to provide the necessary input to a sensor model.
12-
- It enables a sensor model to check whether the input matches its requirements.
11+
- Enable the environment simulation to provide the necessary input to a sensor model.
12+
- Enable a sensor model to check whether the input matches its requirements.
1313
If the input does not match the requirements, the sensor model may terminate the simulation.
1414
15-
NOTE: Sensor-view-configuration data is intended for the automatic configuration of the sensor view interface between environment simulation and sensor model.
16-
The data is not intended as a mechanism to parametrize a generic sensor model.
15+
NOTE: Sensor-view-configuration data is intended for the automatic configuration of the sensor view interface between an environment simulation and sensor model.
16+
The data is not intended to be a mechanism for parametrizing a generic sensor model.
1717

1818
During the initialization phase, there are two sources for sensor-view-configuration data:
1919

20-
1. Sensor-view-configuration data may be provided by the sensor model to the environment simulation.
20+
1. Sensor-view configuration data may be provided by the sensor model to the environment simulation.
2121
In this case, the data describes the input configuration that is requested by the sensor model.
22-
If no such data is provided by the sensor model, then the environment simulation will fall back to manual configuration of the sensor view.
22+
If the sensor model does not provide such data, then the environment simulation will fall back to manual configuration of the sensor view.
2323
24-
2. Sensor-view-configuration data may be provided by the environment simulation.
24+
2. Sensor-view configuration data may be provided by the environment simulation.
2525
In response to the request by the sensor model, or based on manual configuration, the environment simulation configures the input and provides a new message that describes the actual configuration.
2626
2727
The configuration requested by the sensor model may differ from the configuration provided by the environment simulation.

doc/architecture/trace_file_formats.adoc

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
[#top-1a2f4b0c-195c-4f18-89ad-d48a123bd8c1]
22
= OSI trace file formats
33

4-
There are multiple formats for storing multiple serialized OSI messages into one trace file.
4+
There are multiple formats for storing multiple serialized OSI messages in one trace file.
55

66
*.osi::
77
Binary trace file.
@@ -16,4 +16,4 @@ Messages are separated by `$$__$$`.
1616
*.txth::
1717
Human-readable plain-text trace file.
1818
Messages are separated by newlines.
19-
Such a file may be used for manual checks.
19+
These files may be used for manual checks.

0 commit comments

Comments
 (0)