feat: add steering line to telemetry chart. Resolves #3

Author: sapessi

src/live/telemetry_view.rs

@@ -95,13 +95,13 @@ impl LiveTelemetryApp {
                 self.telemetry_points.iter().enumerate().all(|p| {
                     throttle_vec.push([p.0 as f64, p.1.throttle as f64 * 100.]);
                     brake_vec.push([p.0 as f64, p.1.brake as f64 * 100.]);
-                    steering_vec.push([p.0 as f64, p.1.steering as f64]);
+                    steering_vec.push([p.0 as f64, 50. + 50. * p.1.steering_pct as f64]);
                     true
                 });
 
                 let throttle_points = PlotPoints::new(throttle_vec);
                 let brake_points = PlotPoints::new(brake_vec);
-                //let steering_points = PlotPoints::new(steering_vec);
+                let steering_points = PlotPoints::new(steering_vec);
 
                 plot.show_background(false).show(ui, |plot_ui| {
                     plot_ui.line(
@@ -126,6 +126,11 @@ impl LiveTelemetryApp {
                             .fill(0.)
                             .name("Brake"),
                     );
+                    plot_ui.line(
+                        Line::new(steering_points)
+                            .color(Color32::LIGHT_GRAY)
+                            .name("Steering"),
+                    );
                 });
             });
         // make it always repaint. TODO: can we slow down here?

src/telemetry/mod.rs

@@ -55,6 +55,8 @@ pub struct TelemetryPoint {
     pub brake: f32,
     /// Steering wheel angle
     pub steering: f32,
+    // steering angle as a % of max steering
+    pub steering_pct: f32,
     /// Whether ABS is currently active
     pub abs_active: bool,
     /// Latitude in decimal degrees
@@ -102,6 +104,7 @@ impl Default for TelemetryPoint {
             throttle: 0.,
             brake: 0.,
             steering: 0.,
+            steering_pct: 0.,
             abs_active: false,
             lat: 0.,
             lon: 0.,

src/telemetry/producer.rs

@@ -11,6 +11,7 @@ use crate::OcypodeError;
 use super::{SessionInfo, TelemetryPoint, TireInfo};
 
 const CONN_RETRY_WAIT_MS: u64 = 200;
+const MAX_STEERING_ANGLE_DEFAULT: f32 = std::f32::consts::PI;
 pub(crate) const CONN_RETRY_MAX_WAIT_S: u64 = 600;
 
 pub trait SimplifiedTelemetryAccess {
@@ -187,7 +188,6 @@ impl TelemetryProducer for IRacingTelemetryProducer {
                     description: "Could not retrieve telemetry",
                 }
             })?;
-
         Ok(SessionInfo {
             track_name: ir_session_info.weekend.track_name,
             max_steering_angle: telemetry.get_float("SteeringWheelAngleMax").unwrap_or(0.),
@@ -212,6 +212,15 @@ impl TelemetryProducer for IRacingTelemetryProducer {
                 }
             })?;
 
+        let steering_pct = if telemetry.get_float("SteeringWheelAngle").unwrap_or(0.)
+            > MAX_STEERING_ANGLE_DEFAULT
+        {
+            telemetry.get_float("SteeringWheelAngle").unwrap_or(0.)
+                / telemetry.get_float("SteeringWheelAngleMax").unwrap_or(0.)
+        } else {
+            telemetry.get_float("SteeringWheelAngle").unwrap_or(0.) / MAX_STEERING_ANGLE_DEFAULT
+        };
+
         let measurement = TelemetryPoint {
             point_no: self.point_no,
             lap_dist: telemetry.get_float("LapDist").unwrap_or(0.),
@@ -226,6 +235,8 @@ impl TelemetryProducer for IRacingTelemetryProducer {
             throttle: telemetry.get_float("Throttle").unwrap_or(0.),
             brake: telemetry.get_float("BrakeRaw").unwrap_or(0.),
             steering: telemetry.get_float("SteeringWheelAngle").unwrap_or(0.),
+            // this might result in an unhappy division by 0. Do we want to panic in this case because it's unexpected?
+            steering_pct,
             abs_active: telemetry.get_bool("BrakeABSactive").unwrap_or(false),
             lat: telemetry.get_float("Lat").unwrap_or(0.),
             lon: telemetry.get_float("Lon").unwrap_or(0.),