Presence calibration results are never written to disk (lost on every restart)

Ouverte Adaptée aux débutants
#27 0 commentaires 0 réactions 0 personnes assignées Voir sur GitHub

Personne n'a encore pris cette issue.

Évaluation

Difficulté
2/5
Temps estimé
1-3 heures
Accessibilité débutants
68/100
Type d'issue
Bug
Clarté
Clairement spécifiée
Activité
Calme
Stack technique
rust
Domaine
backend

Piste de recherche

Lisez crates/podd-core/src/sensor/presence.rs autour des lignes 145–151 et comparez-le au chemin de persistance dans config/mqtt.rs autour de la ligne 292. Suivez les appels à Config::save et la gestion des modifications de configuration ; le travail est terminé lorsque les valeurs de référence de calibration sont enregistrées et restent disponibles après le redémarrage d’un processus ou d’une tâche de capteur.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Description

bug

PresenseManager::update_calibration computes new baselines and does self.config_tx.send(config) (crates/podd-core/src/sensor/presence.rs:145-151) under a comment that literally says "update config file" — but it never calls Config::save. Unlike the MQTT set-action path (config/mqtt.rs:292), nothing subscribes to config_tx to persist it, so a calibration only lives in the in-memory watch value. After any process/sensor-task restart the presence config reverts to whatever is on disk (or None, re-triggering the "please calibrate" warning). Calibration should persist via Config::save like handle_action does.


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

Langage dominant
Rust
Étoiles
0
Forks
0
Métriques de merge des PR
Aucune PR mergée en 30 j

Préparer son environnement

Ce projet ne fournit ni conteneur de développement, ni Dockerfile, ni guide de contribution : l'installation est à votre charge. Commencez par son README, et consultez notre guide de la première contribution pour les étapes générales.

Par où commencer

  1. Lisez l'issue en entier, puis le guide de contribution du projet.
  2. Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
  3. Forkez le dépôt et travaillez sur une branche.
  4. Ouvrez une pull request qui référence le numéro de l'issue.

Autres issues de GoogleBot42/podd

Toutes les issues de GoogleBot42/podd

Issues similaires

Plus d'issues Rust

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.