Clarify set_adapters() adapter_weights documentation and type hints
Mantenedores costumam responder em até 1 dia
Avaliação
- Dificuldade
- 2/5
- Tempo estimado
- 1-3 horas
- Facilidade para iniciantes
- 68/100
- Tipo de issue
- Documentação
- Clareza
- Claramente especificada
- Status de atividade
- Ativa
- Domínio
- machine-learning
Direção de pesquisa
Leia LoraBaseMixin.set_adapters() e sua docstring e anotação de tipo. A issue diz que o método aceita pesos escalares, do tipo dicionário, lista e None, incluindo pesos por adaptador e pesos de componentes aninhados; confira as formas aceitas em relação à implementação existente. O trabalho estará concluído quando a documentação explicar essas formas e a anotação for mais específica sem alterar o comportamento.
Escrita pelo modelo de indexação a partir do texto da issue.
Descrição
Follow-up to #15128.
During the review of #15128, @BenjaminBossan suggested improving the adapter_weights docstring and type annotation in LoraBaseMixin.set_adapters(), and @sayakpaul agreed. The PR was merged before I could include these changes. On current main, the docstring still describes adapter_weights as list[float, float], but the method also accepts scalars, dictionaries, lists of weights, and None. The type annotation also uses dict without specifying the key and value types.
A small follow-up could:
- Explain the supported weight formats and how the same component-weight dictionary is used for multiple adapters.
- Explain per-adapter and nested component weights where supported.
- Make the type annotation more specific without changing how the method works.
I already wrote and tested these changes while working on #15128. Would a small follow-up PR for these changes be appropriate?
- Linguagem predominante
- Python
- Estrelas
- 34.9k
- Forks
- 7.5k
- Merge médio
- 2d 13h
- PRs com merge (30d)
- 64
Preparar o ambiente
- Sem Dockerfile nem arquivo Docker Compose
- Tem um modelo de pull request
- Ler o guia de contribuição
Primeiros passos
- Leia a issue inteira e depois o guia de contribuição do projeto.
- Comente na issue dizendo que vai assumir — evita que duas pessoas façam o mesmo trabalho.
- Faça um fork do repositório e trabalhe em uma branch.
- Abra um pull request que referencie o número da issue.
Mais de huggingface/diffusers
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 72/100
huggingface/diffusers#15211 · 3 comentários ·
Mantenedores costumam responder em até 1 dia
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 82/100
huggingface/diffusers#15210 ·
Mantenedores costumam responder em até 1 dia
-
UniPCMultistepScheduler.set_timesteps(sigmas=...) raises TypeError for a list when use_flow_sigmas=TrueTalvez já em andamento @marlonferreira1800-hue assumiu há 2 dias. Abertabug
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 85/100
huggingface/diffusers#15207 ·
Mantenedores costumam responder em até 1 dia
-
Dificuldade 1/5 Menos de uma hora Facilidade para iniciantes 78/100
huggingface/diffusers#15170 · 1 comentário ·
Mantenedores costumam responder em até 1 dia
-
bug
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 70/100
huggingface/diffusers#15163 · 3 comentários ·
Mantenedores costumam responder em até 1 dia
Todas as issues de huggingface/diffusers
Issues semelhantes
-
Dificuldade 1/5 Menos de uma hora Facilidade para iniciantes 75/100
SACGF/variantgrid#2791 ·
Mantenedores costumam responder em até 1 dia
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 66/100
Mantenedores costumam responder em até 1 dia
-
bot-triaged oncall: cpu inductor
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 73/100
Mantenedores costumam responder em até 1 dia
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 82/100
Mantenedores costumam responder em até 1 dia
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 76/100
kubescape/cel-admission-library#141 · 1 comentário ·
Mantenedores costumam responder em até 1 dia