huggingface / huggingface/diffusers

[core] refactor `attention_processor.py` the easy way

Aperta
#10,022 6 commenti 0 reazioni 1 assegnatario Rivendicata da @DN6 Vedi su GitHub
wip
Lingua principale
Python
Stelle
34.5k
Fork
7.3k
Merge medio
3g 3h
PR unite (30g)
91

Descrizione

With @DN6 we have been discussing an idea about breaking up `src/diffusers/models/attention_processor.py` as it's getting excruciatingly longer and longer. The idea is simple and won't very likely require multiple rounds of PRs.

* Create a module named `attention_processor`.
* Split the attention processor classes with respect to the model names they are used. This makes sense because pretty much most of the models (at least the widely used ones) have their own attention processors at this point. For example, `FluxAttnProcessor2_0`, `FluxAttnProcessor2_0_NPU`, `FusedFluxAttnProcessor2_0` could go to `attention_processor/flux_attention_processors.py`.
* Other attention processors (such as `AttnProcessor`, `AttnProcessor2_0`, etc.) which are shared across different models could live in a common file i.e., `attention_processor/common.py`.

Since `attention_prcoessor/` will have an init, I don't think there will be any breaking changes.

Guida per i contributori

Apri la guida per i contributori

Valutazione

Questa issue non è ancora stata valutata.

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.