Sensor ack packets for DisablePiezo/GetHeaterOffset/ClearAlarm (0xA9/0xAA/0xAD) are not decoded
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 3/5
- Tiempo estimado
- 1-2 días
- Aptitud para principiantes
- 45/100
- Tipo de issue
- Error
- Claridad
- Bastante claro
- Estado de actividad
- Tranquilo
- Stack tecnológico
- rust
- Área
- backend, embedded-iot
Línea de trabajo
Read crates/pod-proto/src/sensor/command.rs and the SensorPacket::parse match in crates/pod-proto/src/sensor/packet.rs to understand the command and packet types. Check how other sensor acknowledgements are decoded, then run the relevant codec or packet tests if available. Done means acknowledgements for 0xA9 and 0xAA decode instead of reaching Unexpected; confirm ClearAlarm's opcode and payload before including it.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
The three // TODO add resp packet + 0x80 markers in crates/pod-proto/src/sensor/command.rs:21,28,32 sit on DisablePiezo (0x29), GetHeaterOffset (0x2A), and ClearAlarm (0x2D). SensorPacket::parse (crates/pod-proto/src/sensor/packet.rs:135-160) has no arm for their acknowledgements (0xA9/0xAA/0xAD), so sending any of these would make the reply fall to the _ => Unexpected branch, which the codec logs as an error (codec.rs:85). All three commands are currently #[allow(dead_code)] and unused, so this only bites once they're wired up. ClearAlarm is additionally flagged UNVERIFIED / 'seems to be crashing the mcu' (command.rs:29), so its opcode/payload needs confirming before use.
Filed from a repo-wide code audit (working tree at bcad746).
- Lenguaje dominante
- Rust
- Estrellas
- 0
- Forks
- 0
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Preparar el entorno
Este proyecto no incluye contenedor de desarrollo, Dockerfile ni guía de contribución, así que la configuración corre por tu cuenta: empieza por su README y consulta nuestra guía para la primera contribución para los pasos generales.
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 GoogleBot42/podd
-
documentation
Dificultad 1/5 Menos de una hora Aptitud para principiantes 84/100
GoogleBot42/podd#63 ·
-
documentation
Dificultad 1/5 1-3 horas Aptitud para principiantes 78/100
GoogleBot42/podd#62 ·
-
documentation good first issue
Dificultad 2/5 1-3 horas Aptitud para principiantes 69/100
GoogleBot42/podd#60 ·
-
documentation
Dificultad 1/5 Menos de una hora Aptitud para principiantes 85/100
GoogleBot42/podd#61 ·
-
enhancement
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
GoogleBot42/podd#58 ·
Todos los issues de GoogleBot42/podd
Issues similares
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
rubys/roundhouse#1917 ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
Los mantenedores suelen responder en 2 días
-
good first issue
Dificultad 2/5 1-3 horas Aptitud para principiantes 88/100
Los mantenedores suelen responder en 1 día
-
[cosmic-applet-battery] Division by zero displays NaN% when UPower reports max_brightness = 0Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 84/100
pop-os/cosmic-applets#1619 ·
Los mantenedores suelen responder en 3 días
-
binding:python bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
Los mantenedores suelen responder en 1 día