Hacktoberfest 2026: le issue che i maintainer hanno segnato per ottobre, aperte e adatte ai principianti. Sfoglia le issue Hacktoberfest

-s3_fast with -r, -g or -b also runs the full step 3 with LED fits

Aperta Adatta ai principianti
  • Qualcosa non va?
#176 0 commenti 0 reazioni 0 assegnatari Vedi su GitHub

I maintainer di solito rispondono entro 1 giorno

Nessuno ha ancora preso questa issue.

Valutazione

Difficoltà
2/5
Tempo stimato
1-3 ore
Idoneità per principianti
84/100
Tipo di issue
Bug
Chiarezza
Specificata chiaramente
Stato di attività
Attiva
Stack tecnologico
python
Ambito
cli, testing

Direzione di ricerca

Leggi ledsa/core/parser_arguments_run.py intorno alle righe 60–75 e segui come le opzioni dei colori impostano step_3 e step_3_fast. Aggiungi test per verificare che -s3_fast -r esegua solo il passaggio rapido e che -r da solo continui a eseguire il passaggio 3; esegui la suite di test pertinente. Il lavoro è concluso quando entrambe le combinazioni di argomenti selezionano il passaggio previsto.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Descrizione

bug

Problem

python -m ledsa -s3_fast -r runs the full step 3, with the LED fit on every LED of every image, and then the fast step 3 again. -g and -b behave the same way. Only -s3_fast -rgb, the combination the README shows, runs the fast step alone.

Cause

parser_arguments_run.py, line 60:

if args.red or args.green or args.blue or args.rgb and not args.step_3_fast:
    args.step_3 = True

and binds tighter than or, so the condition reads red or green or blue or (rgb and not step_3_fast), and the not step_3_fast guard only applies to -rgb. With -r, step_3 is set, so the full step 3 runs (line 70) before the fast one (line 75).

Impact

The fast run overwrites the result files of the full run, so the values that end up in analysis/channel*/ are those of the fast step. What is lost is the runtime of the LED fits, by far the slowest part of step 3, and logfiles/warnings.log collects the warnings of failed fits that were never asked for.

Fix

Group the colour flags before the guard:

if (args.red or args.green or args.blue or args.rgb) and not args.step_3_fast:
    args.step_3 = True

and add a test that -s3_fast -r runs only the fast step 3, while -r alone still runs step 3.

Lingua principale
Python
Stelle
7
Fork
4
Merge medio
3h 55m
PR unite (30g)
32

Preparare l'ambiente

Questo progetto non fornisce container di sviluppo, Dockerfile né guida per i contributori, quindi l'ambiente è a tuo carico: parti dal suo README e consulta la nostra guida al primo contributo per i passaggi generali.

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Altre issue di FireDynamics/LEDSmokeAnalysis

Tutte le issue di FireDynamics/LEDSmokeAnalysis

Issue simili

Altre issue su Python

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.