Refactor: Split up sensor.c
Evaluación
- Dificultad
- 5/5
- Tiempo estimado
- Más de una semana
- Aptitud para principiantes
- 15/100
- Tipo de issue
- Refactorización
- Claridad
- Necesita aclaración
- Estado de actividad
- Activo
- Stack tecnológico
- c
- Área
- embedded-iot
Línea de trabajo
Start by reading sensor.c alongside its header and listing every exported function, then compare that list against the driver and app-layer modules named in the issue. The header is flagged as the main complication, so decide which declarations move with their implementation before editing. Done means sensor.c no longer lives in mod, its functions sit in driver or the app layer, and the build still links; rev 3 platform-specific sensor functions should then be possible.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Version: v2.8.0 && v3.0.0
Describe the feature you'd like
The sensor module in its current form has no place living in mod based on our idea of what the library should be.
Many of these functions belong in driver, and others should live in the app layer.
Some subject matter expert should make an effort to move functions to their truly correct locations. The header is going to be the biggest complication, I expect, but that's not a huge deal.
This will also allow us to implement platform-specific versions of sensor functions on rev 3.
Describe alternatives you've considered
Leaving as-is.
Additional context
N/A
Points of Contact
Mod maintainer for v2.8.0 and beyond: @NArmistead
Additional SME: @ETSells
- Lenguaje dominante
- C
- Estrellas
- 4
- Forks
- 2
- Merge medio
- 5 d 2 h
- PR fusionados (30 d)
- 6
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Tiene una plantilla de pull request
- Sin 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 SunDevilRocketry/mod
-
Sensor: Magnetometer calibrationPosiblemente ocupada @NArmistead la tomó hace 27 días. Abiertoenhancement
Dificultad 5/5 Más de una semana Aptitud para principiantes 20/100
SunDevilRocketry/mod#163 · 1 comentario · 1 asignado ·
-
Telemetry: Create ICDPosiblemente ocupada @ETSells la tomó hace 26 días. Abiertoadvanced documentation
Dificultad 5/5 Más de una semana Aptitud para principiantes 15/100
SunDevilRocketry/mod#161 · 1 asignado ·
-
Mod: Make library platform independentPosiblemente ocupada @ETSells la tomó hace 25 días. Abiertoadvanced refactor
Dificultad 4/5 3-5 días Aptitud para principiantes 35/100
SunDevilRocketry/mod#160 · 1 comentario · 1 asignado ·
-
Sensor: Rename and add LoRa to sensor mutexQuizá libre de nuevo @ferociousPandaPickle la tomó hace 35 días y no hay ningún pull request abierto. Abiertoenhancement good first issue
Dificultad 3/5 1-2 días Aptitud para principiantes 64/100
SunDevilRocketry/mod#153 · 1 comentario · 1 asignado ·
-
enhancement
Dificultad 5/5 Más de una semana Aptitud para principiantes 30/100
SunDevilRocketry/mod#136 ·
Todos los issues de SunDevilRocketry/mod
Issues similares
-
examples/dnsdoctor: cleanup.sh stops at the first missing piece of a setup.sh that did not finishAbiertoseverity: low
Dificultad 2/5 1-3 horas Aptitud para principiantes 70/100
luainkernel/lunatik#2263 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 Medio día Aptitud para principiantes 72/100
vercel-labs/scriptc#2910 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
darktable-org/darktable#22939 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 82/100
espressif/idf-extra-components#903 ·
Los mantenedores suelen responder en 1 día
-
os: input_password leaves terminal echo disabled when reading failsPosiblemente ocupada @davlgd la tomó hoy. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
Los mantenedores suelen responder en 1 día