From a10698962eb4c94406d9efb9b71fc09227a77afa Mon Sep 17 00:00:00 2001
From: hodasemi <michaelh.95@t-online.de>
Date: Wed, 18 Jan 2023 17:02:20 +0100
Subject: [PATCH 1/6] Add GamePhase logic

---
 .../leaderboard/leaderboard_entry.xml         |   6 +-
 .../elements/leaderboard/leaderboard_grid.xml |   2 +-
 src/overlay/elements/leaderboard/mod.rs       | 113 +++++++++++++-----
 src/overlay/elements/pedals/mod.rs            |  13 ++
 src/overlay/elements/radar/mod.rs             |  13 ++
 src/overlay/elements/watermark/mod.rs         |  20 +++-
 src/overlay/rfactor_data.rs                   |  41 +++++--
 7 files changed, 163 insertions(+), 45 deletions(-)

diff --git a/src/overlay/elements/leaderboard/leaderboard_entry.xml b/src/overlay/elements/leaderboard/leaderboard_entry.xml
index a61afb4..b3c182e 100644
--- a/src/overlay/elements/leaderboard/leaderboard_entry.xml
+++ b/src/overlay/elements/leaderboard/leaderboard_entry.xml
@@ -1,12 +1,12 @@
 <?xml-model href="../gui.xsd" type="application/xml" schematypens="http://www.w3.org/2001/XMLSchema"?>
 <root>
-    <grid id="grid" x_dim="9" y_dim="1">
+    <grid id="grid" x_dim="11" y_dim="1">
         <label id="place" x_slot="0" y_slot="0" text_color="black" text_alignment="right"></label>
         <label
             id="name"
             x_slot="1" y_slot="0" x_size="6" text_color="black" text_alignment="left"></label>
         <label
-            id="time_behind"
-            x_slot="7" y_slot="0" x_size="2" text_color="black" text_alignment="right"></label>
+            id="time"
+            x_slot="7" y_slot="0" x_size="4" text_color="black" text_alignment="right"></label>
     </grid>
 </root>
\ No newline at end of file
diff --git a/src/overlay/elements/leaderboard/leaderboard_grid.xml b/src/overlay/elements/leaderboard/leaderboard_grid.xml
index 0ad24dc..cab3c03 100644
--- a/src/overlay/elements/leaderboard/leaderboard_grid.xml
+++ b/src/overlay/elements/leaderboard/leaderboard_grid.xml
@@ -1,5 +1,5 @@
 <?xml-model href="../gui.xsd" type="application/xml" schematypens="http://www.w3.org/2001/XMLSchema"?>
 <root reference_width="2560" reference_height="1440">
-    <grid id="main_grid" x_dim="1" y_dim="25" x_offset="10" y_offset="10" width="350" height="875"
+    <grid id="main_grid" x_dim="1" y_dim="25" x_offset="10" y_offset="10" width="400" height="875"
         vert_align="top" hori_align="left" margin="0" padding="0"> </grid>
 </root>
\ No newline at end of file
diff --git a/src/overlay/elements/leaderboard/mod.rs b/src/overlay/elements/leaderboard/mod.rs
index f189029..e453eee 100644
--- a/src/overlay/elements/leaderboard/mod.rs
+++ b/src/overlay/elements/leaderboard/mod.rs
@@ -53,7 +53,14 @@ impl LeaderBoard {
             .to_string()
     }
 
-    fn race_leaderboard(&mut self, vehicle_scorings: &[VehicleScoringInfoV01]) -> Result<()> {
+    fn update_leaderboard<F>(
+        &mut self,
+        vehicle_scorings: &[VehicleScoringInfoV01],
+        f: F,
+    ) -> Result<()>
+    where
+        F: Fn(&mut LeaderBoardEntry, &VehicleScoringInfoV01) -> Result<()>,
+    {
         for vehicle_scoring in vehicle_scorings {
             let driver_name = Self::c_char_to_string(vehicle_scoring.mDriverName);
 
@@ -69,8 +76,8 @@ impl LeaderBoard {
                     }
 
                     entry.update_place(vehicle_scoring.mPlace)?;
-                    entry.update_time_behind_leader(vehicle_scoring.mTimeBehindLeader)?;
-                    entry.update_time_behind_next(vehicle_scoring.mTimeBehindNext)?;
+
+                    f(entry, vehicle_scoring)?;
                 }
                 None => {
                     let entry = LeaderBoardEntry::new(
@@ -80,6 +87,7 @@ impl LeaderBoard {
                         vehicle_scoring.mPlace,
                         vehicle_scoring.mTimeBehindLeader,
                         vehicle_scoring.mTimeBehindNext,
+                        vehicle_scoring.mBestLapTime,
                     )?;
 
                     self.entries.push(entry);
@@ -120,19 +128,39 @@ impl LeaderBoard {
             }
         }
 
-        if self.entries.is_empty() {
-            self.gui.disable()?;
-        } else {
-            self.gui.enable()?;
-        }
-
         Ok(())
     }
+
+    fn race_leaderboard(&mut self, vehicle_scorings: &[VehicleScoringInfoV01]) -> Result<()> {
+        self.update_leaderboard(vehicle_scorings, |entry, scoring| {
+            entry.update_time_behind_leader(scoring.mTimeBehindLeader)
+        })
+    }
+
+    fn quali_leaderboard(&mut self, vehicle_scorings: &[VehicleScoringInfoV01]) -> Result<()> {
+        self.update_leaderboard(vehicle_scorings, |entry, scoring| {
+            entry.update_best_lap(scoring.mBestLapTime)
+        })
+    }
 }
 
 impl UiOverlay for LeaderBoard {}
 
 impl DataReceiver for LeaderBoard {
+    fn game_phase_change(&mut self, phase: GamePhase) -> Result<()> {
+        match phase {
+            GamePhase::Practice | GamePhase::Qualifying | GamePhase::Race => self.gui.enable(),
+            _ => self.gui.disable(),
+        }
+    }
+
+    fn update_for_phase(&self, phase: GamePhase) -> bool {
+        match phase {
+            GamePhase::Practice | GamePhase::Qualifying | GamePhase::Race => true,
+            _ => false,
+        }
+    }
+
     fn scoring_update(
         &mut self,
         phase: GamePhase,
@@ -141,13 +169,13 @@ impl DataReceiver for LeaderBoard {
         write_log!("=================== leader board: scoring update ===================");
 
         match phase {
-            GamePhase::TestDay => self.gui.disable()?,
-            GamePhase::Practice => self.gui.disable()?,
-            GamePhase::Qualifying => self.gui.disable()?,
-            GamePhase::Warmup => self.gui.disable()?,
-            GamePhase::Race => {
-                self.race_leaderboard(vehicle_scorings)?;
+            GamePhase::Practice | GamePhase::Qualifying => {
+                self.quali_leaderboard(vehicle_scorings)?
             }
+
+            GamePhase::Race => self.race_leaderboard(vehicle_scorings)?,
+
+            _ => (),
         }
 
         write_log!("leader board update finished");
@@ -177,13 +205,14 @@ struct LeaderBoardEntry {
     place: u8,
     time_behind_leader: f64,
     time_behind_next: f64,
+    best_lap: f64,
 
     snippet: Arc<GuiSnippet>,
 
     grid: Arc<Grid>,
     name_label: Arc<Label>,
     place_label: Arc<Label>,
-    time_behind_label: Arc<Label>,
+    time_label: Arc<Label>,
 
     place_updated: bool,
 }
@@ -196,17 +225,18 @@ impl LeaderBoardEntry {
         place: u8,
         time_behind_leader: f64,
         time_behind_next: f64,
+        best_lap: f64,
     ) -> Result<Self> {
         let snippet = GuiSnippet::from_str(gui_handler, LeaderBoard::ENTRY)?;
 
         let background = snippet.element("grid")?;
         let name_label: Arc<Label> = snippet.element("name")?;
         let place_label: Arc<Label> = snippet.element("place")?;
-        let time_behind_label: Arc<Label> = snippet.element("time_behind")?;
+        let time_label: Arc<Label> = snippet.element("time")?;
 
         name_label.set_text(&name)?;
         place_label.set_text(place)?;
-        time_behind_label.set_text(format!("{:.3}", time_behind_leader))?;
+        time_label.set_text("---")?;
 
         Ok(Self {
             id,
@@ -215,13 +245,14 @@ impl LeaderBoardEntry {
             place,
             time_behind_leader,
             time_behind_next,
+            best_lap,
 
             snippet,
 
             grid: background,
             name_label,
             place_label,
-            time_behind_label,
+            time_label,
 
             place_updated: true,
         })
@@ -264,21 +295,49 @@ impl LeaderBoardEntry {
     }
 
     pub fn update_time_behind_leader(&mut self, time: f64) -> Result<()> {
-        self.time_behind_leader = time;
+        if self.time_behind_leader != time {
+            self.time_behind_leader = time;
 
-        // check if we are leader
-        if self.time_behind_leader == 0.0 {
-            self.time_behind_label.set_text("---")
-        } else {
-            self.time_behind_label
-                .set_text(format!("+{:.3}", self.time_behind_leader))
+            // check if we are leader
+            if self.time_behind_leader == 0.0 {
+                self.time_label.set_text("---")?;
+            } else {
+                self.time_label
+                    .set_text(format!("+{:.3}", self.time_behind_leader))?;
+            }
         }
+
+        Ok(())
+    }
+
+    pub fn update_best_lap(&mut self, time: f64) -> Result<()> {
+        if self.best_lap != time {
+            self.best_lap = time;
+
+            if self.best_lap < 0.0 {
+                self.time_label.set_text("---")?;
+            } else {
+                let text = if self.best_lap > 60.0 {
+                    let full_minutes = (self.best_lap / 60.0).floor();
+                    let remainder = self.best_lap - (full_minutes * 60.0);
+
+                    format!("{:.0}:{:.3}", full_minutes, remainder)
+                } else {
+                    format!("{:.3}", self.best_lap)
+                };
+
+                self.time_label.set_text(text)?;
+            }
+        }
+
+        Ok(())
     }
 
     pub fn update_time_behind_next(&mut self, time: f64) -> Result<()> {
         self.time_behind_next = time;
 
-        Ok(())
+        self.time_label
+            .set_text(format!("+{:.3}", self.time_behind_next))
     }
 
     pub fn needs_resorting(&self) -> bool {
diff --git a/src/overlay/elements/pedals/mod.rs b/src/overlay/elements/pedals/mod.rs
index 6975ec9..59ac61e 100644
--- a/src/overlay/elements/pedals/mod.rs
+++ b/src/overlay/elements/pedals/mod.rs
@@ -231,6 +231,19 @@ impl Pedals {
 impl UiOverlay for Pedals {}
 
 impl DataReceiver for Pedals {
+    fn game_phase_change(&mut self, _phase: GamePhase) -> Result<()> {
+        Ok(())
+    }
+
+    fn update_for_phase(&self, phase: GamePhase) -> bool {
+        match phase {
+            GamePhase::Practice | GamePhase::Qualifying | GamePhase::Race | GamePhase::Warmup => {
+                true
+            }
+            _ => false,
+        }
+    }
+
     fn scoring_update(
         &mut self,
         _phase: GamePhase,
diff --git a/src/overlay/elements/radar/mod.rs b/src/overlay/elements/radar/mod.rs
index fb94c36..e4ddb81 100644
--- a/src/overlay/elements/radar/mod.rs
+++ b/src/overlay/elements/radar/mod.rs
@@ -268,6 +268,19 @@ impl Radar {
 impl UiOverlay for Radar {}
 
 impl DataReceiver for Radar {
+    fn game_phase_change(&mut self, _phase: GamePhase) -> Result<()> {
+        Ok(())
+    }
+
+    fn update_for_phase(&self, phase: GamePhase) -> bool {
+        match phase {
+            GamePhase::Practice | GamePhase::Qualifying | GamePhase::Race | GamePhase::Warmup => {
+                true
+            }
+            _ => false,
+        }
+    }
+
     fn scoring_update(
         &mut self,
         _phase: GamePhase,
diff --git a/src/overlay/elements/watermark/mod.rs b/src/overlay/elements/watermark/mod.rs
index 093b89f..1cd0aa5 100644
--- a/src/overlay/elements/watermark/mod.rs
+++ b/src/overlay/elements/watermark/mod.rs
@@ -26,11 +26,7 @@ impl Watermark {
 impl UiOverlay for Watermark {}
 
 impl DataReceiver for Watermark {
-    fn scoring_update(
-        &mut self,
-        phase: GamePhase,
-        _vehicle_scoring: &[VehicleScoringInfoV01],
-    ) -> Result<()> {
+    fn game_phase_change(&mut self, phase: GamePhase) -> Result<()> {
         match phase {
             GamePhase::TestDay => self.gui.enable()?,
             _ => self.gui.disable()?,
@@ -39,6 +35,20 @@ impl DataReceiver for Watermark {
         Ok(())
     }
 
+    fn update_for_phase(&self, phase: GamePhase) -> bool {
+        match phase {
+            _ => false,
+        }
+    }
+
+    fn scoring_update(
+        &mut self,
+        _phase: GamePhase,
+        _vehicle_scoring: &[VehicleScoringInfoV01],
+    ) -> Result<()> {
+        Ok(())
+    }
+
     fn telemetry_update(
         &mut self,
         _player_id: Option<i32>,
diff --git a/src/overlay/rfactor_data.rs b/src/overlay/rfactor_data.rs
index 32fc85d..0c3c76b 100644
--- a/src/overlay/rfactor_data.rs
+++ b/src/overlay/rfactor_data.rs
@@ -8,6 +8,10 @@ use crate::write_log;
 use super::UiOverlay;
 
 pub trait DataReceiver {
+    fn game_phase_change(&mut self, phase: GamePhase) -> Result<()>;
+
+    fn update_for_phase(&self, phase: GamePhase) -> bool;
+
     fn scoring_update(
         &mut self,
         phase: GamePhase,
@@ -21,13 +25,14 @@ pub trait DataReceiver {
     ) -> Result<()>;
 }
 
-#[derive(Clone, Copy, Debug)]
+#[derive(Clone, Copy, Debug, PartialEq, Eq, PartialOrd, Ord)]
 pub enum GamePhase {
     TestDay,
     Practice,
     Qualifying,
     Warmup,
     Race,
+    None,
 }
 
 impl TryFrom<i32> for GamePhase {
@@ -53,6 +58,7 @@ pub struct RFactorData {
 
     start_time: Instant,
     player_id: Option<i32>,
+    previous_game_phase: GamePhase,
 
     receivers: Vec<Rc<RefCell<dyn UiOverlay>>>,
 }
@@ -69,6 +75,7 @@ impl RFactorData {
 
             start_time,
             player_id: None,
+            previous_game_phase: GamePhase::None,
 
             receivers: Vec::new(),
         })
@@ -107,14 +114,28 @@ impl RFactorData {
                 }
             }
 
-            let phase = GamePhase::try_from(scoring_info.mSession)?;
+            {
+                let phase = GamePhase::try_from(scoring_info.mSession)?;
 
-            write_log!(format!("GamePhase: {:?}", phase));
+                if self.previous_game_phase != phase {
+                    self.previous_game_phase = phase;
+
+                    for receiver in self.receivers.iter() {
+                        receiver
+                            .borrow_mut()
+                            .game_phase_change(self.previous_game_phase)?;
+                    }
+                }
+
+                write_log!(format!("GamePhase: {:?}", self.previous_game_phase));
+            }
 
             for receiver in self.receivers.iter() {
-                receiver
-                    .borrow_mut()
-                    .scoring_update(phase, &vehicle_scorings)?;
+                let mut rec_mut = receiver.borrow_mut();
+
+                if rec_mut.update_for_phase(self.previous_game_phase) {
+                    rec_mut.scoring_update(self.previous_game_phase, &vehicle_scorings)?;
+                }
             }
         }
 
@@ -124,9 +145,11 @@ impl RFactorData {
             write_log!("new telemetry update");
 
             for receiver in self.receivers.iter() {
-                receiver
-                    .borrow_mut()
-                    .telemetry_update(self.player_id, &telemetries)?;
+                let mut rec_mut = receiver.borrow_mut();
+
+                if rec_mut.update_for_phase(self.previous_game_phase) {
+                    rec_mut.telemetry_update(self.player_id, &telemetries)?;
+                }
             }
         }
 
-- 
2.47.0


From d238556946574119cd8e146f5ab9e9dd902a6d70 Mon Sep 17 00:00:00 2001
From: hodasemi <michaelh.95@t-online.de>
Date: Thu, 19 Jan 2023 07:39:34 +0100
Subject: [PATCH 2/6] Add scoring update timeout

---
 src/overlay/elements/pedals/mod.rs |  44 ++++++----
 src/overlay/elements/radar/mod.rs  | 128 ++++++++++++++++-------------
 src/overlay/rfactor_data.rs        |  95 +++++++++++++--------
 3 files changed, 156 insertions(+), 111 deletions(-)

diff --git a/src/overlay/elements/pedals/mod.rs b/src/overlay/elements/pedals/mod.rs
index 59ac61e..d7825b8 100644
--- a/src/overlay/elements/pedals/mod.rs
+++ b/src/overlay/elements/pedals/mod.rs
@@ -26,6 +26,8 @@ pub struct Pedals {
     throttle: Arc<ProgressBar>,
     _history: Arc<Icon>,
 
+    enable: bool,
+
     throttle_samples: HeapRb<f32>,
     brake_samples: HeapRb<f32>,
 
@@ -138,6 +140,8 @@ impl Pedals {
             throttle,
             _history: history,
 
+            enable: false,
+
             throttle_samples,
             brake_samples,
 
@@ -202,7 +206,7 @@ impl Pedals {
         let command_buffer =
             CommandBuffer::new_primary().build(self.device.clone(), self.queue.clone())?;
 
-        {
+        if self.enable {
             let mut recorder = command_buffer.begin(VkCommandBufferBeginInfo::new(
                 VK_COMMAND_BUFFER_USAGE_ONE_TIME_SUBMIT_BIT
                     | VK_COMMAND_BUFFER_USAGE_SIMULTANEOUS_USE_BIT,
@@ -231,7 +235,18 @@ impl Pedals {
 impl UiOverlay for Pedals {}
 
 impl DataReceiver for Pedals {
-    fn game_phase_change(&mut self, _phase: GamePhase) -> Result<()> {
+    fn game_phase_change(&mut self, phase: GamePhase) -> Result<()> {
+        match phase {
+            GamePhase::None => {
+                self.enable = false;
+                self.gui.disable()?;
+            }
+            _ => {
+                self.enable = true;
+                self.gui.enable()?;
+            }
+        }
+
         Ok(())
     }
 
@@ -257,25 +272,18 @@ impl DataReceiver for Pedals {
         player_id: Option<i32>,
         telemetries: &[rF2VehicleTelemetry],
     ) -> Result<()> {
-        match player_id {
-            Some(id) => {
-                self.gui.enable()?;
+        if let Some(id) = player_id {
+            if let Some(telemetry) = telemetries.iter().find(|telemetry| telemetry.id == id) {
+                let brake = 1.0 - telemetry.unfiltered_brake as f32;
+                let throttle = 1.0 - telemetry.unfiltered_throttle as f32;
 
-                if let Some(telemetry) = telemetries.iter().find(|telemetry| telemetry.id == id) {
-                    let brake = 1.0 - telemetry.unfiltered_brake as f32;
-                    let throttle = 1.0 - telemetry.unfiltered_throttle as f32;
+                self.throttle.set_progress(throttle)?;
+                self.brake.set_progress(brake)?;
 
-                    self.throttle.set_progress(throttle)?;
-                    self.brake.set_progress(brake)?;
+                self.throttle_samples.push_overwrite(throttle);
+                self.brake_samples.push_overwrite(brake);
 
-                    self.throttle_samples.push_overwrite(throttle);
-                    self.brake_samples.push_overwrite(brake);
-
-                    self.update_vertex_buffers()?;
-                }
-            }
-            None => {
-                self.gui.disable()?;
+                self.update_vertex_buffers()?;
             }
         }
 
diff --git a/src/overlay/elements/radar/mod.rs b/src/overlay/elements/radar/mod.rs
index e4ddb81..32f415b 100644
--- a/src/overlay/elements/radar/mod.rs
+++ b/src/overlay/elements/radar/mod.rs
@@ -73,6 +73,8 @@ pub struct Radar {
     car_width: f32,
     car_height: f32,
 
+    enable: bool,
+
     device: Arc<Device>,
     queue: Arc<Mutex<Queue>>,
 
@@ -171,6 +173,8 @@ impl Radar {
             car_width,
             car_height,
 
+            enable: false,
+
             device,
             queue,
 
@@ -216,7 +220,7 @@ impl Radar {
         let command_buffer =
             CommandBuffer::new_primary().build(self.device.clone(), self.queue.clone())?;
 
-        {
+        if self.enable {
             let mut recorder = command_buffer.begin(VkCommandBufferBeginInfo::new(
                 VK_COMMAND_BUFFER_USAGE_ONE_TIME_SUBMIT_BIT
                     | VK_COMMAND_BUFFER_USAGE_SIMULTANEOUS_USE_BIT,
@@ -268,7 +272,12 @@ impl Radar {
 impl UiOverlay for Radar {}
 
 impl DataReceiver for Radar {
-    fn game_phase_change(&mut self, _phase: GamePhase) -> Result<()> {
+    fn game_phase_change(&mut self, phase: GamePhase) -> Result<()> {
+        match phase {
+            GamePhase::None => self.enable = false,
+            _ => self.enable = true,
+        }
+
         Ok(())
     }
 
@@ -299,68 +308,71 @@ impl DataReceiver for Radar {
 
         self.cars.clear();
 
-        if let Some(player_id) = player_id {
-            // make sure there are enough cars in buffer
-            if self.car_handles.len() < telemetries.len() {
-                let size_diff = telemetries.len() - self.car_handles.len();
+        if self.enable {
+            if let Some(player_id) = player_id {
+                // make sure there are enough cars in buffer
+                if self.car_handles.len() < telemetries.len() {
+                    let size_diff = telemetries.len() - self.car_handles.len();
 
-                for _ in 0..size_diff {
-                    self.car_handles
-                        .push(self.create_car_object(vec2(0.0, 0.0), [0.0, 0.0, 0.0, 0.0])?);
+                    for _ in 0..size_diff {
+                        self.car_handles
+                            .push(self.create_car_object(vec2(0.0, 0.0), [0.0, 0.0, 0.0, 0.0])?);
+                    }
+                }
+
+                let mut player_position = CarPosition::default();
+                let mut other_positions = Vec::new();
+
+                for telemetry in telemetries {
+                    let car = CarPosition::new(
+                        convert_vec(telemetry.position),
+                        [
+                            convert_vec(telemetry.orientation[0]),
+                            convert_vec(telemetry.orientation[1]),
+                            convert_vec(telemetry.orientation[2]),
+                        ],
+                    );
+
+                    if telemetry.id == player_id {
+                        player_position = car
+                    } else {
+                        other_positions.push(car);
+                    }
+                }
+
+                // update radar objects
+                let mut buffer_car_index = 0;
+
+                for other_position in other_positions {
+                    let diff = player_position.position - other_position.position;
+                    let distance = diff.magnitude();
+
+                    // check if car is close enough to the players car
+                    if distance < self.config.radar_car_distance {
+                        let offset =
+                            diff.xz() * (self.radar_extent / self.config.radar_car_distance);
+
+                        let buffered_car = self.car_handles[buffer_car_index].clone();
+                        buffer_car_index += 1;
+                        buffered_car.update(
+                            self.ortho,
+                            offset,
+                            player_position.rotation,
+                            other_position.rotation,
+                            self.radar_center,
+                            self.car_width,
+                            self.car_height,
+                            [0.9, 0.9, 0.0, 0.9],
+                        )?;
+
+                        self.cars.push(buffered_car);
+                    }
                 }
             }
 
-            let mut player_position = CarPosition::default();
-            let mut other_positions = Vec::new();
-
-            for telemetry in telemetries {
-                let car = CarPosition::new(
-                    convert_vec(telemetry.position),
-                    [
-                        convert_vec(telemetry.orientation[0]),
-                        convert_vec(telemetry.orientation[1]),
-                        convert_vec(telemetry.orientation[2]),
-                    ],
-                );
-
-                if telemetry.id == player_id {
-                    player_position = car
-                } else {
-                    other_positions.push(car);
-                }
-            }
-
-            // update radar objects
-            let mut buffer_car_index = 0;
-
-            for other_position in other_positions {
-                let diff = player_position.position - other_position.position;
-                let distance = diff.magnitude();
-
-                // check if car is close enough to the players car
-                if distance < self.config.radar_car_distance {
-                    let offset = diff.xz() * (self.radar_extent / self.config.radar_car_distance);
-
-                    let buffered_car = self.car_handles[buffer_car_index].clone();
-                    buffer_car_index += 1;
-                    buffered_car.update(
-                        self.ortho,
-                        offset,
-                        player_position.rotation,
-                        other_position.rotation,
-                        self.radar_center,
-                        self.car_width,
-                        self.car_height,
-                        [0.9, 0.9, 0.0, 0.9],
-                    )?;
-
-                    self.cars.push(buffered_car);
-                }
-            }
+            write_log!(format!("other cars: {:?}", self.cars.len()));
         }
 
-        write_log!(format!("other cars: {:?}", self.cars.len()));
-
         Ok(())
     }
 }
diff --git a/src/overlay/rfactor_data.rs b/src/overlay/rfactor_data.rs
index 0c3c76b..c542c3b 100644
--- a/src/overlay/rfactor_data.rs
+++ b/src/overlay/rfactor_data.rs
@@ -1,7 +1,11 @@
 use anyhow::Result;
 use rfactor_sm_reader::*;
 
-use std::{cell::RefCell, rc::Rc, time::Instant};
+use std::{
+    cell::RefCell,
+    rc::Rc,
+    time::{Duration, Instant},
+};
 
 use crate::write_log;
 
@@ -25,6 +29,8 @@ pub trait DataReceiver {
     ) -> Result<()>;
 }
 
+const GAME_PHASE_TIME_OUT: Duration = Duration::from_secs(2);
+
 #[derive(Clone, Copy, Debug, PartialEq, Eq, PartialOrd, Ord)]
 pub enum GamePhase {
     TestDay,
@@ -56,6 +62,7 @@ pub struct RFactorData {
     telemetry_reader: TelemetryReader,
     scoring_reader: ScoringReader,
 
+    last_scoring_read: Duration,
     start_time: Instant,
     player_id: Option<i32>,
     previous_game_phase: GamePhase,
@@ -73,6 +80,7 @@ impl RFactorData {
             telemetry_reader: TelemetryReader::new(start_time.elapsed().as_secs_f32())?,
             scoring_reader: ScoringReader::new(start_time.elapsed().as_secs_f32())?,
 
+            last_scoring_read: start_time.elapsed(),
             start_time,
             player_id: None,
             previous_game_phase: GamePhase::None,
@@ -93,48 +101,65 @@ impl RFactorData {
         write_log!(" =================== update RFactorData ===================");
 
         // get scoring info
-        if let Some((scoring_info, vehicle_scorings)) =
-            self.scoring_reader.vehicle_scoring(self.now())
-        {
-            write_log!(format!(
-                "new scoring info: vehicles: {}",
-                scoring_info.mNumVehicles
-            ));
+        match self.scoring_reader.vehicle_scoring(self.now()) {
+            Some((scoring_info, vehicle_scorings)) => {
+                self.last_scoring_read = self.start_time.elapsed();
 
-            // check for player id
-            if scoring_info.mNumVehicles == 0 {
-                self.player_id = None;
-            } else if self.player_id.is_none() {
-                for vehicle_scoring in vehicle_scorings.iter() {
-                    if vehicle_scoring.mIsPlayer != 0 {
-                        write_log!(format!("player found: {}", vehicle_scoring.mID));
-                        self.player_id = Some(vehicle_scoring.mID);
-                        break;
-                    }
-                }
-            }
+                write_log!(format!(
+                    "new scoring info: vehicles: {}",
+                    scoring_info.mNumVehicles
+                ));
 
-            {
-                let phase = GamePhase::try_from(scoring_info.mSession)?;
-
-                if self.previous_game_phase != phase {
-                    self.previous_game_phase = phase;
-
-                    for receiver in self.receivers.iter() {
-                        receiver
-                            .borrow_mut()
-                            .game_phase_change(self.previous_game_phase)?;
+                // check for player id
+                if scoring_info.mNumVehicles == 0 {
+                    self.player_id = None;
+                } else if self.player_id.is_none() {
+                    for vehicle_scoring in vehicle_scorings.iter() {
+                        if vehicle_scoring.mIsPlayer != 0 {
+                            write_log!(format!("player found: {}", vehicle_scoring.mID));
+                            self.player_id = Some(vehicle_scoring.mID);
+                            break;
+                        }
                     }
                 }
 
-                write_log!(format!("GamePhase: {:?}", self.previous_game_phase));
+                {
+                    let phase = GamePhase::try_from(scoring_info.mSession)?;
+
+                    if self.previous_game_phase != phase {
+                        self.previous_game_phase = phase;
+
+                        for receiver in self.receivers.iter() {
+                            receiver
+                                .borrow_mut()
+                                .game_phase_change(self.previous_game_phase)?;
+                        }
+                    }
+
+                    write_log!(format!("GamePhase: {:?}", self.previous_game_phase));
+                }
+
+                for receiver in self.receivers.iter() {
+                    let mut rec_mut = receiver.borrow_mut();
+
+                    if rec_mut.update_for_phase(self.previous_game_phase) {
+                        rec_mut.scoring_update(self.previous_game_phase, &vehicle_scorings)?;
+                    }
+                }
             }
+            None => {
+                let now = self.start_time.elapsed();
 
-            for receiver in self.receivers.iter() {
-                let mut rec_mut = receiver.borrow_mut();
+                if now > (self.last_scoring_read + GAME_PHASE_TIME_OUT) {
+                    if self.previous_game_phase != GamePhase::None {
+                        self.previous_game_phase = GamePhase::None;
 
-                if rec_mut.update_for_phase(self.previous_game_phase) {
-                    rec_mut.scoring_update(self.previous_game_phase, &vehicle_scorings)?;
+                        for receiver in self.receivers.iter() {
+                            receiver
+                                .borrow_mut()
+                                .game_phase_change(self.previous_game_phase)?;
+                        }
+                    }
                 }
             }
         }
-- 
2.47.0


From 420b8fa9141a2a80fe356da591d1ec6d91b01d31 Mon Sep 17 00:00:00 2001
From: hodasemi <michaelh.95@t-online.de>
Date: Thu, 19 Jan 2023 07:40:16 +0100
Subject: [PATCH 3/6] Leaderboard improvements

---
 .../leaderboard/leaderboard_entry.xml         |   2 +-
 src/overlay/elements/leaderboard/mod.rs       | 102 ++++++++++++++----
 2 files changed, 81 insertions(+), 23 deletions(-)

diff --git a/src/overlay/elements/leaderboard/leaderboard_entry.xml b/src/overlay/elements/leaderboard/leaderboard_entry.xml
index b3c182e..262de6f 100644
--- a/src/overlay/elements/leaderboard/leaderboard_entry.xml
+++ b/src/overlay/elements/leaderboard/leaderboard_entry.xml
@@ -1,6 +1,6 @@
 <?xml-model href="../gui.xsd" type="application/xml" schematypens="http://www.w3.org/2001/XMLSchema"?>
 <root>
-    <grid id="grid" x_dim="11" y_dim="1">
+    <grid x_dim="11" y_dim="1" padding="2" margin="2">
         <label id="place" x_slot="0" y_slot="0" text_color="black" text_alignment="right"></label>
         <label
             id="name"
diff --git a/src/overlay/elements/leaderboard/mod.rs b/src/overlay/elements/leaderboard/mod.rs
index e453eee..3d796df 100644
--- a/src/overlay/elements/leaderboard/mod.rs
+++ b/src/overlay/elements/leaderboard/mod.rs
@@ -16,7 +16,7 @@ use crate::write_log;
 
 pub struct LeaderBoard {
     gui_handler: Arc<GuiHandler>,
-    gui: Arc<GuiBuilder>,
+    leaderboard: Arc<GuiBuilder>,
     main_grid: Arc<Grid>,
 
     entries: Vec<LeaderBoardEntry>,
@@ -36,7 +36,7 @@ impl LeaderBoard {
 
         Ok(Self {
             gui_handler: gui_handler.clone(),
-            gui,
+            leaderboard: gui,
             main_grid,
 
             entries: Vec::new(),
@@ -85,7 +85,15 @@ impl LeaderBoard {
                         vehicle_scoring.mID,
                         driver_name,
                         vehicle_scoring.mPlace,
-                        vehicle_scoring.mTimeBehindLeader,
+                        {
+                            let laps_behind = vehicle_scoring.mLapsBehindLeader;
+
+                            if laps_behind != 0 {
+                                BehindLeader::Laps(laps_behind)
+                            } else {
+                                BehindLeader::Time(vehicle_scoring.mTimeBehindLeader)
+                            }
+                        },
                         vehicle_scoring.mTimeBehindNext,
                         vehicle_scoring.mBestLapTime,
                     )?;
@@ -133,7 +141,13 @@ impl LeaderBoard {
 
     fn race_leaderboard(&mut self, vehicle_scorings: &[VehicleScoringInfoV01]) -> Result<()> {
         self.update_leaderboard(vehicle_scorings, |entry, scoring| {
-            entry.update_time_behind_leader(scoring.mTimeBehindLeader)
+            let laps_behind = scoring.mLapsBehindLeader;
+
+            if laps_behind != 0 {
+                entry.update_time_behind_leader(BehindLeader::Laps(laps_behind))
+            } else {
+                entry.update_time_behind_leader(BehindLeader::Time(scoring.mTimeBehindLeader))
+            }
         })
     }
 
@@ -149,9 +163,25 @@ impl UiOverlay for LeaderBoard {}
 impl DataReceiver for LeaderBoard {
     fn game_phase_change(&mut self, phase: GamePhase) -> Result<()> {
         match phase {
-            GamePhase::Practice | GamePhase::Qualifying | GamePhase::Race => self.gui.enable(),
-            _ => self.gui.disable(),
+            GamePhase::Practice | GamePhase::Qualifying => {
+                for entry in self.entries.iter_mut() {
+                    entry.update_best_lap(-1.0)?;
+                }
+
+                self.leaderboard.enable()?
+            }
+
+            GamePhase::Race => {
+                for entry in self.entries.iter_mut() {
+                    entry.update_time_behind_leader(BehindLeader::Time(0.0))?;
+                }
+
+                self.leaderboard.enable()?;
+            }
+            _ => self.leaderboard.disable()?,
         }
+
+        Ok(())
     }
 
     fn update_for_phase(&self, phase: GamePhase) -> bool {
@@ -198,18 +228,23 @@ impl DataReceiver for LeaderBoard {
     }
 }
 
+#[derive(Debug, Clone, Copy, PartialEq, PartialOrd)]
+enum BehindLeader {
+    Time(f64),
+    Laps(i32),
+}
+
 struct LeaderBoardEntry {
     id: i32,
 
     name: String,
     place: u8,
-    time_behind_leader: f64,
+    behind: BehindLeader,
     time_behind_next: f64,
     best_lap: f64,
 
     snippet: Arc<GuiSnippet>,
 
-    grid: Arc<Grid>,
     name_label: Arc<Label>,
     place_label: Arc<Label>,
     time_label: Arc<Label>,
@@ -223,13 +258,12 @@ impl LeaderBoardEntry {
         id: i32,
         name: String,
         place: u8,
-        time_behind_leader: f64,
+        behind: BehindLeader,
         time_behind_next: f64,
         best_lap: f64,
     ) -> Result<Self> {
         let snippet = GuiSnippet::from_str(gui_handler, LeaderBoard::ENTRY)?;
 
-        let background = snippet.element("grid")?;
         let name_label: Arc<Label> = snippet.element("name")?;
         let place_label: Arc<Label> = snippet.element("place")?;
         let time_label: Arc<Label> = snippet.element("time")?;
@@ -243,13 +277,12 @@ impl LeaderBoardEntry {
 
             name,
             place,
-            time_behind_leader,
+            behind,
             time_behind_next,
             best_lap,
 
             snippet,
 
-            grid: background,
             name_label,
             place_label,
             time_label,
@@ -281,7 +314,11 @@ impl LeaderBoardEntry {
     }
 
     pub fn change_background_color(&self, color: Color) -> Result<()> {
-        self.grid.set_background(color)
+        self.name_label.set_background(color)?;
+        self.place_label.set_background(color)?;
+        self.time_label.set_background(color)?;
+
+        Ok(())
     }
 
     pub fn update_place(&mut self, place: u8) -> Result<()> {
@@ -294,16 +331,37 @@ impl LeaderBoardEntry {
         self.place_label.set_text(self.place)
     }
 
-    pub fn update_time_behind_leader(&mut self, time: f64) -> Result<()> {
-        if self.time_behind_leader != time {
-            self.time_behind_leader = time;
+    pub fn update_time_behind_leader(&mut self, behind: BehindLeader) -> Result<()> {
+        if self.behind != behind {
+            self.behind = behind;
 
-            // check if we are leader
-            if self.time_behind_leader == 0.0 {
-                self.time_label.set_text("---")?;
-            } else {
-                self.time_label
-                    .set_text(format!("+{:.3}", self.time_behind_leader))?;
+            match self.behind {
+                BehindLeader::Time(time_behind) => {
+                    // check if we are leader
+                    if time_behind == 0.0 {
+                        self.time_label.set_text("---")?;
+                    } else {
+                        let text = if time_behind > 60.0 {
+                            let full_minutes = (self.best_lap / 60.0).floor();
+                            let remainder = self.best_lap - (full_minutes * 60.0);
+
+                            format!("+{:.0}:{:.0}", full_minutes, remainder)
+                        } else {
+                            format!("+{:.3}", time_behind)
+                        };
+
+                        self.time_label.set_text(text)?;
+                    }
+                }
+                BehindLeader::Laps(laps_behind) => {
+                    let text = if laps_behind == 1 {
+                        format!("+{} Lap", laps_behind)
+                    } else {
+                        format!("+{} Laps", laps_behind)
+                    };
+
+                    self.time_label.set_text(text)?;
+                }
             }
         }
 
-- 
2.47.0


From 7cb07f1260b6fdbb0d6d44907608d7c44db492ca Mon Sep 17 00:00:00 2001
From: hodasemi <michaelh.95@t-online.de>
Date: Thu, 19 Jan 2023 10:23:21 +0100
Subject: [PATCH 4/6] Add delta board

---
 .../elements/leaderboard/deltaboard_grid.xml  |   6 +
 .../elements/leaderboard/leaderboard_entry.rs | 215 +++++++++++
 .../elements/leaderboard/leaderboard_grid.xml |   3 +-
 src/overlay/elements/leaderboard/mod.rs       | 350 +++++++-----------
 src/overlay/elements/pedals/mod.rs            |   2 +-
 src/overlay/elements/radar/mod.rs             |   2 +-
 6 files changed, 365 insertions(+), 213 deletions(-)
 create mode 100644 src/overlay/elements/leaderboard/deltaboard_grid.xml
 create mode 100644 src/overlay/elements/leaderboard/leaderboard_entry.rs

diff --git a/src/overlay/elements/leaderboard/deltaboard_grid.xml b/src/overlay/elements/leaderboard/deltaboard_grid.xml
new file mode 100644
index 0000000..9f9f760
--- /dev/null
+++ b/src/overlay/elements/leaderboard/deltaboard_grid.xml
@@ -0,0 +1,6 @@
+<?xml-model href="../gui.xsd" type="application/xml" schematypens="http://www.w3.org/2001/XMLSchema"?>
+<root reference_width="2560" reference_height="1440">
+    <!-- max 5 entries, each entry gets 39 pixel (at 1440p) -->
+    <grid id="main_grid" x_dim="1" y_dim="5" x_offset="-410" y_offset="10" width="400" height="195"
+        vert_align="top" hori_align="right" margin="0" padding="0"> </grid>
+</root>
\ No newline at end of file
diff --git a/src/overlay/elements/leaderboard/leaderboard_entry.rs b/src/overlay/elements/leaderboard/leaderboard_entry.rs
new file mode 100644
index 0000000..8a1acb4
--- /dev/null
+++ b/src/overlay/elements/leaderboard/leaderboard_entry.rs
@@ -0,0 +1,215 @@
+use std::sync::Arc;
+
+use anyhow::Result;
+
+use ui::prelude::*;
+use utilities::prelude::Color;
+
+#[derive(Debug, Clone, Copy, PartialEq, PartialOrd)]
+pub enum BehindLeader {
+    Time(f64),
+    Laps(i32),
+}
+
+pub struct LeaderBoardEntry {
+    id: i32,
+
+    name: String,
+    place: u8,
+    behind: BehindLeader,
+    time_behind_next: f64,
+    best_lap: f64,
+
+    snippet: Arc<GuiSnippet>,
+
+    name_label: Arc<Label>,
+    place_label: Arc<Label>,
+    time_label: Arc<Label>,
+
+    place_updated: bool,
+}
+
+impl LeaderBoardEntry {
+    const ENTRY: &str = include_str!("leaderboard_entry.xml");
+
+    pub fn empty(gui_handler: &Arc<GuiHandler>) -> Result<Self> {
+        Self::new(
+            gui_handler,
+            -1,
+            "".to_string(),
+            0,
+            BehindLeader::Laps(0),
+            0.0,
+            0.0,
+        )
+    }
+
+    pub fn new(
+        gui_handler: &Arc<GuiHandler>,
+        id: i32,
+        name: String,
+        place: u8,
+        behind: BehindLeader,
+        time_behind_next: f64,
+        best_lap: f64,
+    ) -> Result<Self> {
+        let snippet = GuiSnippet::from_str(gui_handler, Self::ENTRY)?;
+
+        let name_label: Arc<Label> = snippet.element("name")?;
+        let place_label: Arc<Label> = snippet.element("place")?;
+        let time_label: Arc<Label> = snippet.element("time")?;
+
+        name_label.set_text(&name)?;
+        place_label.set_text(place)?;
+        time_label.set_text("---")?;
+
+        Ok(Self {
+            id,
+
+            name,
+            place,
+            behind,
+            time_behind_next,
+            best_lap,
+
+            snippet,
+
+            name_label,
+            place_label,
+            time_label,
+
+            place_updated: true,
+        })
+    }
+
+    pub fn change_id(&mut self, id: i32) {
+        self.id = id;
+    }
+
+    pub fn id(&self) -> i32 {
+        self.id
+    }
+
+    pub fn place(&self) -> u8 {
+        self.place
+    }
+
+    pub fn name(&self) -> &str {
+        &self.name
+    }
+
+    pub fn change_name(&mut self, name: String) -> Result<()> {
+        self.name = name;
+
+        self.name_label.set_text(&self.name)
+    }
+
+    pub fn snippet(&self) -> Arc<GuiSnippet> {
+        self.snippet.clone()
+    }
+
+    pub fn change_background_color(&self, color: Color) -> Result<()> {
+        self.time_label.set_background(color)?;
+        self.name_label.set_background(color)?;
+        self.place_label.set_background(color)?;
+
+        Ok(())
+    }
+
+    pub fn _highlight(&self, color: Color) -> Result<()> {
+        self.name_label.set_background(color)
+    }
+
+    pub fn update_place(&mut self, place: u8) -> Result<()> {
+        if self.place != place {
+            self.place_updated = true;
+        }
+
+        self.place = place;
+
+        self.place_label.set_text(self.place)
+    }
+
+    pub fn update_time_behind_leader(&mut self, behind: BehindLeader) -> Result<()> {
+        if self.behind != behind {
+            self.behind = behind;
+
+            match self.behind {
+                BehindLeader::Time(time_behind) => {
+                    // check if we are leader
+                    if time_behind == 0.0 {
+                        self.time_label.set_text("---")?;
+                    } else {
+                        let text = if time_behind > 60.0 {
+                            let full_minutes = (self.best_lap / 60.0).floor();
+                            let remainder = self.best_lap - (full_minutes * 60.0);
+
+                            format!("+{:.0}:{:.0}", full_minutes, remainder)
+                        } else {
+                            format!("+{:.3}", time_behind)
+                        };
+
+                        self.time_label.set_text(text)?;
+                    }
+                }
+                BehindLeader::Laps(laps_behind) => {
+                    let text = if laps_behind == 1 {
+                        format!("+{} Lap", laps_behind)
+                    } else {
+                        format!("+{} Laps", laps_behind)
+                    };
+
+                    self.time_label.set_text(text)?;
+                }
+            }
+        }
+
+        Ok(())
+    }
+
+    pub fn update_best_lap(&mut self, time: f64) -> Result<()> {
+        if self.best_lap != time {
+            self.best_lap = time;
+
+            if self.best_lap < 0.0 {
+                self.time_label.set_text("---")?;
+            } else {
+                let text = if self.best_lap > 60.0 {
+                    let full_minutes = (self.best_lap / 60.0).floor();
+                    let remainder = self.best_lap - (full_minutes * 60.0);
+
+                    format!("{:.0}:{:.3}", full_minutes, remainder)
+                } else {
+                    format!("{:.3}", self.best_lap)
+                };
+
+                self.time_label.set_text(text)?;
+            }
+        }
+
+        Ok(())
+    }
+
+    pub fn update_time_behind_next(&mut self, time: f64) -> Result<()> {
+        self.time_behind_next = time;
+
+        let text = if self.time_behind_next > 60.0 {
+            let full_minutes = (self.time_behind_next / 60.0).floor();
+            let remainder = self.time_behind_next - (full_minutes * 60.0);
+
+            format!("+{:.0}:{:.3}", full_minutes, remainder)
+        } else {
+            format!("+{:.3}", self.time_behind_next)
+        };
+
+        self.time_label.set_text(text)
+    }
+
+    pub fn needs_resorting(&self) -> bool {
+        self.place_updated
+    }
+
+    pub fn resorting_finished(&mut self) {
+        self.place_updated = false;
+    }
+}
diff --git a/src/overlay/elements/leaderboard/leaderboard_grid.xml b/src/overlay/elements/leaderboard/leaderboard_grid.xml
index cab3c03..7b88df6 100644
--- a/src/overlay/elements/leaderboard/leaderboard_grid.xml
+++ b/src/overlay/elements/leaderboard/leaderboard_grid.xml
@@ -1,5 +1,6 @@
 <?xml-model href="../gui.xsd" type="application/xml" schematypens="http://www.w3.org/2001/XMLSchema"?>
 <root reference_width="2560" reference_height="1440">
-    <grid id="main_grid" x_dim="1" y_dim="25" x_offset="10" y_offset="10" width="400" height="875"
+    <!-- max 25 entries, each entry gets 39 pixel (at 1440p) -->
+    <grid id="main_grid" x_dim="1" y_dim="25" x_offset="10" y_offset="10" width="400" height="975"
         vert_align="top" hori_align="left" margin="0" padding="0"> </grid>
 </root>
\ No newline at end of file
diff --git a/src/overlay/elements/leaderboard/mod.rs b/src/overlay/elements/leaderboard/mod.rs
index 3d796df..ee662c3 100644
--- a/src/overlay/elements/leaderboard/mod.rs
+++ b/src/overlay/elements/leaderboard/mod.rs
@@ -1,3 +1,7 @@
+mod leaderboard_entry;
+
+use leaderboard_entry::*;
+
 use std::{
     ffi::{c_char, CStr},
     sync::Arc,
@@ -17,32 +21,53 @@ use crate::write_log;
 pub struct LeaderBoard {
     gui_handler: Arc<GuiHandler>,
     leaderboard: Arc<GuiBuilder>,
-    main_grid: Arc<Grid>,
+    deltaboard: Arc<GuiBuilder>,
 
-    entries: Vec<LeaderBoardEntry>,
+    leaderboard_grid: Arc<Grid>,
+    deltaboard_grid: Arc<Grid>,
+
+    leaderboard_entries: Vec<LeaderBoardEntry>,
+    deltaboard_entries: [LeaderBoardEntry; 5],
+
+    leaderboard_redraw: bool,
+    last_player_id: i32,
 
     entry_backgrounds: [Color; 2],
-    player_background: Color,
+    _player_background: Color,
 }
 
 impl LeaderBoard {
-    const GRID: &str = include_str!("leaderboard_grid.xml");
-    const ENTRY: &str = include_str!("leaderboard_entry.xml");
+    const LEADERBOARD: &str = include_str!("leaderboard_grid.xml");
+    const DELTABOARD: &str = include_str!("deltaboard_grid.xml");
 
     pub fn new(gui_handler: &Arc<GuiHandler>) -> Result<Self> {
-        let gui = GuiBuilder::from_str(gui_handler, Self::GRID)?;
+        let leaderboard = GuiBuilder::from_str(gui_handler, Self::LEADERBOARD)?;
+        let deltaboard = GuiBuilder::from_str(gui_handler, Self::DELTABOARD)?;
 
-        let main_grid = gui.element("main_grid")?;
+        let leaderboard_grid = leaderboard.element("main_grid")?;
+        let deltaboard_grid = deltaboard.element("main_grid")?;
 
         Ok(Self {
             gui_handler: gui_handler.clone(),
-            leaderboard: gui,
-            main_grid,
+            leaderboard,
+            deltaboard,
+            leaderboard_grid,
+            deltaboard_grid,
 
-            entries: Vec::new(),
+            leaderboard_entries: Vec::new(),
+            deltaboard_entries: [
+                LeaderBoardEntry::empty(gui_handler)?,
+                LeaderBoardEntry::empty(gui_handler)?,
+                LeaderBoardEntry::empty(gui_handler)?,
+                LeaderBoardEntry::empty(gui_handler)?,
+                LeaderBoardEntry::empty(gui_handler)?,
+            ],
+
+            leaderboard_redraw: false,
+            last_player_id: -1,
 
             entry_backgrounds: [Color::try_from("#838383")?, Color::try_from("#545454")?],
-            player_background: Color::try_from("#b4bf26")?,
+            _player_background: Color::try_from("#b4bf26")?,
         })
     }
 
@@ -66,7 +91,7 @@ impl LeaderBoard {
 
             // check driver list
             match self
-                .entries
+                .leaderboard_entries
                 .iter_mut()
                 .find(|entry| vehicle_scoring.mID == entry.id())
             {
@@ -98,7 +123,7 @@ impl LeaderBoard {
                         vehicle_scoring.mBestLapTime,
                     )?;
 
-                    self.entries.push(entry);
+                    self.leaderboard_entries.push(entry);
                 }
             }
         }
@@ -106,14 +131,15 @@ impl LeaderBoard {
         write_log!("create entries");
 
         // check if entry count in grid is the same as the gathered entries
-        let force_update = if !self.entries.is_empty()
+        let force_update = if !self.leaderboard_entries.is_empty()
             && self
-                .main_grid
-                .child_at(0, self.entries.len() - 1)?
+                .leaderboard_grid
+                .child_at(0, self.leaderboard_entries.len() - 1)?
                 .is_none()
         {
-            for i in 0..self.entries.len() {
-                self.main_grid.detach(0, i)?;
+            for (i, entry) in self.leaderboard_entries.iter().enumerate() {
+                self.leaderboard_grid.detach(0, i)?;
+                entry.snippet().set_visibility(false)?;
             }
 
             true
@@ -122,17 +148,80 @@ impl LeaderBoard {
         };
 
         // check if any entry needs resorting
-        if force_update || self.entries.iter().any(|entry| entry.needs_resorting()) {
+        if force_update
+            || self
+                .leaderboard_entries
+                .iter()
+                .any(|entry| entry.needs_resorting())
+        {
             write_log!("leader board update required");
 
-            self.entries
+            self.leaderboard_entries
                 .sort_by(|lhs, rhs| lhs.place().cmp(&rhs.place()));
 
-            for (i, entry) in self.entries.iter_mut().enumerate() {
+            for (i, entry) in self.leaderboard_entries.iter_mut().enumerate() {
                 entry.resorting_finished();
                 entry.change_background_color(self.entry_backgrounds[i % 2])?;
 
-                self.main_grid.attach(entry.snippet(), 0, i, 1, 1)?;
+                self.leaderboard_grid.attach(entry.snippet(), 0, i, 1, 1)?;
+            }
+
+            self.leaderboard_redraw = true;
+        }
+
+        // update delta board
+        if self.last_player_id != -1 {
+            write_log!("update delta board");
+
+            if let Some((index, _player_entry)) = self
+                .leaderboard_entries
+                .iter()
+                .enumerate()
+                .find(|(_index, entry)| entry.id() == self.last_player_id)
+            {
+                let mut start_index = if index >= 2 { index - 2 } else { 0 };
+
+                let max = self.leaderboard_entries.len().min(5);
+
+                write_log!(format!(
+                    "Delta Board: start {} - count {}",
+                    start_index, max
+                ));
+
+                // clear old entries
+                for i in 0..5 {
+                    if let Some(child) = self.deltaboard_grid.detach(0, i)? {
+                        if let Some(visiblity) = child.visibility() {
+                            visiblity.set_visibility(false)?;
+                        }
+                    }
+                }
+
+                // add new entries
+                for i in 0..max {
+                    if let Some(leaderboard_entry) = self.leaderboard_entries.get(start_index) {
+                        if let Some(entry) = self.deltaboard_entries.get_mut(i) {
+                            entry.change_id(leaderboard_entry.id());
+                            entry.update_place(leaderboard_entry.place())?;
+                            entry.change_background_color(self.entry_backgrounds[i % 2])?;
+
+                            if entry.name() != leaderboard_entry.name() {
+                                entry.change_name(leaderboard_entry.name().to_string())?;
+                            }
+
+                            if let Some(vehicle_scoring) = vehicle_scorings
+                                .iter()
+                                .find(|scoring| scoring.mID == entry.id())
+                            {
+                                entry.update_time_behind_next(vehicle_scoring.mTimeBehindNext)?;
+                            }
+
+                            self.deltaboard_grid.attach(entry.snippet(), 0, i, 1, 1)?;
+                        }
+
+                        start_index += 1;
+                    }
+                }
             }
         }
 
@@ -164,21 +253,27 @@ impl DataReceiver for LeaderBoard {
     fn game_phase_change(&mut self, phase: GamePhase) -> Result<()> {
         match phase {
             GamePhase::Practice | GamePhase::Qualifying => {
-                for entry in self.entries.iter_mut() {
+                for entry in self.leaderboard_entries.iter_mut() {
                     entry.update_best_lap(-1.0)?;
                 }
 
-                self.leaderboard.enable()?
+                self.leaderboard.enable()?;
+                self.deltaboard.enable()?;
             }
 
             GamePhase::Race => {
-                for entry in self.entries.iter_mut() {
+                for entry in self.leaderboard_entries.iter_mut() {
                     entry.update_time_behind_leader(BehindLeader::Time(0.0))?;
                 }
 
                 self.leaderboard.enable()?;
+                self.deltaboard.enable()?;
+            }
+            _ => {
+                self.last_player_id = -1;
+                self.leaderboard.disable()?;
+                self.deltaboard.disable()?;
             }
-            _ => self.leaderboard.disable()?,
         }
 
         Ok(())
@@ -200,10 +295,12 @@ impl DataReceiver for LeaderBoard {
 
         match phase {
             GamePhase::Practice | GamePhase::Qualifying => {
-                self.quali_leaderboard(vehicle_scorings)?
+                self.quali_leaderboard(vehicle_scorings)?;
             }
 
-            GamePhase::Race => self.race_leaderboard(vehicle_scorings)?,
+            GamePhase::Race => {
+                self.race_leaderboard(vehicle_scorings)?;
+            }
 
             _ => (),
         }
@@ -218,191 +315,24 @@ impl DataReceiver for LeaderBoard {
         player_id: Option<i32>,
         _telemetries: &[rF2VehicleTelemetry],
     ) -> Result<()> {
+        if self.leaderboard_redraw {
+            self.leaderboard_redraw = false;
+
+            // if let Some(player_id) = player_id {
+            //     if let Some(entry) = self.entries.iter().find(|entry| entry.id() == player_id) {
+            //         write_log!(format!(
+            //             "Update player entry background color: {:?}",
+            //             self.player_background
+            //         ));
+            //         entry.highlight(self.player_background)?;
+            //     }
+            // }
+        }
+
         if let Some(player_id) = player_id {
-            if let Some(entry) = self.entries.iter().find(|entry| entry.id() == player_id) {
-                entry.change_background_color(self.player_background)?;
-            }
+            self.last_player_id = player_id;
         }
 
         Ok(())
     }
 }
-
-#[derive(Debug, Clone, Copy, PartialEq, PartialOrd)]
-enum BehindLeader {
-    Time(f64),
-    Laps(i32),
-}
-
-struct LeaderBoardEntry {
-    id: i32,
-
-    name: String,
-    place: u8,
-    behind: BehindLeader,
-    time_behind_next: f64,
-    best_lap: f64,
-
-    snippet: Arc<GuiSnippet>,
-
-    name_label: Arc<Label>,
-    place_label: Arc<Label>,
-    time_label: Arc<Label>,
-
-    place_updated: bool,
-}
-
-impl LeaderBoardEntry {
-    pub fn new(
-        gui_handler: &Arc<GuiHandler>,
-        id: i32,
-        name: String,
-        place: u8,
-        behind: BehindLeader,
-        time_behind_next: f64,
-        best_lap: f64,
-    ) -> Result<Self> {
-        let snippet = GuiSnippet::from_str(gui_handler, LeaderBoard::ENTRY)?;
-
-        let name_label: Arc<Label> = snippet.element("name")?;
-        let place_label: Arc<Label> = snippet.element("place")?;
-        let time_label: Arc<Label> = snippet.element("time")?;
-
-        name_label.set_text(&name)?;
-        place_label.set_text(place)?;
-        time_label.set_text("---")?;
-
-        Ok(Self {
-            id,
-
-            name,
-            place,
-            behind,
-            time_behind_next,
-            best_lap,
-
-            snippet,
-
-            name_label,
-            place_label,
-            time_label,
-
-            place_updated: true,
-        })
-    }
-
-    pub fn id(&self) -> i32 {
-        self.id
-    }
-
-    pub fn place(&self) -> u8 {
-        self.place
-    }
-
-    pub fn name(&self) -> &str {
-        &self.name
-    }
-
-    pub fn change_name(&mut self, name: String) -> Result<()> {
-        self.name = name;
-
-        self.name_label.set_text(&self.name)
-    }
-
-    pub fn snippet(&self) -> Arc<GuiSnippet> {
-        self.snippet.clone()
-    }
-
-    pub fn change_background_color(&self, color: Color) -> Result<()> {
-        self.name_label.set_background(color)?;
-        self.place_label.set_background(color)?;
-        self.time_label.set_background(color)?;
-
-        Ok(())
-    }
-
-    pub fn update_place(&mut self, place: u8) -> Result<()> {
-        if self.place != place {
-            self.place_updated = true;
-        }
-
-        self.place = place;
-
-        self.place_label.set_text(self.place)
-    }
-
-    pub fn update_time_behind_leader(&mut self, behind: BehindLeader) -> Result<()> {
-        if self.behind != behind {
-            self.behind = behind;
-
-            match self.behind {
-                BehindLeader::Time(time_behind) => {
-                    // check if we are leader
-                    if time_behind == 0.0 {
-                        self.time_label.set_text("---")?;
-                    } else {
-                        let text = if time_behind > 60.0 {
-                            let full_minutes = (self.best_lap / 60.0).floor();
-                            let remainder = self.best_lap - (full_minutes * 60.0);
-
-                            format!("+{:.0}:{:.0}", full_minutes, remainder)
-                        } else {
-                            format!("+{:.3}", time_behind)
-                        };
-
-                        self.time_label.set_text(text)?;
-                    }
-                }
-                BehindLeader::Laps(laps_behind) => {
-                    let text = if laps_behind == 1 {
-                        format!("+{} Lap", laps_behind)
-                    } else {
-                        format!("+{} Laps", laps_behind)
-                    };
-
-                    self.time_label.set_text(text)?;
-                }
-            }
-        }
-
-        Ok(())
-    }
-
-    pub fn update_best_lap(&mut self, time: f64) -> Result<()> {
-        if self.best_lap != time {
-            self.best_lap = time;
-
-            if self.best_lap < 0.0 {
-                self.time_label.set_text("---")?;
-            } else {
-                let text = if self.best_lap > 60.0 {
-                    let full_minutes = (self.best_lap / 60.0).floor();
-                    let remainder = self.best_lap - (full_minutes * 60.0);
-
-                    format!("{:.0}:{:.3}", full_minutes, remainder)
-                } else {
-                    format!("{:.3}", self.best_lap)
-                };
-
-                self.time_label.set_text(text)?;
-            }
-        }
-
-        Ok(())
-    }
-
-    pub fn update_time_behind_next(&mut self, time: f64) -> Result<()> {
-        self.time_behind_next = time;
-
-        self.time_label
-            .set_text(format!("+{:.3}", self.time_behind_next))
-    }
-
-    pub fn needs_resorting(&self) -> bool {
-        self.place_updated
-    }
-
-    pub fn resorting_finished(&mut self) {
-        self.place_updated = false;
-    }
-}
diff --git a/src/overlay/elements/pedals/mod.rs b/src/overlay/elements/pedals/mod.rs
index d7825b8..3fbb83e 100644
--- a/src/overlay/elements/pedals/mod.rs
+++ b/src/overlay/elements/pedals/mod.rs
@@ -237,7 +237,7 @@ impl UiOverlay for Pedals {}
 impl DataReceiver for Pedals {
     fn game_phase_change(&mut self, phase: GamePhase) -> Result<()> {
         match phase {
-            GamePhase::None => {
+            GamePhase::None | GamePhase::TestDay => {
                 self.enable = false;
                 self.gui.disable()?;
             }
diff --git a/src/overlay/elements/radar/mod.rs b/src/overlay/elements/radar/mod.rs
index 32f415b..ef9cb43 100644
--- a/src/overlay/elements/radar/mod.rs
+++ b/src/overlay/elements/radar/mod.rs
@@ -274,7 +274,7 @@ impl UiOverlay for Radar {}
 impl DataReceiver for Radar {
     fn game_phase_change(&mut self, phase: GamePhase) -> Result<()> {
         match phase {
-            GamePhase::None => self.enable = false,
+            GamePhase::None | GamePhase::TestDay => self.enable = false,
             _ => self.enable = true,
         }
 
-- 
2.47.0


From a19dbf4d0d714668aa138a4d2a95eaa51b4acba0 Mon Sep 17 00:00:00 2001
From: hodasemi <michaelh.95@t-online.de>
Date: Thu, 19 Jan 2023 11:40:24 +0100
Subject: [PATCH 5/6] Fix typo

---
 src/overlay/elements/leaderboard/deltaboard_grid.xml   | 2 +-
 src/overlay/elements/leaderboard/leaderboard_entry.xml | 2 +-
 src/overlay/elements/leaderboard/leaderboard_grid.xml  | 2 +-
 src/overlay/elements/pedals/pedals.xml                 | 2 +-
 src/overlay/elements/watermark/watermark.xml           | 2 +-
 5 files changed, 5 insertions(+), 5 deletions(-)

diff --git a/src/overlay/elements/leaderboard/deltaboard_grid.xml b/src/overlay/elements/leaderboard/deltaboard_grid.xml
index 9f9f760..ad12bae 100644
--- a/src/overlay/elements/leaderboard/deltaboard_grid.xml
+++ b/src/overlay/elements/leaderboard/deltaboard_grid.xml
@@ -1,4 +1,4 @@
-<?xml-model href="../gui.xsd" type="application/xml" schematypens="http://www.w3.org/2001/XMLSchema"?>
+<?xml-model href="../gui.xsd" type="application/xml" schematypes="http://www.w3.org/2001/XMLSchema"?>
 <root reference_width="2560" reference_height="1440">
     <!-- max 5 entries, each entry gets 39 pixel (at 1440p) -->
     <grid id="main_grid" x_dim="1" y_dim="5" x_offset="-410" y_offset="10" width="400" height="195"
diff --git a/src/overlay/elements/leaderboard/leaderboard_entry.xml b/src/overlay/elements/leaderboard/leaderboard_entry.xml
index 262de6f..c67a5c2 100644
--- a/src/overlay/elements/leaderboard/leaderboard_entry.xml
+++ b/src/overlay/elements/leaderboard/leaderboard_entry.xml
@@ -1,4 +1,4 @@
-<?xml-model href="../gui.xsd" type="application/xml" schematypens="http://www.w3.org/2001/XMLSchema"?>
+<?xml-model href="../gui.xsd" type="application/xml" schematypes="http://www.w3.org/2001/XMLSchema"?>
 <root>
     <grid x_dim="11" y_dim="1" padding="2" margin="2">
         <label id="place" x_slot="0" y_slot="0" text_color="black" text_alignment="right"></label>
diff --git a/src/overlay/elements/leaderboard/leaderboard_grid.xml b/src/overlay/elements/leaderboard/leaderboard_grid.xml
index 7b88df6..4857d08 100644
--- a/src/overlay/elements/leaderboard/leaderboard_grid.xml
+++ b/src/overlay/elements/leaderboard/leaderboard_grid.xml
@@ -1,4 +1,4 @@
-<?xml-model href="../gui.xsd" type="application/xml" schematypens="http://www.w3.org/2001/XMLSchema"?>
+<?xml-model href="../gui.xsd" type="application/xml" schematypes="http://www.w3.org/2001/XMLSchema"?>
 <root reference_width="2560" reference_height="1440">
     <!-- max 25 entries, each entry gets 39 pixel (at 1440p) -->
     <grid id="main_grid" x_dim="1" y_dim="25" x_offset="10" y_offset="10" width="400" height="975"
diff --git a/src/overlay/elements/pedals/pedals.xml b/src/overlay/elements/pedals/pedals.xml
index feaf73c..bc1a6b7 100644
--- a/src/overlay/elements/pedals/pedals.xml
+++ b/src/overlay/elements/pedals/pedals.xml
@@ -1,4 +1,4 @@
-<?xml-model href="../gui.xsd" type="application/xml" schematypens="http://www.w3.org/2001/XMLSchema"?>
+<?xml-model href="../gui.xsd" type="application/xml" schematypes="http://www.w3.org/2001/XMLSchema"?>
 <root reference_width="2560" reference_height="1440">
     <grid x_dim="10" y_dim="2" x_offset="-850" y_offset="-190" width="200" height="160"
         vert_align="bottom" hori_align="right" margin="3" padding="3" background="#686868">
diff --git a/src/overlay/elements/watermark/watermark.xml b/src/overlay/elements/watermark/watermark.xml
index 6b4b81a..2c963ae 100644
--- a/src/overlay/elements/watermark/watermark.xml
+++ b/src/overlay/elements/watermark/watermark.xml
@@ -1,4 +1,4 @@
-<?xml-model href="../gui.xsd" type="application/xml" schematypens="http://www.w3.org/2001/XMLSchema"?>
+<?xml-model href="../gui.xsd" type="application/xml" schematypes="http://www.w3.org/2001/XMLSchema"?>
 <root reference_width="2560" reference_height="1440">
     <grid x_dim="1" y_dim="1" x_offset="10" y_offset="10" width="300" height="50"
         vert_align="top" hori_align="left" margin="2" padding="2" background="#c9c9c9">
-- 
2.47.0


From 2bdec6f9c7f2ac13c3bfaea4929d8a84dd8cf2c2 Mon Sep 17 00:00:00 2001
From: hodasemi <michaelh.95@t-online.de>
Date: Thu, 19 Jan 2023 13:41:08 +0100
Subject: [PATCH 6/6] highlight delta board player entry

---
 .../elements/leaderboard/leaderboard_entry.rs |  4 --
 .../leaderboard/leaderboard_entry.xml         |  4 +-
 src/overlay/elements/leaderboard/mod.rs       | 45 +++++++++++++------
 3 files changed, 33 insertions(+), 20 deletions(-)

diff --git a/src/overlay/elements/leaderboard/leaderboard_entry.rs b/src/overlay/elements/leaderboard/leaderboard_entry.rs
index 8a1acb4..6d5a9db 100644
--- a/src/overlay/elements/leaderboard/leaderboard_entry.rs
+++ b/src/overlay/elements/leaderboard/leaderboard_entry.rs
@@ -116,10 +116,6 @@ impl LeaderBoardEntry {
         Ok(())
     }
 
-    pub fn _highlight(&self, color: Color) -> Result<()> {
-        self.name_label.set_background(color)
-    }
-
     pub fn update_place(&mut self, place: u8) -> Result<()> {
         if self.place != place {
             self.place_updated = true;
diff --git a/src/overlay/elements/leaderboard/leaderboard_entry.xml b/src/overlay/elements/leaderboard/leaderboard_entry.xml
index c67a5c2..5d1f74f 100644
--- a/src/overlay/elements/leaderboard/leaderboard_entry.xml
+++ b/src/overlay/elements/leaderboard/leaderboard_entry.xml
@@ -4,9 +4,9 @@
         <label id="place" x_slot="0" y_slot="0" text_color="black" text_alignment="right"></label>
         <label
             id="name"
-            x_slot="1" y_slot="0" x_size="6" text_color="black" text_alignment="left"></label>
+            x_slot="1" y_slot="0" x_size="7" text_color="black"></label>
         <label
             id="time"
-            x_slot="7" y_slot="0" x_size="4" text_color="black" text_alignment="right"></label>
+            x_slot="8" y_slot="0" x_size="3" text_color="black" text_alignment="right"></label>
     </grid>
 </root>
\ No newline at end of file
diff --git a/src/overlay/elements/leaderboard/mod.rs b/src/overlay/elements/leaderboard/mod.rs
index ee662c3..3fee545 100644
--- a/src/overlay/elements/leaderboard/mod.rs
+++ b/src/overlay/elements/leaderboard/mod.rs
@@ -33,7 +33,7 @@ pub struct LeaderBoard {
     last_player_id: i32,
 
     entry_backgrounds: [Color; 2],
-    _player_background: Color,
+    player_background: Color,
 }
 
 impl LeaderBoard {
@@ -67,7 +67,7 @@ impl LeaderBoard {
             last_player_id: -1,
 
             entry_backgrounds: [Color::try_from("#838383")?, Color::try_from("#545454")?],
-            _player_background: Color::try_from("#b4bf26")?,
+            player_background: Color::try_from("#b4bf26")?,
         })
     }
 
@@ -128,6 +128,9 @@ impl LeaderBoard {
             }
         }
 
+        // TODO: when people disconnect there needs to be a remove function
+        // vehicle_scorings.len() != self.leaderboard_entries.len()
+
         write_log!("create entries");
 
         // check if entry count in grid is the same as the gathered entries
@@ -161,9 +164,14 @@ impl LeaderBoard {
 
             for (i, entry) in self.leaderboard_entries.iter_mut().enumerate() {
                 entry.resorting_finished();
-                entry.change_background_color(self.entry_backgrounds[i % 2])?;
 
-                self.leaderboard_grid.attach(entry.snippet(), 0, i, 1, 1)?;
+                // don't break here, just skip adding to grid
+                // because resorting_finished should be called for every entry
+                if i < self.leaderboard_grid.dimensions().1 {
+                    entry.change_background_color(self.entry_backgrounds[i % 2])?;
+
+                    self.leaderboard_grid.attach(entry.snippet(), 0, i, 1, 1)?;
+                }
             }
 
             self.leaderboard_redraw = true;
@@ -203,7 +211,12 @@ impl LeaderBoard {
                         if let Some(entry) = self.deltaboard_entries.get_mut(i) {
                             entry.change_id(leaderboard_entry.id());
                             entry.update_place(leaderboard_entry.place())?;
-                            entry.change_background_color(self.entry_backgrounds[i % 2])?;
+
+                            if entry.id() == self.last_player_id {
+                                entry.change_background_color(self.player_background)?;
+                            } else {
+                                entry.change_background_color(self.entry_backgrounds[i % 2])?;
+                            }
 
                             if entry.name() != leaderboard_entry.name() {
                                 entry.change_name(leaderboard_entry.name().to_string())?;
@@ -318,15 +331,19 @@ impl DataReceiver for LeaderBoard {
         if self.leaderboard_redraw {
             self.leaderboard_redraw = false;
 
-            // if let Some(player_id) = player_id {
-            //     if let Some(entry) = self.entries.iter().find(|entry| entry.id() == player_id) {
-            //         write_log!(format!(
-            //             "Update player entry background color: {:?}",
-            //             self.player_background
-            //         ));
-            //         entry.highlight(self.player_background)?;
-            //     }
-            // }
+            if let Some(player_id) = player_id {
+                if let Some(entry) = self
+                    .leaderboard_entries
+                    .iter()
+                    .find(|entry| entry.id() == player_id)
+                {
+                    write_log!(format!(
+                        "Update player entry background color: {:?}",
+                        self.player_background
+                    ));
+                    entry.change_background_color(self.player_background)?;
+                }
+            }
         }
 
         if let Some(player_id) = player_id {
-- 
2.47.0