huggingface / huggingface/diffusers

[Q] Possibly unused `self.final_alpha_cumprod`

Aperta
#9,395 7 commenti 0 reazioni 0 assegnatari Vedi su GitHub
stale
Lingua principale
Python
Stelle
34.5k
Fork
7.3k
Merge medio
3g 3h
PR unite (30g)
91

Descrizione

Hello team, quick question to make sure I understand the behavior of the `step` function in LCM Scheduler.

https://github.com/huggingface/diffusers/blob/a7361dccdc581147620bbd74a6d295cd92daf616/src/diffusers/schedulers/scheduling_lcm.py#L534-L543

Here, it seems that the condition `prev_timestep >= 0` is always `True`, because `timestep` and `self.timesteps[prev_step_index]` cannot be negative. This would mean that `self.final_alpha_cumprod` is never used. Is there a way in which `prev_timestep` can be negative?

Guida per i contributori

Apri la guida per i contributori

Direzione di ricerca

Leggi src/diffusers/schedulers/scheduling_lcm.py alle righe collegate della funzione di step, quindi traccia come timestep e prev_step_index determinano prev_timestep. Verifica se prev_timestep può essere negativo e se self.final_alpha_cumprod è raggiungibile; il lavoro è completato quando la conclusione è stata registrata e sono state identificate eventuali attività di pulizia o documentazione necessarie.

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

Valutazione

Stack tecnologico
python
Ambito
machine-learning
Tipo di issue
Refactoring
Difficoltà
3/5
Tempo stimato
1-2 giorni
Stato di attività
Ferma
Chiarezza
Da chiarire
Idoneità per principianti
35/100

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.