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
Expose Telemetry Data #5153
base: dev
Are you sure you want to change the base?
Expose Telemetry Data #5153
Conversation
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.
Reviewed 16 of 16 files at r1, all commit messages.
Reviewable status: complete! all files reviewed, all discussions resolved (waiting on @Axel1092)
Unreal/CarlaUE4/Plugins/Carla/Source/Carla/Vehicle/CarlaWheeledVehicle.cpp, line 787 at r1 (raw file):
Wheels.Add(WheelTelemetryData); }
tab
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.
Reviewable status: complete! all files reviewed, all discussions resolved (waiting on @Axel1092)
Unreal/CarlaUE4/Plugins/Carla/Source/Carla/Vehicle/CarlaWheeledVehicle.cpp, line 787 at r1 (raw file):
Previously, jackbart94 (Jacopo Bartiromo) wrote…
tab
Done
* Exposed omega instead of rpm * Added tire_load, normalized_long_force, normalized_lat_force
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. |
Hi, I was able to implement this code for my project & I'm able to use the command
Inside this, I get a structure like you've mentioned & I can access the VehicleTelemetryData using operator. But when I try to get the WheelTelemetryData using
Can you help me with this issue? |
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. |
Hi, is there a plan to merge this PR to dev? The telemetry info exposed here to the client is pretty useful. Thanks in advance, |
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. |
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.
Reviewed 12 of 16 files at r1, 4 of 4 files at r2.
Reviewable status: 4 of 16 files reviewed, all discussions resolved (waiting on @Axel1092 and @jackbart94)
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.
Reviewed 12 of 12 files at r3, all commit messages.
Reviewable status: complete! all files reviewed, all discussions resolved (waiting on @Axel1092)
Description
This PR exposes the telemetry data to the client:
This will return a
VehicleTelemetryData
structure with the following informationWhere has this been tested?
Possible Drawbacks
This change is