diffusers
when an InputParam is both required and has a default: set required = False and warn
#14521
Merged

when an InputParam is both required and has a default: set required = False and warn #14521

yiyixuxu merged 10 commits into main from required-input-no-default
yiyixuxu
yiyixuxu yiyixuxu marked this pull request as draft 78 days ago
github-actions github-actions added modular-pipelines
github-actions github-actions added size/S
HuggingFaceDocBuilderDev
yiyixuxu yiyixuxu marked this pull request as ready for review 13 days ago
yiyixuxu drop the default of a required InputParam, with a warning
3be086bc
yiyixuxu rebase follow-ups: new qwenimage21 site, keep MiniMax-H3 steps requir…
a73e9678
yiyixuxu yiyixuxu force pushed from 9cc14db6 to a73e9678 12 days ago
github-actions github-actions added tests
github-actions github-actions added fixes-issue
github-actions github-actions removed size/S
github-actions github-actions added size/M
yiyixuxu regenerate auto docstrings
f5c8ae5f
yiyixuxu reword the required-plus-default gotcha
5142760f
yiyixuxu drop the InputParam unit test
949e55ff
github-actions github-actions removed tests
yiyixuxu require num_inference_steps in blocks that run after set_timesteps
53ac3af3
github-actions github-actions added tests
yiyixuxu flip the drop the required
c95198ea
chessing234
chessing234 requested changes on 2026-10-23
yiyixuxu yiyixuxu changed the title drop the default of a required InputParam, with a warning when an InputParam is both required and has a default: set required = False and warn 12 days ago
yiyixuxu fix numbering
2bca3434
yiyixuxu yiyixuxu requested a review from sayakpaul sayakpaul 12 days ago
sayakpaul sayakpaul added this to the Release 0.43.0 milestone 9 days ago
sayakpaul
sayakpaul approved these changes on 2026-10-26
yiyixuxu [agent docs] call expected_workflow_defaults a dict, not a table
e4e81e1f
yiyixuxu [modular tests] every pipeline builds without a required-with-default…
bcc2f90c
yiyixuxu yiyixuxu merged cda991a6 into main 8 days ago
yiyixuxu yiyixuxu deleted the required-input-no-default branch 8 days ago

Login to write a write a comment.

Login via GitHub

Assignees
No one assigned
Labels
Milestone