Skip to content

adds overloads to aggregator::determineTimestamp - #40

Open
niniemann wants to merge 13 commits into
rock-simulation:masterfrom
niniemann:detection-types-timestamp
Open

adds overloads to aggregator::determineTimestamp#40
niniemann wants to merge 13 commits into
rock-simulation:masterfrom
niniemann:detection-types-timestamp

Conversation

@niniemann

Copy link
Copy Markdown

For the stream aligner and transformer to work, access to a timestamp in
the message is needed. By default a member variable called "time" is
assumed to exist, which is not the case in the objectDetectionMessages.
Therefore, overloads to determineTimestamp are provided which correctly
access the timestamp in the header of the messages.

hwiedPro and others added 13 commits January 28, 2019 09:57
For the stream aligner and transformer to work, access to a timestamp in
the message is needed. By default a member variable called "time" is
assumed to exist, which is not the case in the objectDetectionMessages.
Therefore, overloads to determineTimestamp are provided which correctly
access the timestamp in the header of the messages.
@niniemann
niniemann force-pushed the detection-types-timestamp branch from 9face33 to 878dfe1 Compare September 10, 2019 12:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants