Harmonize `cam_id` vs `camera_ip` naming in pyro_camera_api
Los mantenedores suelen responder en 1 día
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 88/100
- Tipo de issue
- Refactorización
- Claridad
- Bien especificado
- Estado de actividad
- Tranquilo
- Stack tecnológico
- python
- Área
- backend-api-design
Línea de trabajo
Revisa los usos de cam_id en pyro_camera_api/main.py, api/routes_stream.py y api/routes_cameras.py. Compáralos con las variables de bucle camera_ip en camera/patrol.py y camera/stuck_detector.py, y luego ejecuta las pruebas o comprobaciones pertinentes. La tarea está terminada cuando se hayan renombrado los nombres engañosos de las variables de bucle sin cambiar el comportamiento.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
This issue is part of the epic pyronear/roadmap#12 (Camera support).
Several call sites in pyro_camera_api use cam_id as a loop variable when iterating dicts keyed by camera IP (CAMERA_REGISTRY, PATROL_FLAGS, STUCK_CHECK_FLAGS, stream_workers, stream_processes, RAW_CONFIG, ...). This is misleading: cam_id is used elsewhere in the project for the camera-pose identifier ("{ip}_{p}"), not for plain IPs.
The loop functions in camera/patrol.py and camera/stuck_detector.py already use camera_ip correctly — the call sites should match.
Files to update: pyro_camera_api/main.py, api/routes_stream.py, api/routes_cameras.py.
Pure rename, no behavior change. Spotted while reviewing #360.
Also (merged from #408): api/routes_control.py is 1100+ lines. Split it by concern (moves, presets, zoom, azimuth, calibration) while doing the rename.
- Lenguaje dominante
- Python
- Estrellas
- 20
- Forks
- 9
- Merge medio
- 1 d 12 h
- PR fusionados (30 d)
- 9
Preparar el entorno
- Incluye un Dockerfile o un 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 pyronear/pyro-engine
-
module: engine type: enhancement
Dificultad 5/5 Más de una semana Aptitud para principiantes 35/100
pyronear/pyro-engine#467 ·
Los mantenedores suelen responder en 1 día
-
module: vision type: enhancement
Dificultad 5/5 Más de una semana Aptitud para principiantes 25/100
pyronear/pyro-engine#466 ·
Los mantenedores suelen responder en 1 día
-
module: vision type: enhancement
Dificultad 5/5 Más de una semana Aptitud para principiantes 45/100
pyronear/pyro-engine#465 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 55/100
pyronear/pyro-engine#463 ·
Los mantenedores suelen responder en 1 día
-
topic: documentation
Dificultad 5/5 Más de una semana Aptitud para principiantes 35/100
pyronear/pyro-engine#455 ·
Los mantenedores suelen responder en 1 día
Todos los issues de pyronear/pyro-engine
Issues similares
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 79/100
vllm-project/vllm-omni#10051 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
Bug Testing
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
SCIInstitute/SCIRun#2931 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
Los mantenedores suelen responder en 5 días
-
Author Page: barom-kangAbiertoauthor-page correction
Dificultad 2/5 1-3 horas Aptitud para principiantes 64/100
acl-org/acl-anthology#10827 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 91/100
marimo-team/marimo#11768 ·
Los mantenedores suelen responder en 1 día