Harmonize `cam_id` vs `camera_ip` naming in pyro_camera_api
I maintainer di solito rispondono entro 2 giorni
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 2/5
- Tempo stimato
- 1-3 ore
- Idoneità per principianti
- 88/100
- Tipo di issue
- Refactoring
- Chiarezza
- Specificata chiaramente
- Stato di attività
- Tranquilla
- Stack tecnologico
- python
- Ambito
- backend-api-design
Direzione di ricerca
Esamina gli utilizzi di cam_id in pyro_camera_api/main.py, api/routes_stream.py e api/routes_cameras.py. Confrontali con le variabili di ciclo camera_ip in camera/patrol.py e camera/stuck_detector.py, quindi esegui i test o i controlli pertinenti. L'attività è completata quando i nomi fuorvianti delle variabili di ciclo sono stati rinominati senza modificare il comportamento.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
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.
- Lingua principale
- Python
- Stelle
- 20
- Fork
- 9
- Merge medio
- 7g 19h
- PR unite (30g)
- 9
Preparare l'ambiente
- Include un Dockerfile o un file Docker Compose
- Nessun modello di pull request
- Leggi la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di pyronear/pyro-engine
-
module: engine type: enhancement
Difficoltà 5/5 Più di una settimana Idoneità per principianti 35/100
pyronear/pyro-engine#467 ·
I maintainer di solito rispondono entro 2 giorni
-
module: vision type: enhancement
Difficoltà 5/5 Più di una settimana Idoneità per principianti 25/100
pyronear/pyro-engine#466 ·
I maintainer di solito rispondono entro 2 giorni
-
module: vision type: enhancement
Difficoltà 5/5 Più di una settimana Idoneità per principianti 45/100
pyronear/pyro-engine#465 ·
I maintainer di solito rispondono entro 2 giorni
-
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 55/100
pyronear/pyro-engine#463 ·
I maintainer di solito rispondono entro 2 giorni
-
topic: documentation
Difficoltà 5/5 Più di una settimana Idoneità per principianti 35/100
pyronear/pyro-engine#455 ·
I maintainer di solito rispondono entro 2 giorni
Tutte le issue di pyronear/pyro-engine
Issue simili
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
FinanceFlash/unvibecode#509 ·
-
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 82/100
FAIRmat-NFDI/nomad-distro-dev#161 ·
-
bug v2
Difficoltà 2/5 1-3 ore Idoneità per principianti 76/100
modelcontextprotocol/python-sdk#3821 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
-
bug severity:low
Difficoltà 2/5 1-3 ore Idoneità per principianti 68/100
I maintainer di solito rispondono entro 2 giorni
-
Log-odds calibration uses upper middle score instead of median for even candidate setsForse già presa @flysheep-ai l’ha presa oggi. Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 82/100
I maintainer di solito rispondono entro 1 giorno