Skip to content

Fix the Kandinsky pipeline example #3690

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

Closed
wants to merge 1 commit into from
Closed

Fix the Kandinsky pipeline example #3690

wants to merge 1 commit into from

Conversation

freespirit
Copy link
Contributor

  • correct model id
  • use the new KandinskyPriorPipelineOutput member names

- correct model id
- use the new KandinskyPriorPipelineOutput member names
@freespirit
Copy link
Contributor Author

freespirit commented Jun 6, 2023

I think the KandinskyPriorPipelineOutput members were changed in 32ea214
Also, I think there are other examples to be fixed, but I have used only this pipeline so far so I'm not sure how many are there (I guess in each Kandinsky-type pipeline)

edit:
I noticed the example in pipeline_kandinsky_prior.py is still referring to the old names so it would lead to error.
The rest of the examples usually destructure the Output pipeline_kandinsky_prior.py as well as pipeline_kandinsky_inpaint.py. However, they still use the old names so they may be updated as well.

edit2: I was too quick to open this PR. I should have opened an issue and discussed all potential changes. It's the first time I'm using the Kandinsky pipeline (and model) but I'm willing to help

@freespirit
Copy link
Contributor Author

Issue should be fixed with more details in #3695
Closing this one.

@freespirit freespirit closed this Jun 6, 2023
@freespirit freespirit deleted the patch-1 branch June 6, 2023 20:05
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.

1 participant