add screen and capture for statistics

Author: SliDeeN

app/src/main/java/com/mg4/control/model/StatsPending.kt

@@ -14,6 +14,7 @@ data class PendingTrip(
     val odometerStart: Int?,
     val energyStart: Float?,
     val socStart: Float?,
+    /** Première température relevée. Ne sert plus qu'à reprendre un état de l'ancien format. */
     val tempC: Float?,
     /** Dernier relevé observé pendant le trajet — celui qui servira de fin en cas de coupure. */
     val lastMs: Long,
@@ -23,6 +24,9 @@ data class PendingTrip(
     val climateLast: Float?,
     val accessoriesLast: Float?,
     val regenLast: Float?,
+    /** Somme et nombre des températures relevées, pour en faire une moyenne. */
+    val tempSum: Float = 0f,
+    val tempCount: Int = 0,
     /** Distance accumulée en intégrant la vitesse, en km. Voir [Trip.integratedKm]. */
     val integratedKm: Float = 0f,
     /** Vitesse du relevé précédent, l'autre côté du trapèze. */
@@ -32,18 +36,37 @@ data class PendingTrip(
      * distinguerait pas d'une distance jamais mesurée — et l'écran annoncerait un trajet immobile.
      */
     val integrated: Boolean = false,
-)
+) {
+    val averageTempC: Float? get() = moyenne(tempSum, tempCount, tempC)
+}
 
 data class PendingCharge(
     val startMs: Long,
     val socStart: Float?,
     val type: ChargeType?,
+    /** Première température relevée. Ne sert plus qu'à reprendre un état de l'ancien format. */
     val tempC: Float?,
     val powerSum: Float,
     val powerCount: Int,
     val lastMs: Long,
     val socLast: Float?,
-)
+    val tempSum: Float = 0f,
+    val tempCount: Int = 0,
+) {
+    val averageTempC: Float? get() = moyenne(tempSum, tempCount, tempC)
+}
+
+/**
+ * Moyenne des températures relevées pendant un trajet ou une charge, **et non celle du départ**.
+ *
+ * Le capteur extérieur de la MG4 est lent et mal placé : une voiture qui sort d'un garage annonce
+ * la température du garage pendant plusieurs minutes. Sous contact, le collecteur relève à cadence
+ * fixe, la moyenne des échantillons vaut donc une moyenne dans le temps.
+ *
+ * [repli] couvre un état enregistré avant cette mesure, qui ne portait que la première valeur.
+ */
+private fun moyenne(sum: Float, count: Int, repli: Float?): Float? =
+    if (count > 0) (sum / count).roundTenth() else repli
 
 /** Les deux ensemble, tels qu'ils sont enregistrés. */
 data class PendingState(

app/src/main/java/com/mg4/control/model/StatsTracker.kt

@@ -85,6 +85,8 @@ class StatsTracker(private val capacityKwh: Float = StatsSettings.DEFAULT_CAPACI
                 energyStart = snapshot.energySinceStartKwh,
                 socStart = snapshot.socPercent,
                 tempC = snapshot.outsideTempC,
+                tempSum = snapshot.outsideTempC ?: 0f,
+                tempCount = if (snapshot.outsideTempC != null) 1 else 0,
                 lastMs = snapshot.timestampMs,
                 odometerLast = snapshot.odometerKm,
                 energyLast = snapshot.energySinceStartKwh,
@@ -101,6 +103,9 @@ class StatsTracker(private val capacityKwh: Float = StatsSettings.DEFAULT_CAPACI
                 climateLast = snapshot.climateSinceStartKwh ?: enCours.climateLast,
                 accessoriesLast = snapshot.accessoriesSinceStartKwh ?: enCours.accessoriesLast,
                 regenLast = snapshot.regenSinceStartKwh ?: enCours.regenLast,
+                // Moyenne et non valeur de départ : voir [moyenne].
+                tempSum = enCours.tempSum + (snapshot.outsideTempC ?: 0f),
+                tempCount = enCours.tempCount + if (snapshot.outsideTempC != null) 1 else 0,
                 integratedKm = enCours.integratedKm + (pas ?: 0f),
                 integrated = enCours.integrated || pas != null,
                 // Une vitesse manquée ne réinitialise pas le trapèze : le prochain intervalle
@@ -129,6 +134,8 @@ class StatsTracker(private val capacityKwh: Float = StatsSettings.DEFAULT_CAPACI
                 socStart = snapshot.socPercent,
                 type = snapshot.chargeType,
                 tempC = snapshot.outsideTempC,
+                tempSum = snapshot.outsideTempC ?: 0f,
+                tempCount = if (snapshot.outsideTempC != null) 1 else 0,
                 powerSum = 0f,
                 powerCount = 0,
                 lastMs = snapshot.timestampMs,
@@ -140,6 +147,10 @@ class StatsTracker(private val capacityKwh: Float = StatsSettings.DEFAULT_CAPACI
                 lastMs = snapshot.timestampMs,
                 socLast = snapshot.socPercent ?: chargeEnCours.socLast,
                 type = chargeEnCours.type ?: snapshot.chargeType,
+                // Une charge de nuit traverse plusieurs degrés : la moyenne dit quelque chose,
+                // la température du branchement ne dit rien.
+                tempSum = chargeEnCours.tempSum + (snapshot.outsideTempC ?: 0f),
+                tempCount = chargeEnCours.tempCount + if (snapshot.outsideTempC != null) 1 else 0,
             )
         } else if (snapshot.charging == false && chargeEnCours != null) {
             events += Event.ChargeEnded(
@@ -191,7 +202,7 @@ class StatsTracker(private val capacityKwh: Float = StatsSettings.DEFAULT_CAPACI
             regenKwh = p.regenLast,
             socStart = p.socStart,
             socEnd = p.socLast,
-            outsideTempC = p.tempC,
+            outsideTempC = p.averageTempC,
             integratedKm = integre,
         )
     }
@@ -206,7 +217,7 @@ class StatsTracker(private val capacityKwh: Float = StatsSettings.DEFAULT_CAPACI
             socEnd = socEnd,
             energyKwh = delta?.takeIf { it > 0f }?.let { (it / 100f * capacityKwh).roundTenth() },
             measuredPowerKw = if (p.powerCount > 0) (p.powerSum / p.powerCount).roundTenth() else null,
-            outsideTempC = p.tempC,
+            outsideTempC = p.averageTempC,
         )
     }
 

app/src/main/res/values-de/strings.xml

@@ -610,7 +610,7 @@
     <string name="stats_detail_accessories">Nebenverbraucher</string>
     <string name="stats_detail_regen">Rekuperation</string>
     <string name="stats_detail_battery">Batterie</string>
-    <string name="stats_detail_temp">Außentemperatur</string>
+    <string name="stats_detail_temp">Durchschnittstemperatur</string>
     <string name="stats_detail_power_measured">Mittlere Leistung, gemessen</string>
     <string name="stats_detail_power_computed">Mittlere Leistung, abgeleitet (Energie ÷ Dauer)</string>
     <string name="stats_detail_tariff">Angewandter Tarif</string>

app/src/main/res/values-en/strings.xml

@@ -610,7 +610,7 @@
     <string name="stats_detail_accessories">Accessories</string>
     <string name="stats_detail_regen">Regenerative braking</string>
     <string name="stats_detail_battery">Battery</string>
-    <string name="stats_detail_temp">Outside temperature</string>
+    <string name="stats_detail_temp">Average temperature</string>
     <string name="stats_detail_power_measured">Average power, measured</string>
     <string name="stats_detail_power_computed">Average power, derived (energy ÷ duration)</string>
     <string name="stats_detail_tariff">Rate applied</string>

app/src/main/res/values-es/strings.xml

@@ -610,7 +610,7 @@
     <string name="stats_detail_accessories">Accesorios</string>
     <string name="stats_detail_regen">Frenada regenerativa</string>
     <string name="stats_detail_battery">Batería</string>
-    <string name="stats_detail_temp">Temperatura exterior</string>
+    <string name="stats_detail_temp">Temperatura media</string>
     <string name="stats_detail_power_measured">Potencia media, medida</string>
     <string name="stats_detail_power_computed">Potencia media, deducida (energía ÷ duración)</string>
     <string name="stats_detail_tariff">Tarifa aplicada</string>

app/src/main/res/values-it/strings.xml

@@ -610,7 +610,7 @@
     <string name="stats_detail_accessories">Accessori</string>
     <string name="stats_detail_regen">Frenata rigenerativa</string>
     <string name="stats_detail_battery">Batteria</string>
-    <string name="stats_detail_temp">Temperatura esterna</string>
+    <string name="stats_detail_temp">Temperatura media</string>
     <string name="stats_detail_power_measured">Potenza media, misurata</string>
     <string name="stats_detail_power_computed">Potenza media, dedotta (energia ÷ durata)</string>
     <string name="stats_detail_tariff">Tariffa applicata</string>

app/src/main/res/values-pt/strings.xml

@@ -610,7 +610,7 @@
     <string name="stats_detail_accessories">Acessórios</string>
     <string name="stats_detail_regen">Travagem regenerativa</string>
     <string name="stats_detail_battery">Bateria</string>
-    <string name="stats_detail_temp">Temperatura exterior</string>
+    <string name="stats_detail_temp">Temperatura média</string>
     <string name="stats_detail_power_measured">Potência média, medida</string>
     <string name="stats_detail_power_computed">Potência média, deduzida (energia ÷ duração)</string>
     <string name="stats_detail_tariff">Tarifa aplicada</string>

app/src/main/res/values/strings.xml

@@ -623,7 +623,7 @@
     <string name="stats_detail_accessories">Accessoires</string>
     <string name="stats_detail_regen">Récupération au freinage</string>
     <string name="stats_detail_battery">Batterie</string>
-    <string name="stats_detail_temp">Température extérieure</string>
+    <string name="stats_detail_temp">Température moyenne</string>
     <string name="stats_detail_power_measured">Puissance moyenne mesurée</string>
     <string name="stats_detail_power_computed">Puissance moyenne déduite (énergie ÷ durée)</string>
     <string name="stats_detail_tariff">Tarif appliqué</string>

app/src/test/java/com/mg4/control/model/StatsTrackerTest.kt

@@ -27,6 +27,7 @@ class StatsTrackerTest {
         regen: Float? = null,
         vitesse: Float? = null,
         pasMs: Long = 30_000L,
+        temp: Float? = null,
     ): EnergySnapshot {
         horloge += pasMs
         return EnergySnapshot(
@@ -39,6 +40,7 @@ class StatsTrackerTest {
             powerKw = puissance,
             regenSinceStartKwh = regen,
             speedKmh = vitesse,
+            outsideTempC = temp,
         )
     }
 
@@ -99,6 +101,27 @@ class StatsTrackerTest {
         assertEquals(20f, trip.distance, 0.01f)
     }
 
+    @Test
+    fun `la temperature retenue est la moyenne du trajet, pas celle du depart`() {
+        val t = StatsTracker()
+        // Départ dans un garage à 18°, puis la vraie température dehors.
+        t.onSnapshot(snap(odo = 10_000, temp = 18f), ready = true)
+        t.onSnapshot(snap(odo = 10_010, temp = 6f), ready = true)
+        t.onSnapshot(snap(odo = 10_020, temp = 4f), ready = true)
+        val trip = finDe(t.onSnapshot(snap(odo = 10_020, temp = 4f), ready = false))!!
+        assertEquals("(18 + 6 + 4) / 3", 9.3f, trip.outsideTempC!!, 0.05f)
+    }
+
+    @Test
+    fun `une charge de nuit retient la temperature moyenne`() {
+        val t = StatsTracker()
+        t.onSnapshot(snap(charge = true, soc = 20f, temp = 12f, type = ChargeType.AC), ready = false)
+        t.onSnapshot(snap(charge = true, soc = 50f, temp = 6f, pasMs = 3_600_000L), ready = false)
+        val session = (t.onSnapshot(snap(charge = false, soc = 80f, temp = 3f, pasMs = 3_600_000L), ready = false)
+            .single() as StatsTracker.Event.ChargeEnded).session
+        assertEquals("(12 + 6) / 2 — le relevé de fin clôt la session", 9f, session.outsideTempC!!, 0.05f)
+    }
+
     @Test
     fun `mettre le contact sans rouler ne cree pas de trajet`() {
         val t = StatsTracker()