-
Notifications
You must be signed in to change notification settings - Fork 129
Streaming Interface #718
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
Merged
Merged
Streaming Interface #718
Changes from all commits
Commits
Show all changes
9 commits
Select commit
Hold shift + click to select a range
bfe355e
Add initial consolidated osi_streaming.proto
ThomasNaderBMW 2f4d22d
Add architecture png for osi-streaming
ThomasNaderBMW 5324e14
Update architecture_overview.adoc by osi-streaming
ThomasNaderBMW f2c837f
Add StreamingUpdate description
thomassedlmayer 103b873
Add missing imports and order field numbers
thomassedlmayer 31272a2
Clarify host vehicle data ID handling
ThomasNaderBMW 05b7531
Add documentation for StreamingUpdate
thomassedlmayer b6a8c4a
Add example to the note of StreamingUpdate
thomassedlmayer 9f9536f
Add to build infrastructure
pmai File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,10 @@ | ||
ifndef::include-only-once[] | ||
:root-path: ../ | ||
include::{root-path}_config.adoc[] | ||
endif::[] | ||
= Streaming update | ||
|
||
The `StreamingUpdate` message provides an interface to transmit a subset of ground truth and/or vehicle internal data. | ||
This interface mainly addresses applications with low latency requirements and no need for highly consistent and complete data, e.g. visualization applications. | ||
Static and/or non-relevant objects can be omitted as required for the specific use case. | ||
Note that the receiver of partial updates can only rely on the most up-to-date information at the corresponding timestamp. E.g. omitting objects does not indicate static behaviour but it may be sufficient for the use case to update certain objects at a later point in time. |
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,80 @@ | ||
syntax = "proto2"; | ||
|
||
option optimize_for = SPEED; | ||
|
||
import "osi_version.proto"; | ||
import "osi_common.proto"; | ||
import "osi_environment.proto"; | ||
import "osi_object.proto"; | ||
import "osi_trafficsign.proto"; | ||
import "osi_trafficlight.proto"; | ||
import "osi_hostvehicledata.proto"; | ||
|
||
package osi3; | ||
|
||
// | ||
// \brief The streaming update interface enables simulation entities to send | ||
// partial updates to other modules that favor performance (especially latency) | ||
// over data completeness/consistency (e.g. visualization applications). | ||
// | ||
// Static and/or non-relevant objects can be omitted as required for the | ||
// specific use case. Adding an object's unique id to the repeated field \c | ||
// obsolete_id indicates that it will no longer be updated from then on. | ||
// | ||
// \note The receiver of partial streaming update messages can only rely on the | ||
// most up-to-date information at the corresponding timestamp. E.g. omitting | ||
// objects does not indicate static behaviour but it may be sufficient for the | ||
// use case to update certain objects at a later point in time. | ||
// | ||
message StreamingUpdate | ||
{ | ||
// The interface version used by the sender. | ||
// | ||
optional InterfaceVersion version = 1; | ||
|
||
// The data timestamp where the information of contained objects is calculated. | ||
// | ||
// Zero time is arbitrary but must be identical for all messages. | ||
// Zero time does not need to coincide with the UNIX epoch. | ||
// Recommended is the starting time point of the simulation. | ||
// | ||
optional Timestamp timestamp = 2; | ||
|
||
// The list of stationary objects (excluding traffic signs and traffic | ||
// lights). | ||
// | ||
repeated StationaryObject stationary_object_update = 3; | ||
|
||
// The list of moving objects. | ||
// | ||
repeated MovingObject moving_object_update = 4; | ||
|
||
// The list of traffic signs. | ||
// | ||
repeated TrafficSign traffic_sign_update = 5; | ||
|
||
// The list of traffic lights. | ||
// | ||
repeated TrafficLight traffic_light_update = 6; | ||
|
||
// Conditions of the environment. | ||
// | ||
optional EnvironmentalConditions environmental_conditions_update = 7; | ||
|
||
// Host vehicle data. | ||
// | ||
// Host vehicle data is data that the host vehicle knows about itself, | ||
// e.g. from location sensors, internal sensors and ECU bus data, etc., | ||
// that is made available to sensors as input. | ||
// | ||
// The ID inside this message allows an association to moving object data. | ||
// | ||
repeated HostVehicleData host_vehicle_data_update = 8; | ||
|
||
// Entities that will no longer be updated, because they are considered | ||
// obsolete by the sender. | ||
// | ||
// \note IDs are globally unique. | ||
// | ||
repeated Identifier obsolete_id = 9; | ||
} |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What has changed here? Or is this highlighting some bug in the diff-function from GitHub?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Probably some CRLF/LF stuff? I'm working on some documentation additions on the StreamingUpdate message right now anyway. I'll try deleting and adding the last line. Maybe that does the trick.