Skip to content

Commit a901d1c

Browse files
dkalinowskiprzepeck
authored andcommitted
OpenVINO/GenAI 12-08-2025 (#3569)
- Drop support for build tools 19, move to 22 (because xgrammar doesnt build with buildtools 19 - Patch nsync (dep of TF) to build on 22
1 parent 98bcee3 commit a901d1c

File tree

13 files changed

+66
-75
lines changed

13 files changed

+66
-75
lines changed

Makefile

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -74,9 +74,9 @@ FUZZER_BUILD ?= 0
7474
# NOTE: when changing any value below, you'll need to adjust WORKSPACE file by hand:
7575
# - uncomment source build section, comment binary section
7676
# - adjust binary version path - version variable is not passed to WORKSPACE file!
77-
OV_SOURCE_BRANCH ?= cc73719a2c667106c454f3abe663e312832365f1 # master 2025-07-29
77+
OV_SOURCE_BRANCH ?= c619ac6a5960d3edddcebcfc9b80d9f112d76b4f # master 2025-08-09
7878
OV_CONTRIB_BRANCH ?= c39462ca8d7c550266dc70cdbfbe4fc8c5be0677 # master / 2024-10-31
79-
OV_TOKENIZERS_BRANCH ?= d9a2bb05e750f1c628ada5ed81457f880719fd73 # master 2025-07-25
79+
OV_TOKENIZERS_BRANCH ?= a2c94833148c4c6a3d14223f9a666d361dab5b43 # master 2025-08-01
8080

8181
OV_SOURCE_ORG ?= openvinotoolkit
8282
OV_CONTRIB_ORG ?= openvinotoolkit
@@ -161,11 +161,11 @@ ifeq ($(findstring ubuntu,$(BASE_OS)),ubuntu)
161161
ifeq ($(BASE_OS_TAG),24.04)
162162
OS=ubuntu24
163163
INSTALL_DRIVER_VERSION ?= "24.52.32224"
164-
DLDT_PACKAGE_URL ?= https://storage.openvinotoolkit.org/repositories/openvino/packages/nightly/2025.3.0-19616-cc73719a2c6/openvino_toolkit_ubuntu24_2025.3.0.dev20250729_x86_64.tgz
164+
DLDT_PACKAGE_URL ?= https://storage.openvinotoolkit.org/repositories/openvino/packages/nightly/2025.3.0-19757-b3c74fae04a/openvino_toolkit_ubuntu24_2025.3.0.dev20250812_x86_64.tgz
165165
else ifeq ($(BASE_OS_TAG),22.04)
166166
OS=ubuntu22
167167
INSTALL_DRIVER_VERSION ?= "24.39.31294"
168-
DLDT_PACKAGE_URL ?= https://storage.openvinotoolkit.org/repositories/openvino/packages/nightly/2025.3.0-19616-cc73719a2c6/openvino_toolkit_ubuntu22_2025.3.0.dev20250729_x86_64.tgz
168+
DLDT_PACKAGE_URL ?= https://storage.openvinotoolkit.org/repositories/openvino/packages/nightly/2025.3.0-19757-b3c74fae04a/openvino_toolkit_ubuntu22_2025.3.0.dev20250812_x86_64.tgz
169169
endif
170170
endif
171171
ifeq ($(BASE_OS),redhat)
@@ -174,7 +174,7 @@ ifeq ($(BASE_OS),redhat)
174174
BASE_IMAGE ?= registry.access.redhat.com/ubi9/ubi:$(BASE_OS_TAG_REDHAT)
175175
BASE_IMAGE_RELEASE=registry.access.redhat.com/ubi9/ubi-minimal:$(BASE_OS_TAG_REDHAT)
176176
DIST_OS=redhat
177-
DLDT_PACKAGE_URL ?= https://storage.openvinotoolkit.org/repositories/openvino/packages/nightly/2025.3.0-19616-cc73719a2c6/openvino_toolkit_rhel8_2025.3.0.dev20250729_x86_64.tgz
177+
DLDT_PACKAGE_URL ?= https://storage.openvinotoolkit.org/repositories/openvino/packages/nightly/2025.3.0-19757-b3c74fae04a/openvino_toolkit_rhel8_2025.3.0.dev20250812_x86_64.tgz
178178
INSTALL_DRIVER_VERSION ?= "24.52.32224"
179179
endif
180180

WORKSPACE

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -424,6 +424,7 @@ http_archive(
424424
"@mediapipe//third_party:org_tensorflow_custom_ops.diff",
425425
"@mediapipe//third_party:org_tensorflow_objc_build_fixes.diff",
426426
"tf_2.18_logging.patch",
427+
"tf_nsync_chrono.patch",
427428
],
428429
patch_args = [
429430
"-p1",

ci/lib_search.py

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -122,6 +122,7 @@ def check_dir(start_dir):
122122
'upb_warning_turn_off.patch',
123123
'partial_2.18.patch',
124124
'tf_2.18_logging.patch',
125+
'tf_nsync_chrono.patch',
125126
'vehicle_images.txt',
126127
'bazel_rules_apple.patch',
127128
"go.sum",
@@ -227,6 +228,7 @@ def check_func(start_dir):
227228
'tftext.patch',
228229
'partial_2.18.patch',
229230
'tf_2.18_logging.patch',
231+
'tf_nsync_chrono.patch',
230232
'zlib.LICENSE.txt',
231233
'bazel_rules_apple.patch',
232234
'yarn.lock',

common_settings.bzl

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -187,6 +187,8 @@ WINDOWS_COMMON_STATIC_LIBS_COPTS = [
187187
"/GS",
188188
"/DYNAMICBASE",
189189
"/Qspectre",
190+
"/wd4305", # abseil after switch to build tools 22
191+
"/wd4324", # genai after switch to build tools 22
190192
"/wd4068",
191193
"/wd4458",
192194
"/wd4100",

demos/common/export_models/requirements.txt

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -3,8 +3,8 @@
33
--extra-index-url "https://storage.openvinotoolkit.org/simple/wheels/pre-release"
44
--pre
55
optimum-intel
6-
openvino-tokenizers<=2025.3.0.0.dev20250729
7-
openvino<=2025.3.0.dev20250729
6+
openvino-tokenizers<=2025.3.0.0.dev20250812
7+
openvino<=2025.3.0.dev20250812
88
nncf>=2.11.0
99
sentence_transformers
1010
sentencepiece==0.2.0

external/cb.patch

Lines changed: 0 additions & 30 deletions
This file was deleted.

external/tf_nsync_chrono.patch

Lines changed: 31 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,31 @@
1+
diff --git a/tensorflow/workspace2.bzl b/tensorflow/workspace2.bzl
2+
index be83c971749..bd80e19896e 100644
3+
--- a/tensorflow/workspace2.bzl
4+
+++ b/tensorflow/workspace2.bzl
5+
@@ -396,7 +396,7 @@ def _tf_repositories():
6+
7+
tf_http_archive(
8+
name = "nsync",
9+
- patch_file = ["//third_party:nsync.patch"],
10+
+ patch_file = ["//third_party:nsync.patch", "//third_party:nsync_chrono.patch"],
11+
sha256 = "1d63e967973733d2c97e841e3c05fac4d3fa299f01d14c86f2695594c7a4a2ec",
12+
strip_prefix = "nsync-1.29.2",
13+
system_build_file = "//third_party/systemlibs:nsync.BUILD",
14+
diff --git a/third_party/nsync_chrono.patch b/third_party/nsync_chrono.patch
15+
new file mode 100644
16+
index 00000000000..7f0f8539727
17+
--- /dev/null
18+
+++ b/third_party/nsync_chrono.patch
19+
@@ -0,0 +1,12 @@
20+
+diff --git a/platform/c++11/platform.h b/platform/c++11/platform.h
21+
+index 2c80e0b..4fbc6d0 100644
22+
+--- a/platform/c++11/platform.h
23+
++++ b/platform/c++11/platform.h
24+
+@@ -28,6 +28,7 @@
25+
+
26+
+ #include <mutex>
27+
+ #include <condition_variable>
28+
++#include <chrono>
29+
+
30+
+ #include "nsync_cpp.h"
31+
+

src/test/llm/llmnode_test.cpp

Lines changed: 11 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -2680,7 +2680,8 @@ TEST_P(LLMHttpParametersValidationTest, missingContentInMessage) {
26802680
#endif
26812681

26822682
if (params.modelName.find("vlm") != std::string::npos) {
2683-
genAiTemplateParsing = true; // VLM models always use GenAI template parsing
2683+
ASSERT_EQ(status.getCode(), ovms::StatusCode::OK); // GenAI accepts such messages, so we expect a successful response
2684+
return;
26842685
}
26852686

26862687
if (genAiTemplateParsing) {
@@ -3189,9 +3190,15 @@ TEST_P(LLMHttpParametersValidationTest, MessagesWithOnlyRole) {
31893190
}
31903191
)";
31913192

3192-
ASSERT_EQ(
3193-
handler->dispatchToProcessor(endpointChatCompletions, requestBody, &response, comp, responseComponents, writer, multiPartParser),
3194-
ovms::StatusCode::MEDIAPIPE_EXECUTION_ERROR);
3193+
if (params.modelName.find("vlm") != std::string::npos) {
3194+
ASSERT_EQ(
3195+
handler->dispatchToProcessor(endpointChatCompletions, requestBody, &response, comp, responseComponents, writer, multiPartParser),
3196+
ovms::StatusCode::OK); // GenAI supports such messages
3197+
} else {
3198+
ASSERT_EQ(
3199+
handler->dispatchToProcessor(endpointChatCompletions, requestBody, &response, comp, responseComponents, writer, multiPartParser),
3200+
ovms::StatusCode::MEDIAPIPE_EXECUTION_ERROR);
3201+
}
31953202
}
31963203

31973204
TEST_P(LLMHttpParametersValidationTest, SpeculativeDecodingNoSDSpecificParametersProvided) {

third_party/llm_engine/llm_engine.bzl

Lines changed: 7 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -24,12 +24,11 @@ def llm_engine():
2424
new_git_repository(
2525
name = "llm_engine",
2626
remote = "https://github.com/openvinotoolkit/openvino.genai",
27-
commit = "31b2c55dadcbfb1e4413eeb32c355c81e5bd436a", # master 2025-07-29
27+
commit = "d6b2e816ba8edc52f3f4ac459240f664df67c1b8", # master 2025-08-13
2828
build_file = "@_llm_engine//:BUILD",
2929
init_submodules = True,
3030
recursive_init_submodules = True,
3131
patch_args = ["-p1"],
32-
patches = ["cb.patch"],
3332
)
3433
# when using local repository manually run: git submodule update --recursive
3534
#native.new_local_repository(
@@ -66,6 +65,9 @@ def _impl(repository_ctx):
6665
"WIN32": "True",
6766
"X86_64": "True",
6867
"BUILD_TOKENIZERS": "OFF",
68+
"ENABLE_SAMPLES": "OFF",
69+
"ENABLE_TOOLS": "OFF",
70+
"ENABLE_TESTS": "OFF",
6971
"ENABLE_XGRAMMAR": "ON",
7072
"""
7173
jobs_param = "\"-j 8\"" # on Windows we do not need to specify number of jobs, it's set to all available cores number
@@ -82,6 +84,9 @@ def _impl(repository_ctx):
8284
"CMAKE_ARCHIVE_OUTPUT_DIRECTORY": "lib",
8385
"ENABLE_SYSTEM_ICU": "True",
8486
"BUILD_TOKENIZERS": "OFF",
87+
"ENABLE_SAMPLES": "OFF",
88+
"ENABLE_TOOLS": "OFF",
89+
"ENABLE_TESTS": "OFF",
8590
"ENABLE_XGRAMMAR": "ON",
8691
"""
8792
jobs_param = "\"-j 8\"" # on Linux we need to specify jobs number, by default it's set to 1

windows_build.bat

Lines changed: 1 addition & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -56,26 +56,20 @@ set "setPythonPath=%cd%\bazel-out\x64_windows-opt\bin\src\python\binding"
5656
set "BAZEL_SH=C:\opt\msys64\usr\bin\bash.exe"
5757

5858
:: Bazel compilation settings
59-
set VS_2019_PRO="C:\Program Files (x86)\Microsoft Visual Studio\2019\Professional"
6059
set VS_2022_BT="C:\Program Files (x86)\Microsoft Visual Studio\2022\BuildTools"
61-
IF /I EXIST %VS_2019_PRO% goto :msvc_pro
6260
IF /I EXIST %VS_2022_BT% goto :msvc_bt ELSE goto :msvc_error
6361

6462
:msvc_error
6563
echo [ERROR] Required MSVC compiler not installed
6664
goto :exit_build_error
67-
:msvc_pro
68-
echo [INFO] Using MSVC %VS_2019_PRO%
69-
set BAZEL_VS=%VS_2019_PRO%
70-
goto :msvc_end
7165
:msvc_bt
7266
echo [INFO] Using MSVC %VS_2022_BT%
7367
set BAZEL_VS=%VS_2022_BT%
7468

7569
:: Bazel compilation settings end
7670
:msvc_end
7771
set "BAZEL_VC=%BAZEL_VS:"=%\VC"
78-
set "BAZEL_VC_FULL_VERSION=14.29.30133"
72+
set "BAZEL_VC_FULL_VERSION=14.44.35207"
7973

8074
:: Set proper PATH environment variable: Remove other python paths and add c:\opt with bazel to PATH
8175
set "PATH=%setPath%"

0 commit comments

Comments
 (0)