-
Notifications
You must be signed in to change notification settings - Fork 467
Add Use this model
snippets for top diffusers models
#1642
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
@@ -449,6 +726,8 @@ export const diffusers = (model: ModelData): string[] => { | |||
} | |||
} else if (model.tags.includes("textual_inversion")) { | |||
return diffusers_textual_inversion(model); | |||
} else if (model.tags.includes("FluxFillPipeline")) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Example that doesn't need extra libraries to install
model.tags.includes("StableDiffusionInpaintPipeline") || | ||
model.tags.includes("StableDiffusionXLInpaintPipeline") | ||
) { | ||
return diffusers_inpainting(model); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Example that doesn't need extra libraries to install
Still a draft as we are unsure whether the super custom ones (all of those that need the installation of a custom library) should stay in the PR or not |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
maybe let's split this into two:
diffusers
only first and the rest (3d model) seperate or can be a follow-up
Sounds good. Do you think we should keep the ones that do use diffusers but require extra libraries to function? |
For now let's merge this with the simplest: just added to the ones that don't require installing any auxiliary libraries for now |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Okay with the diffusers snippets, let's remove the Hunyuan3D snippet from this PR and we can do that in a follow-up too.
cc: @Wauplin if you have any strong feelings about the diffusers snippets.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Approving diffusers snippets, LGTM :) (only from high-level perspective, I trust you on the details)
Slack: https://huggingface.slack.com/archives/C04SZ8E2Q6S/p1752664171386139