Skip to content

Fix CogView3 lazy export to match the actual output class name#13951

Open
rumitvn wants to merge 1 commit into
huggingface:mainfrom
rumitvn:fix/cogview3-output-lazy-import
Open

Fix CogView3 lazy export to match the actual output class name#13951
rumitvn wants to merge 1 commit into
huggingface:mainfrom
rumitvn:fix/cogview3-output-lazy-import

Conversation

@rumitvn

@rumitvn rumitvn commented Jun 14, 2026

Copy link
Copy Markdown

The cogview3 package advertises CogView3PlusPipelineOutput in its lazy _import_structure, but the output class is defined as CogView3PipelineOutput (no "Plus") in pipeline_output.py. Because of the mismatch, from diffusers.pipelines.cogview3 import CogView3PlusPipelineOutput raises an ImportError.

This points the lazy export at the real class name. The pipeline module already imports CogView3PipelineOutput, and the API docs autodoc that same name, so this just brings the lazy export in line with both.

# before: raises ImportError
from diffusers.pipelines.cogview3 import CogView3PlusPipelineOutput
# after: resolves to the class defined in pipeline_output.py
from diffusers.pipelines.cogview3 import CogView3PipelineOutput

The neighbouring CogView3PlusPipeline export is the actual pipeline class name and is intentionally left as-is — only the output-class export had the stray "Plus".

Addresses Issue 1 of #13648.

The lazy _import_structure advertised CogView3PlusPipelineOutput, but
the class is defined as CogView3PipelineOutput in pipeline_output.py,
so importing the advertised symbol raised an ImportError.
@github-actions github-actions Bot added pipelines size/S PR with diff < 50 LOC labels Jun 14, 2026
@github-actions

Copy link
Copy Markdown
Contributor

Hi @rumitvn, thanks for the PR! It does not appear to link an issue it fixes. If this PR addresses an existing issue, please add a closing keyword (e.g. Fixes #1234) to the PR description so the issue is linked. See the contribution guide for more details. If this PR intentionally does not fix a tracked issue, a maintainer can add the no-issue-needed label to silence this reminder.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pipelines size/S PR with diff < 50 LOC

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant