Skip to content
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

Feature/groundtruth configuration message #581

Draft
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

pmai
Copy link
Contributor

@pmai pmai commented Sep 20, 2021

This PR contains the unrelated changes to introduce a GroundTruthInitConfiguration message from PR #407, which is work in progress.

@pmai pmai added this to the V4.0.0 milestone Sep 20, 2021
@pmai pmai self-assigned this Sep 20, 2021
@stefancyliax stefancyliax modified the milestones: V4.0.0, V3.5.0 Nov 10, 2021
@kmeids
Copy link

kmeids commented Mar 16, 2022

@pmai,
I recall that this came out of an initial bigger pull request during V3.4.
Currently this is still on Draft, do we have any update on this topic?

@ThomasNaderBMW
Copy link
Contributor

@tbleher : Is this change here interesting for you and needed?
The omit_static_information part of sensorViewConfiguration is already merged.

@tbleher
Copy link
Contributor

tbleher commented Apr 26, 2022

@ThomasNaderBMW Thanks for highlighting this commit :)
For our pedestrian model, current OSMP plus the already merged omit_static_information flag provides all we need for now :)

I'm not yet clear how the the new message would fit into OSMP. The new GroundTruthInit message is sent very early. If I understood correctly, the the proposed configuration message would influence the GroundTruthInit message, and thus would have to be exchanged before the GroundTruthInit message. How would this work in OSMP and FMI?

@stefancyliax stefancyliax modified the milestones: V3.5.0, V4.0.0 May 9, 2022
@thomassedlmayer
Copy link
Contributor

@thempen I just stumbled upon this older draft by Pierre. This might be interesting for upcoming discussions on the static/dynamic topic.

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.

6 participants