Skip to content

Fix all missing optional import statements from pipeline folders#4272

Merged
patrickvonplaten merged 3 commits into
mainfrom
fix_next_circular_import_error
Jul 25, 2023
Merged

Fix all missing optional import statements from pipeline folders#4272
patrickvonplaten merged 3 commits into
mainfrom
fix_next_circular_import_error

Conversation

@patrickvonplaten

@patrickvonplaten patrickvonplaten commented Jul 25, 2023

Copy link
Copy Markdown
Contributor

Fixes: #4265

@HuggingFaceDocBuilderDev

HuggingFaceDocBuilderDev commented Jul 25, 2023

Copy link
Copy Markdown

The documentation is not available anymore as the PR was closed or merged.

@patrickvonplaten patrickvonplaten changed the title Add optional import for SDXL pipelines Fix all missing optional import statements from pipeline folders Jul 25, 2023
@patrickvonplaten

Copy link
Copy Markdown
Contributor Author

@sayakpaul in a follow-up PR it would be great if we could extend the dependency tester to also test that every import from a pipeline folder works, e.g.:

from diffusers.pipelines.{all_folders} import respective class

@patrickvonplaten patrickvonplaten merged commit b37dc3b into main Jul 25, 2023
@patrickvonplaten patrickvonplaten deleted the fix_next_circular_import_error branch July 25, 2023 23:46
orpatashnik pushed a commit to orpatashnik/diffusers that referenced this pull request Aug 1, 2023
…gingface#4272)

* fix circular import

* fix imports when watermark not specified

* fix all pipelines
orpatashnik pushed a commit to orpatashnik/diffusers that referenced this pull request Aug 1, 2023
…gingface#4272)

* fix circular import

* fix imports when watermark not specified

* fix all pipelines
orpatashnik pushed a commit to orpatashnik/diffusers that referenced this pull request Aug 1, 2023
…gingface#4272)

* fix circular import

* fix imports when watermark not specified

* fix all pipelines
yoonseokjin pushed a commit to yoonseokjin/diffusers that referenced this pull request Dec 25, 2023
…gingface#4272)

* fix circular import

* fix imports when watermark not specified

* fix all pipelines
AmericanPresidentJimmyCarter pushed a commit to AmericanPresidentJimmyCarter/diffusers that referenced this pull request Apr 26, 2024
…gingface#4272)

* fix circular import

* fix imports when watermark not specified

* fix all pipelines
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.

ImportError: cannot import name 'StableDiffusionXLControlNetPipeline' from 'diffusers.pipelines.controlnet'

2 participants