Skip to content

Integration of Qwen3-VL OpenVINO + GenAI fixes#4038

Open
dkalinowski wants to merge 1 commit intomainfrom
qwen3-vl-integration
Open

Integration of Qwen3-VL OpenVINO + GenAI fixes#4038
dkalinowski wants to merge 1 commit intomainfrom
qwen3-vl-integration

Conversation

Copilot AI review requested due to automatic review settings March 4, 2026 12:42
Copy link
Contributor

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 updates OVMS build dependency pins intended to incorporate Qwen3-VL-related fixes by changing the OpenVINO and OpenVINO GenAI source revisions used during Docker builds.

Changes:

  • Bump OV_SOURCE_BRANCH (OpenVINO) and OV_GENAI_BRANCH (OpenVINO GenAI) commit SHAs used for source builds.
  • Change default GitHub orgs used to clone OpenVINO and OpenVINO GenAI during Docker builds.

Comment on lines +82 to +83
OV_SOURCE_ORG ?= CuriousPanCake
OV_GENAI_ORG ?= yatarkan
Copy link

Copilot AI Mar 4, 2026

Choose a reason for hiding this comment

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

Changing the default OpenVINO/OpenVINO GenAI clone orgs to personal forks makes the Docker build pull source code from non-official repositories by default, which is a supply-chain risk and can break reproducibility/CI if those forks disappear or don’t contain the pinned commits. Keep defaults as openvinotoolkit and rely on make OV_SOURCE_ORG=... OV_GENAI_ORG=... overrides (or a separate developer-only config) when forks are needed.

Suggested change
OV_SOURCE_ORG ?= CuriousPanCake
OV_GENAI_ORG ?= yatarkan
OV_SOURCE_ORG ?= openvinotoolkit
OV_GENAI_ORG ?= openvinotoolkit

Copilot uses AI. Check for mistakes.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants