AlarmConfig.duration units are unverified ("TODO plz verify")

Offen
#36 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

Bewertung

Schwierigkeit
4/5
Geschätzter Aufwand
3-5 Tage
Anfängerfreundlichkeit
38/100
Issue-Typ
Bug
Klarheit
Größtenteils klar
Aktivitätsstatus
Ruhig
Tech-Stack
rust
Bereich
backend

Rechercherichtung

Read crates/podd-core/src/config/mod.rs and sensor/manager.rs, especially the duration flow and in_alarm_window; the issue also points to seconds-based window construction and firmware start-log fixtures in state.rs. Confirm the firmware's duration unit on hardware, then compare it with the host-side alarm window. Done means the unit is verified and the TODO is removed.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Beschreibung

question

AlarmConfig.duration carries the comment duration in seconds (TODO plz verify) (crates/podd-core/src/config/mod.rs:58). This value flows straight into AlarmCommand.duration (sensor/manager.rs:683-685) and is compared against a seconds-based alarm window (in_alarm_window builds alarm_end from Span::seconds(alarm_cfg.duration), manager.rs:655-656). If the firmware actually interprets the field as milliseconds (the FW start log reports dur 600000 ms in the state.rs test fixtures), the host-side window and the firmware run-length would disagree. Worth confirming the unit on hardware and removing the TODO.


Filed from a repo-wide code audit (working tree at bcad746).

Vorherrschende Sprache
Rust
Sterne
0
Forks
0
PR-Merge-Kennzahlen
Keine gemergten PRs in 30 T.

Entwicklungsumgebung

Dieses Projekt bietet weder Dev-Container noch Dockerfile noch Beitragsleitfaden – die Einrichtung liegt bei Ihnen. Beginnen Sie mit der README; die allgemeinen Schritte stehen in unserem Leitfaden für den ersten Beitrag.

Erste Schritte

  1. Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
  3. Forken Sie das Repository und arbeiten Sie in einem Branch.
  4. Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.

Mehr aus GoogleBot42/podd

Alle Issues in GoogleBot42/podd

Ähnliche Issues

Weitere Issues zu Rust

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.