huggingface / huggingface/diffusers

[Q] Possibly unused `self.final_alpha_cumprod`

Offen
#9,395 7 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen
stale
Vorherrschende Sprache
Python
Sterne
34.5k
Forks
7.3k
Ø Merge
3 T. 3 Std.
Gemergte PRs (30 T.)
91

Beschreibung

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?

Beitragsleitfaden

Beitragsleitfaden öffnen

Rechercherichtung

Lies src/diffusers/schedulers/scheduling_lcm.py an den verlinkten Zeilen der Schritt-Funktion und verfolge anschließend, wie timestep und prev_step_index prev_timestep bestimmen. Bestätige, ob prev_timestep negativ sein kann und ob self.final_alpha_cumprod erreichbar ist; abgeschlossen ist die Aufgabe, wenn die Schlussfolgerung festgehalten und jeglicher notwendige Bereinigungs- oder Dokumentationsbedarf identifiziert wurde.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
python
Bereich
machine-learning
Issue-Typ
Refactoring
Schwierigkeit
3/5
Geschätzter Aufwand
1-2 Tage
Aktivitätsstatus
Veraltet
Klarheit
Muss geklärt werden
Anfängerfreundlichkeit
35/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.