Skip to content

update OV to 22nd June build#4316

Open
dtrawins wants to merge 1 commit into
mainfrom
0622
Open

update OV to 22nd June build#4316
dtrawins wants to merge 1 commit into
mainfrom
0622

Conversation

@dtrawins

Copy link
Copy Markdown
Collaborator

🛠 Summary

🧪 Checklist

  • Unit tests added.
  • The documentation updated.
  • Change follows security best practices.
    ``

Copilot AI review requested due to automatic review settings June 22, 2026 23:17

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Updates OpenVINO / OpenVINO GenAI dependencies to the June 22 nightly build and aligns OVMS tokenization request parsing with the expected API behavior (ensuring max_length implies truncation).

Changes:

  • Bump OpenVINO / Tokenizers / GenAI source commit pins and nightly binary URLs to 2026.3.0.0.dev20260622.
  • Enforce OVMS tokenize API contract by setting truncation=true when max_length is provided.
  • Update demo export requirements and unit test expectations to match the new dependency/build behavior.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 1 comment.

File Description
versions.mk Updates OpenVINO/Tokenizers/GenAI commit pins and nightly package URLs to the 2026-06-22 build.
src/tokenize/tokenize_parser.cpp Sets truncation=true when parsing max_length for tokenize requests.
src/test/tokenize_parser_test.cpp Updates unit test to assert truncation is set when max_length is present.
demos/common/export_models/requirements.txt Bumps OpenVINO/OpenVINO-tokenizers nightly versions and the optimum-intel git pin for model export tooling.

Comment on lines 84 to +88
if (it->value.IsInt()) {
size_t max_length = it->value.GetInt();
request.parameters["max_length"] = max_length;
// Keep OVMS tokenize API contract: max_length implies truncation.
// request.parameters["truncation"] = true;
request.parameters["truncation"] = true;
@dtrawins dtrawins requested review from mzegla and przepeck June 23, 2026 08:27
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.

3 participants