Skip to content

Conversation

@openvino-dev-samples
Copy link
Collaborator

CVS-173999

@review-notebook-app
Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@openvino-dev-samples openvino-dev-samples requested review from aleksandr-mokrov, Copilot and sbalandi and removed request for sbalandi December 2, 2025 06:07
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR adds a new notebook demonstrating text-to-image generation using the Qwen-Image model with OpenVINO. The notebook includes model conversion, optimization, and an interactive Gradio interface.

Key changes:

  • Added Gradio helper for creating an interactive demo with prompt enhancement and multi-language support
  • Added README documenting the notebook's purpose and contents
  • Excluded the notebook from Docker-based CI testing

Reviewed changes

Copilot reviewed 3 out of 4 changed files in this pull request and generated 4 comments.

File Description
notebooks/qwen-image/gradio_helper.py Implements Gradio UI with image generation logic, prompt enhancement for English/Chinese, and configurable parameters
notebooks/qwen-image/README.md Documents the Qwen-Image notebook tutorial and installation instructions
.ci/ignore_treon_docker.txt Adds qwen-image notebook to CI exclusion list

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment on lines +32 to +34

return polish_prompt_en(input_prompt)

Copy link

Copilot AI Dec 2, 2025

Choose a reason for hiding this comment

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

The function doesn't handle the case when language detection returns neither 'zh' nor 'en', resulting in an implicit None return. Add a default case: after line 33, add else: return original_prompt or change line 31 to just else: to handle all non-Chinese prompts.

Suggested change
return polish_prompt_en(input_prompt)
return polish_prompt_en(input_prompt)
else:
return input_prompt

Copilot uses AI. Check for mistakes.
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.

1 participant