[tests] refactor v-series pipeline tests by sayakpaul · Pull Request #14653 · huggingface/diffusers · GitHub
Skip to content

[tests] refactor v-series pipeline tests - #14653

Merged
sayakpaul merged 3 commits into
mainfrom
v-series-pipeline-tests
Aug 31, 2026
Merged

sayakpaul merged 3 commits into
mainfrom
v-series-pipeline-tests

Conversation

@sayakpaul

Copy link
Copy Markdown
Member

No description provided.

@github-actions github-actions Bot added size/L PR with diff > 200 LOC tests pipelines and removed size/L PR with diff > 200 LOC labels Aug 31, 2026
@HuggingFaceDocBuilderDev

Copy link
Copy Markdown

Comment on lines +382 to +384
# Offload all models. The inner pipelines free their own (empty) hooks, so the ones installed on this
# pipeline by `enable_model_cpu_offload` have to be freed here.
self.maybe_free_model_hooks()

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Alternatively, we can skip / xfail the corresponding tests.

output_type (`str`, *optional*, defaults to `"pil"`):
The output format of the generate image. Choose between
[PIL](https://pillow.readthedocs.io/en/stable/): `PIL.Image.Image` or `np.array`.
The output format of the generate image. Choose between `"pil"`

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Because we enforce the PyTorch output format in our tests, I think this is needed. But I don't think the changes are intrusive.

@sayakpaul

Copy link
Copy Markdown
Member Author

/diffusers-bot pytest tests/pipelines/visualcloze/

@github-actions

github-actions Bot commented Aug 31, 2026 •

Copy link
Copy Markdown
Contributor

@github-actions github-actions Bot added the size/L PR with diff > 200 LOC label Aug 31, 2026
@sayakpaul
sayakpaul merged commit f9495c7 into main Aug 31, 2026
15 of 16 checks passed
@sayakpaul
sayakpaul deleted the v-series-pipeline-tests branch August 31, 2026 16:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pipelines size/L PR with diff > 200 LOC tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants