Skip to content

[WIP][modular] update required_inputs, adding override_attribute etc - #15136

Open
yiyixuxu wants to merge 1 commit into
conditional-runtime-defaultsfrom
sequential-required-inputs
Open

yiyixuxu wants to merge 1 commit into
conditional-runtime-defaultsfrom
sequential-required-inputs

Conversation

@yiyixuxu

@yiyixuxu yiyixuxu commented Oct 27, 2026 •

Copy link
Copy Markdown
Collaborator

fix #15009
related #15133

…walk

Sequential and loop blocks decide each input's `required` flag and default in one
ordered walk over their sub-blocks; `required_inputs` is derived from `inputs` as
in conditional blocks, and the two overrides are gone. Composites declare their
own `InputParam`/`OutputParam`/`ComponentSpec`/`ConfigSpec` for a derived entry
with `*_overrides`, and get a FutureWarning until they settle a default two
sub-blocks disagree on, a conditional block's per-branch defaults included.
Leaf blocks declare what they write back, docstrings regenerated.

Fixes #15009.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
@github-actions github-actions Bot added documentation Improvements or additions to documentation tests modular-pipelines size/L PR with diff > 200 LOC labels Oct 27, 2026
@HuggingFaceDocBuilderDev

Copy link
Copy Markdown

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation modular-pipelines size/L PR with diff > 200 LOC tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants