Skip to content

[modular]Update model card to include workflow#13195

Open
yiyixuxu wants to merge 2 commits intomainfrom
update-model-card
Open

[modular]Update model card to include workflow#13195
yiyixuxu wants to merge 2 commits intomainfrom
update-model-card

Conversation

@yiyixuxu
Copy link
Collaborator

before https://huggingface.co/YiYiXu/modular-loader-t2i-0704
after https://huggingface.co/YiYiXu/modular-loader-t2i-20260227

the input/output isn't too meaningful for conditional/auto pipeline so it is better we show them for each workflow

yiyi@huggingface.co added 2 commits February 27, 2026 11:08
@yiyixuxu yiyixuxu requested a review from sayakpaul February 27, 2026 11:11
@yiyixuxu
Copy link
Collaborator Author

cc @sayakpaul
I wanted to include workflow into the modelcard now that we support them
can you take a look here? feel free to push directly into this PR if you see things want to change

@HuggingFaceDocBuilderDev

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.

Copy link
Member

@sayakpaul sayakpaul left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you! Just one comment, let me know what you think.

Comment on lines +1172 to +1175
parts.append(f"<details>\n<summary><strong>{wf_name}</strong></summary>\n")
parts.append(f"> **Trigger inputs**: {', '.join(f'`{t}`' for t in trigger_input_names)}\n")
parts.append("*Could not resolve workflow blocks.*\n")
parts.append("</details>\n")
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do we need both?

Here, we already have:

text2image: requires prompt

Then having a collapsible block (essentially with the same info)

Image

seems to be redundant to me.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants