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

Added hvd as a trace file type for HostVehicleData #810

Merged
merged 1 commit into from
May 13, 2024

Conversation

thempen
Copy link
Contributor

@thempen thempen commented May 8, 2024

Added hvd as a supported trace file type for HostVehicleData

Add a description

We are using OSI to send data within our test vehicle. Within the vehicle, we are just sending HostVehicleData information, because other data is not required. To have data for testing and evaluation, tracing that HostVehicleData should be available in a general fashion.

This change is just an extension in the naming conventions, which will allow saving HostVehicleData as hvd files.

Take this checklist as orientation for yourself, if this PR is ready for the Change Control Board:

@thempen thempen requested a review from pmai May 8, 2024 13:56
Copy link
Contributor

@pmai pmai left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

CCB 2024-05-13: Merge as-is.

@pmai pmai added the ReadyToMerge This PR has been approved to merge and will be merged by a member of the CCB. label May 13, 2024
@pmai pmai added this to the V3.7.0 milestone May 13, 2024
Added hvd as a supported trace file type for HostVehicleData

Signed-off-by: Thomas Hempen <Thomas.Hempen@carissma.eu>
@pmai pmai force-pushed the extending-trace-file-definition-hvd branch from 9da54ea to 5163863 Compare May 13, 2024 09:12
@pmai pmai merged commit c49b3da into master May 13, 2024
10 checks passed
@pmai pmai deleted the extending-trace-file-definition-hvd branch May 13, 2024 09:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ReadyToMerge This PR has been approved to merge and will be merged by a member of the CCB.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants