Watch faces display obviously bogus 0 as heart rate
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Aptitud para principiantes
- 45/100
- Tipo de issue
- Error
- Claridad
- Bastante claro
- Estado de actividad
- Estancado
- Stack tecnológico
- cpp
- Área
- embedded-iot
Línea de trabajo
Empieza siguiendo el estado de la frecuencia cardíaca y el valor devuelto en src/components/heartrate/Ppg.cpp y src/heartratetask/HeartRateTask.cpp. Después inspecciona cómo src/displayapp/screens/WatchFaceCasioStyleG7710.cpp utiliza el estado del tracker y enoughData. Se considera terminado cuando la esfera del reloj ya no muestra un cero incorrecto mientras los datos no están disponibles y proporciona un indicador adecuado del estado no disponible.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Verification
- I searched for similar issues (including closed issues) and found none was relevant.
Introduce the issue
In the source code it appears that 0 bpm is being used as part of the state machine for the heart rate tracker, and also an error value. However this ends up translating into the value returned and displayed on screen (at least of 1.15).
See:
https://github.com/InfiniTimeOrg/InfiniTime/blob/075e02a33e7a362197f279edc015b596c4f9cd4e/src/components/heartrate/Ppg.cpp#L156
and:
https://github.com/InfiniTimeOrg/InfiniTime/blob/075e02a33e7a362197f279edc015b596c4f9cd4e/src/heartratetask/HeartRateTask.cpp#L201
The code should probably be reworked such that 0 is a valid state (although ideally NONE of us should see such a number for real). Also a visual indicator that the heart rate tracker "needs to collect more information" would also make sense. Maybe a pulsing heart icon, piece of text like dots . .. ... etc.
This likely means watch face code should be able to check enoughData in addition to whether the tracker is running. https://github.com/InfiniTimeOrg/InfiniTime/blob/075e02a33e7a362197f279edc015b596c4f9cd4e/src/displayapp/screens/WatchFaceCasioStyleG7710.cpp#L295
Preferred solution
No response
Version
No response
- Lenguaje dominante
- C
- Estrellas
- 3.4k
- Forks
- 1.1k
- Merge medio
- 23 h 18 min
- PR fusionados (30 d)
- 1
Preparar el entorno
Inicia el contenedor de desarrollo del proyecto en tu navegador, con tu propia cuenta de GitHub.
- Sin Dockerfile ni archivo de Docker Compose
- Sin plantilla de pull request
- Leer la guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de InfiniTimeOrg/InfiniTime
-
BLE handlers use length fields from the packet without checking how much data arrivedPosiblemente ocupada @DustinHab la tomó hace 37 días. Abierto
Dificultad 4/5 3-5 días Aptitud para principiantes 56/100
InfiniTimeOrg/InfiniTime#2485 ·
-
Unexpected jump back to watch face when button pressedPosiblemente ocupada @SteveAmor la tomó hace 80 días. Abiertobug
Dificultad 4/5 3-5 días Aptitud para principiantes 50/100
InfiniTimeOrg/InfiniTime#2461 · 2 comentarios ·
-
Screen doesn't go to sleep after reset countdown appPosiblemente ocupada @SteveAmor la tomó hace 78 días. Abiertobug
Dificultad 4/5 3-5 días Aptitud para principiantes 45/100
InfiniTimeOrg/InfiniTime#2419 · 4 comentarios ·
-
Dificultad 4/5 3-5 días Aptitud para principiantes 25/100
InfiniTimeOrg/InfiniTime#2410 ·
-
Dificultad 4/5 3-5 días Aptitud para principiantes 45/100
InfiniTimeOrg/InfiniTime#2407 · 4 comentarios ·
Todos los issues de InfiniTimeOrg/InfiniTime
Issues similares
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 88/100
quickjs-ng/quickjs#1836 ·
Los mantenedores suelen responder en 2 días
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 65/100
open-quantum-safe/liboqs#2637 ·
Los mantenedores suelen responder en 1 día
-
Go device SDK: Whisper transcriber keeps buffering and transcribing audio appended after Stop()Posiblemente ocupada @apoorvdarshan la tomó hoy. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 84/100
BasedHardware/omi#22770 ·
Los mantenedores suelen responder en 1 día
-
fido_do_ecdh() reports any getKeyAgreement failure as FIDO_ERR_INTERNALPosiblemente ocupada @brlumen la tomó hoy. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 74/100
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
joncampbell123/dosbox-x#6698 ·
Los mantenedores suelen responder en 1 día