Skip to content

Commit e756273

Browse files
Merge pull request #33 from Wovchena/fix-compilation-for-ubuntu20
Fix compilation for Ubuntu 20
2 parents 5d4d055 + 319464e commit e756273

File tree

3 files changed

+12
-3
lines changed

3 files changed

+12
-3
lines changed

text_generation/causal_lm/cpp/continuous_batching/apps/CMakeLists.txt

+4-1
Original file line numberDiff line numberDiff line change
@@ -17,13 +17,16 @@ FetchContent_MakeAvailable(cxxopts)
1717
FetchContent_MakeAvailable(nlohmann_json)
1818

1919
find_package(OpenVINO REQUIRED COMPONENTS Runtime)
20+
find_package(Threads REQUIRED)
2021

2122
# end of dependencies
2223

2324
set(TARGET_NAME accuracy_sample)
2425
add_executable(${TARGET_NAME} ${TARGET_NAME}.cpp)
2526
target_link_libraries(${TARGET_NAME} PRIVATE openvino::continuous_batching cxxopts::cxxopts)
27+
target_compile_features(${TARGET_NAME} PRIVATE cxx_std_20)
2628

2729
set(TARGET_NAME throughput_benchmark)
2830
add_executable(${TARGET_NAME} ${TARGET_NAME}.cpp)
29-
target_link_libraries(${TARGET_NAME} PRIVATE openvino::continuous_batching nlohmann_json::nlohmann_json cxxopts::cxxopts)
31+
target_link_libraries(${TARGET_NAME} PRIVATE openvino::continuous_batching nlohmann_json::nlohmann_json cxxopts::cxxopts Threads::Threads)
32+
target_compile_features(${TARGET_NAME} PRIVATE cxx_std_20)

text_generation/causal_lm/cpp/continuous_batching/apps/throughput_benchmark.cpp

+1-1
Original file line numberDiff line numberDiff line change
@@ -501,7 +501,7 @@ int main(int argc, char* argv[]) try {
501501

502502
GenerationInfoCollector generation_info_collector;
503503

504-
std::atomic<bool> finishGenerationThread = false;
504+
std::atomic<bool> finishGenerationThread{false};
505505
if (request_rate == "inf") {
506506
std::thread trafficSimulatorThread(trafficSimulator, &pipe, &dataset, request_rate, &generation_info_collector);
507507
trafficSimulatorThread.join();

text_generation/causal_lm/cpp/continuous_batching/library/CMakeLists.txt

+7-1
Original file line numberDiff line numberDiff line change
@@ -38,7 +38,12 @@ add_library(openvino::continuous_batching ALIAS openvino_continuous_batching)
3838

3939
target_include_directories(${TARGET_NAME} PRIVATE "${CMAKE_CURRENT_SOURCE_DIR}/src"
4040
PUBLIC "${CMAKE_CURRENT_SOURCE_DIR}/include")
41-
target_compile_definitions(${TARGET_NAME} PRIVATE OPENVINO_TOKENIZERS_PATH="libopenvino_tokenizers.so")
41+
if(TARGET openvino_tokenizers)
42+
set(OPENVINO_TOKENIZERS_PATH $<TARGET_FILE:openvino_tokenizers>)
43+
else()
44+
message(FATAL_ERROR "${TEST_TARGET_NAME} must be compiled as part of OpenVIINOGenAI to have the path to openvino_tokenizers hardcoded.")
45+
endif()
46+
target_compile_definitions(${TARGET_NAME} PRIVATE OPENVINO_TOKENIZERS_PATH="${OPENVINO_TOKENIZERS_PATH}")
4247
set_target_properties(${TARGET_NAME} PROPERTIES CXX_STANDARD 14 CXX_STANDARD_REQUIRED ON)
4348

4449
target_link_libraries(${TARGET_NAME} PUBLIC openvino::runtime PRIVATE nlohmann_json::nlohmann_json)
@@ -73,3 +78,4 @@ add_executable(${TEST_TARGET_NAME} "src/tests/scheduler.cpp" "src/tests/block_ma
7378
target_link_libraries(${TEST_TARGET_NAME} PUBLIC ${TARGET_NAME} openvino::runtime gtest_main)
7479
target_include_directories(${TEST_TARGET_NAME} PRIVATE "${CMAKE_CURRENT_SOURCE_DIR}/src/"
7580
PUBLIC "${CMAKE_CURRENT_SOURCE_DIR}/include")
81+
target_compile_features(${TEST_TARGET_NAME} PRIVATE cxx_std_20)

0 commit comments

Comments
 (0)