Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

AddUse 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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Open
apolinario wants to merge2 commits intomain
base:main
Choose a base branch
Loading
fromfix-top-media-models

Conversation

apolinario
Copy link
Contributor

@@ -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")) {
Copy link
ContributorAuthor

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);
Copy link
ContributorAuthor

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

@apolinario
Copy link
ContributorAuthor

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

Copy link
Member

@Vaibhavs10Vaibhavs10 left a 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

@apolinario
Copy link
ContributorAuthor

Sounds good. Do you think we should keep the ones that do use diffusers but require extra libraries to function?

@apolinario
Copy link
ContributorAuthor

For now let's merge this with the simplest: just added to the ones that don't require installing any auxiliary libraries for now

@apolinarioapolinario marked this pull request as ready for reviewJuly 18, 2025 14:39
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment
Reviewers

@Vaibhavs10Vaibhavs10Vaibhavs10 left review comments

@SBrandeisSBrandeisAwaiting requested review from SBrandeisSBrandeis is a code owner

@gary149gary149Awaiting requested review from gary149gary149 is a code owner

@WauplinWauplinAwaiting requested review from WauplinWauplin is a code owner

@julien-cjulien-cAwaiting requested review from julien-cjulien-c is a code owner

@pcuencapcuencaAwaiting requested review from pcuencapcuenca is a code owner

@ngxsonngxsonAwaiting requested review from ngxsonngxson is a code owner

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

2 participants
@apolinario@Vaibhavs10

[8]ページ先頭

©2009-2025 Movatter.jp