Skip to content

[tests] refactor j series pipeline tests - #14632

Merged
sayakpaul merged 5 commits into
mainfrom
j-series-pipeline-tests
Aug 28, 2026
Merged

[tests] refactor j series pipeline tests#14632
sayakpaul merged 5 commits into
mainfrom
j-series-pipeline-tests

Conversation

@sayakpaul

Copy link
Copy Markdown
Member

No description provided.

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

if not return_dict:
return image
return (image,)

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.

Mimicking what the edit plus pipeline already does.

@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.

super().tearDown()
# Each sample is bound to its own set of reference images, so the pipeline generates exactly one image per
# prompt and does not expose `num_images_per_prompt`.
optional_input_params = BasePipelineTesterConfig.optional_input_params - {"num_images_per_prompt"}

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Nit. I would prefer that this is not a set operation, and just an explicitly defined set.

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.

Agreed. Propagated this to the rest of the bunch as well.

@sayakpaul
sayakpaul merged commit 9f7aee4 into main Aug 28, 2026
16 checks passed
@sayakpaul
sayakpaul deleted the j-series-pipeline-tests branch August 28, 2026 07:38
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