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

[tests] refactor v-series pipeline tests#14653
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

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.

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

pytest tests/pipelines/visualcloze/ passed on GPU — view logs.

@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