OpenSimulationInterface / open-simulation-interface

A generic interface for the environmental perception of automated driving functions in virtual scenarios.
Other
267 stars 125 forks source link

Update host vehicle data with kinematics data #679

Closed MaikolDrechsler closed 1 year ago

MaikolDrechsler commented 1 year ago

Include velocity, acceleration and orientation rate data related to the vehicle coordinate system in the host_vehicle_data message

Signed-off-by: Maikol Drechsler maikol.drechsler@carissma.eu

#### Reference to a related issue in the repository Due to the deprecation of localization and localization_rmse attributes of host_vehicle_data the velocity, acceleration and yaw_rate of the vehicle cannot be described. The issue had been discussed per e-mail with @ThomasNaderBMW

Add a description

Include a new attribute vehicle_kinametics which include the ego velocity, orientation rate and acceleration in the host_vehicle_data.

Some questions to ask: What is this change? Include a new attribute, that enables the exchange of vehicle kinematics data.
What does it fix? The data lost by the deprecation of the attributes localization and localization_rmse. Is this a bug fix or a feature? Does it break any existing functionality or force me to update to a new version? It is a new optional attribute, no update is forced. How has it been tested? It has been tested locally with a C++ implementation.

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

If you can’t check all of them, please explain why. If all boxes are checked or commented and you have achieved at least one positive review, you can assign the label ReadyForCCBReview!

pmai commented 1 year ago

CCB 2023-01-16: To be changed as discussed (include references to bbcenter_to_rear in description of reference point for VehicleMotion, reorder fields to match derivative structure, and add comment to VehicleLocalization to indicate middle of bounding box reference point), then rediscussed for merge in next CCB.