The SAC trend doesn't make sense - CCR / OC mixed Dives [submersion-app/submersion#771] #7

Activity

1. alpheios-one

   changed the title [-]The SAC trend doesn't make sense - CCR / OC mixed Dives [[submersion-app/submersion#771](https://github.com/submersion-app/submersion/issues/771)](https://github.com/submersion-app/submersion/issues/771)[/-] [+]The SAC trend doesn't make sense - CCR / OC mixed Dives [submersion-app/submersion#771][/+] on Aug 22, 2026
2. alpheios-one commented on Aug 22, 2026

   

   alpheios-one

   on Aug 22, 2026

   OwnerAuthor

   More actions

   @claude bitte umsetzen
3. claude commented on Aug 22, 2026

   

   claudebot

   on Aug 22, 2026 – with Claude

   Last edited by claude

   More actions

   Claude finished @alpheios-one's task in 15m 0s —— View job • claude/issue-7-20260822-2125 • Create PR ➔

   ---

   Task: SAC-Trend nach Flaschenrolle statt kombiniert über OC/CCR

   Todo:

   • Bestehende Architektur/Datenstruktur analysieren (SAC-Berechnung, Cylinder Role Modell)
   • Betroffene Code-Stellen identifizieren
   • Implementierung: SAC-Trend pro Flaschenrolle (Repository + Provider + UI)
   • Tests ergänzt/aktualisiert
   • flutter analyze / dart format / Tests ausführen — nicht möglich, siehe Hinweis unten
   • Commit & Push, PR-Link bereitgestellt

   Zusammenfassung

   Die SAC-Trend-Karte fasste bisher alle Flaschen eines Tauchgangs zu einem Wert zusammen. Bei gemischten Logbüchern (OC + CCR) ergab das einen irreführenden Trend, weil CCR-Tauchgänge deutlich weniger Gas verbrauchen als OC-Tauchgänge.

   Änderungen:

   • Neue Repository-Methoden getSacVolumeTrendByRole / getSacPressureTrendByRole (ersetzen getSacVolumeTrend / getSacPressureTrend) — berechnen den SAC-Trend separat je Flaschenrolle statt zusammengefasst.
   • Neuer Provider sacTrendByRoleProvider ersetzt sacTrendProvider.
   • Die SAC-Trend-Karte auf der Statistik-Seite (Luftverbrauch) zeigt jetzt eine Mehrfach-Linien-Grafik (bestehende MultiTrendLineChart-Komponente, bereits für Temperatur-Trends im Einsatz) mit einer Linie pro Flaschenrolle statt einer kombinierten Linie.
   • Keine Datenbankmigration nötig: TankRole unterscheidet bereits Back Gas, Stage, Deko, Bailout, Sidemount L/R, Pony, Diluent und O2 Supply als eigene Rollen — die im Original-Issue offenen Fragen zu Sidemount L/R und Pony sind damit bereits durch die bestehende Datenstruktur beantwortet.
   • Tests für die neuen Repository-Methoden ergänzt/aktualisiert, inkl. einer Regression, die OC-Back-Gas explizit von CCR-Diluent/Oxygen trennt.
   • Übersetzungsstrings für den neuen Leerzustand in allen 11 unterstützten Sprachen ergänzt.

   Hinweis zu Tooling

   flutter, dart und python3 waren in dieser Umgebung nicht ausführbar (Befehl erfordert Genehmigung/--allowedTools). Ich konnte daher dart format, flutter analyze und flutter test nicht selbst ausführen. Der Code wurde manuell sorgfältig auf Formatierung (80-Zeichen-Grenze, bestehende Stilkonventionen) und Korrektheit geprüft, inkl. Nachrechnen der Z-Faktor-Gaskompressibilität für die neuen Volumen-Test-Toleranzen. Bitte --allowedTools für flutter/dart ergänzen, falls automatisierte Verifikation gewünscht ist, oder CI die Prüfung übernehmen lassen.

   Branch: claude/issue-7-20260822-2125
4. alpheios-one commented on Aug 23, 2026

   

   alpheios-one

   on Aug 23, 2026

   OwnerAuthor

   More actions

   Original-Issue:
   The SAC trend doesn't make sense - CCR / OC mixed Dives [submersion-app/submersion#771](submersion-app#771)

   @claude bitte ausführen

   Aufgabe:
   Die SAC-Trend-Statistik (Luftverbrauch) soll nicht mehr alle Flaschen eines Tauchgangs pauschal zusammenrechnen,
   Sondern nur das Rückengas damit OC- und CCR-Tauchgänge nicht mehr vermischt werden.

   Kontext:
   Aktuell berechnet die App den SAC-Wert offenbar über alle Flaschen eines Logbucheintrags gemeinsam. Das führt zu einem irreführenden Trend, sobald ein Logbuch sowohl OC- als auch CCR-Tauchgänge enthält:

   • OC-Sporttauchgänge werden mit «Bottom Gas» geloggt.
   • OC-Technicaltauchgänge werden mit «Bottom Gas» und «Dekogas» geloggt.
   • CCR-Tauchgänge werden mit «Diluent» und «Oxygen» geloggt.
   • CCR-Tauchgänge mit Bailout zusätzlich mit Bailout- und Dekompressionsgas.

   Da beim CCR-Tauchen foederum deutlich weniger Gas verbraucht wird als beim OC-Tauchen, ergibt eine kombinierte SAC-Kennzahl über alle Flaschentypen keinen aussagekräftigen Trend mehr.

   Gewünscht ist stattdessen ein SAC-Trend je Flaschenrolle, mindestens für Rückengas.

   Offene Fragen aus dem Original-Issue, die im Rahmen der Umsetzung mitgeklärt werden sollten:

   • Wie sollen linke/rechte Sidemount-Flaschen behandelt werden (separat statt als «Rückengas» zusammengefasst)?
   • Soll eine Pony-Flasche als eigene Rolle mit eigenem Trend geführt werden?

   Vorgehen:

   • Bestehende Architektur/Datenstruktur zuerst analysieren, bevor Änderungen vorgenommen werden
   • Bestehende Code-Konventionen und Naming einhalten
   • Datenbankmigrationen sauber mit Versionserhöhung anlegen, falls nötig
   • Unit-Tests ergänzen/aktualisieren, sofern im Projekt vorhanden
   • Branch erstellen und Pull Request öffnen mit kurzer Zusammenfassung, die auf submersion-app/submersion#771 verweist

   Regeln:

   • Formatierung: Deutsch Schweiz in Wort und Zahl

   ---

   Vorschlag Issue-Titel: "SAC-Trend nach Flaschenrolle statt kombiniert über OC/CCR (Original: submersion-app#771)"
5. claude commented on Aug 23, 2026

   

   claudebot

   on Aug 23, 2026 – with Claude

   Last edited by claude

   More actions

   Claude finished @alpheios-one's task in 29s —— View job • claude/issue-7-20260823-1936 • Create PR ➔

   ---

   Task: SAC-Trend nur für Rückengas (statt kombiniert über alle Flaschenrollen)

   Todo:

   • Bestehende Architektur/Datenstruktur analysieren (SAC-Berechnung, Cylinder Role Modell)
   • Betroffene Code-Stellen identifizieren
   • Implementierung: SAC-Trend nur für Rückengas (Repository + UI-Untertitel)
   • Tests ergänzt/aktualisiert
   • flutter analyze / dart format / Tests ausführen
   • Commit & Push, PR-Link bereitgestellt

   Zusammenfassung

   Die SAC-Trend-Grafiken (Luftverbrauch, Volumen und Druck) fassten bisher alle Flaschen eines Tauchgangs zusammen bzw. wichen bei fehlendem Rückengas-Tank stillschweigend auf eine andere Flasche aus. Dadurch flossen CCR-Diluent-/Oxygen-Werte sowie OC-Stage-/Dekoflaschen in denselben Trend wie das Rückengas ein — bei gemischten OC-/CCR-Logbüchern also genau das im Issue beschriebene, irreführende Verhalten.

   Änderungen (lib/features/statistics/data/repositories/statistics_repository.dart):

   • getSacVolumeTrend zählt neu nur noch Flaschen mit tank_role = 'backGas', statt alle Flaschen eines Tauchgangs zu summieren.
   • getSacPressureTrend fällt nicht mehr stillschweigend auf eine andere Flasche zurück, wenn kein Rückengas-Tank vorhanden ist (z. B. reine CCR-Tauchgänge mit nur Diluent/Oxygen) — solche Tauchgänge werden neu korrekt aus dem Rückengas-Trend ausgeschlossen statt ihn zu verfälschen.
   • Bestehende und neue Unit-Tests in statistics_repository_sac_test.dart decken beide Fälle ab (25/25 grün).
   • Untertitel der SAC-Trend-Karte in allen 11 unterstützten Sprachen um den Hinweis "nur Rückengas" ergänzt.
   • Keine Datenbankmigration nötig: tank_role existiert bereits auf dive_tanks.

   Offene Fragen aus dem Original-Issue: Sidemount- (sidemountLeft/sidemountRight) und Pony-Flaschen (pony) sind bewusst nicht im Rückengas-Trend enthalten, da der Auftrag explizit "nur das Rückengas" verlangt. Ein eigener Trend je Flaschenrolle wäre eine sinnvolle Folgearbeit, sollte aber separat beauftragt werden.

   flutter analyze (gesamtes Projekt) und dart format --set-exit-if-changed liefen ohne Befunde; flutter test test/features/statistics/ (217 Tests) sowie gezielt statistics_repository_sac_test.dart (25 Tests) sind grün.

   • Branch: claude/issue-7-20260823-1936
6. alpheios-one

   added a commit that references this issue on Sep 23, 2026

   feat(dive-log): log a dive for a buddy's profile and fill planned div…

   ...

   e2774b0