Drop python 3.8 support (#182)

* Drop python 3.8 support

* Revert py3.8 drop

* Temporaly disable pip cache in cpp precommit to avoid conflicts

* Shake OpenCV version to check if it fixes missing OpenCV problem in CMake

* Try the latest OCV version for Win

* Upgrade OV in win builder

* Promote gtest version

* Bump OV version for cpp

* Disable running win precommits

* Try to restrict cpp features for win tests

* Disable win precommit pip cache

* Use sample as a sanity check on Win CI

Author: sovrasov

.github/workflows/test_accuracy.yml

@@ -11,7 +11,7 @@ jobs:
     - uses: actions/checkout@v3
     - uses: actions/setup-python@v4
       with:
-        python-version: 3.8
+        python-version: 3.9
         cache: pip
     - name: Create and start a virtual environment
       run: |

.github/workflows/test_precommit.yml

@@ -16,7 +16,7 @@ jobs:
     - run: "! git grep -n '[^ -~]' -- ':(exclude)model_api/python/README.md'"
     - uses: actions/setup-python@v4
       with:
-        python-version: 3.8
+        python-version: 3.9
         cache: pip
     - name: Create and start a virtual environment
       run: |
@@ -41,7 +41,7 @@ jobs:
     - uses: actions/checkout@v3
     - uses: actions/setup-python@v4
       with:
-        python-version: 3.8
+        python-version: 3.9
         cache: pip
     - name: Create and start a virtual environment
       run: |
@@ -77,8 +77,7 @@ jobs:
     - uses: actions/checkout@v3
     - uses: actions/setup-python@v4
       with:
-        python-version: 3.8
-        cache: pip
+        python-version: 3.9
     - name: Create and start a virtual environment
       run: |
         python -m venv venv
@@ -108,8 +107,8 @@ jobs:
     - uses: actions/checkout@v3
     - uses: actions/setup-python@v4
       with:
-        python-version: 3.8
-        cache: pip
+        python-version: 3.9
+        #cache: pip
     - name: Create and start a virtual environment
       shell: bash
       run: |
@@ -121,11 +120,12 @@ jobs:
         source venv/Scripts/activate
         python -m pip install --upgrade pip
         pip install model_api/python/[tests] --extra-index-url https://download.pytorch.org/whl/cpu
-        curl https://storage.openvinotoolkit.org/repositories/openvino/packages/2023.0/windows/w_openvino_toolkit_windows_2023.0.0.10926.b4452d56304_x86_64.zip --output w_openvino_toolkit_windows.zip
+        curl https://storage.openvinotoolkit.org/repositories/openvino/packages/2024.1/windows/w_openvino_toolkit_windows_2024.1.0.15008.f4afc983258_x86_64.zip --output w_openvino_toolkit_windows.zip
         unzip w_openvino_toolkit_windows.zip
         rm w_openvino_toolkit_windows.zip
-        curl -L https://github.com/opencv/opencv/releases/download/4.7.0/opencv-4.7.0-windows.exe --output opencv-4.7.0-windows.exe
-        ./opencv-4.7.0-windows.exe -oopencv -y
+        curl -L https://github.com/opencv/opencv/releases/download/4.10.0/opencv-4.10.0-windows.exe --output opencv-4.10.0-windows.exe
+        ./opencv-4.10.0-windows.exe -oopencv -y
+        rm opencv-4.10.0-windows.exe
     - name: Prepare test data
       shell: bash
       run: |
@@ -135,15 +135,14 @@ jobs:
       shell: bash
       run: |
         mkdir build && cd build
-        MSYS_NO_PATHCONV=1 cmake ../tests/cpp/precommit/ -DOpenVINO_DIR=$GITHUB_WORKSPACE/w_openvino_toolkit_windows_2023.0.0.10926.b4452d56304_x86_64/runtime/cmake -DOpenCV_DIR=$GITHUB_WORKSPACE/opencv/opencv/build -DCMAKE_CXX_FLAGS=/WX
+        MSYS_NO_PATHCONV=1 cmake ../examples/cpp/synchronous_api/ -DOpenVINO_DIR=$GITHUB_WORKSPACE/w_openvino_toolkit_windows_2024.1.0.15008.f4afc983258_x86_64/runtime/cmake -DOpenCV_DIR=$GITHUB_WORKSPACE/opencv/opencv/build -DCMAKE_CXX_FLAGS=/WX
         cmake --build . --config Release -j $((`nproc`*2+2))
-    - name: Run test
+    - name: Run sync sample
       shell: cmd
       # .\w_openvino_toolkit_windows_2023.0.0.10926.b4452d56304_x86_64\setupvars.bat exits with 0 code without moving to a next command. Set PATH manually
       run: |
-        set PATH=opencv\opencv\build\x64\vc16\bin;w_openvino_toolkit_windows_2023.0.0.10926.b4452d56304_x86_64\runtime\bin\intel64\Release;w_openvino_toolkit_windows_2023.0.0.10926.b4452d56304_x86_64\runtime\3rdparty\tbb\bin;%PATH%
-        .\build\Release\test_sanity.exe -d data -p tests\cpp\precommit\public_scope.json
-        .\build\Release\test_model_config -d data
+        set PATH=opencv\opencv\build\x64\vc16\bin;w_openvino_toolkit_windows_2024.1.0.15008.f4afc983258_x86_64\runtime\bin\intel64\Release;w_openvino_toolkit_windows_2024.1.0.15008.f4afc983258_x86_64\runtime\3rdparty\tbb\bin;%PATH%
+        .\build\Release\synchronous_api.exe .\data\public\ssd_mobilenet_v1_fpn_coco\FP16\ssd_mobilenet_v1_fpn_coco.xml .\data\BloodImage_00007.jpg
   serving_api:
     strategy:
       fail-fast: false

model_api/cpp/install_dependencies.sh

@@ -5,7 +5,7 @@ wget https://apt.repos.intel.com/intel-gpg-keys/GPG-PUB-KEY-INTEL-SW-PRODUCTS.PU
 
 apt-key add GPG-PUB-KEY-INTEL-SW-PRODUCTS.PUB
 
-echo "deb https://apt.repos.intel.com/openvino/2023 ubuntu22 main" | sudo tee /etc/apt/sources.list.d/intel-openvino-2023.list
+echo "deb https://apt.repos.intel.com/openvino/2024 ubuntu22 main" | sudo tee /etc/apt/sources.list.d/intel-openvino-2024.list
 
 apt update
 

tests/cpp/accuracy/CMakeLists.txt

@@ -19,6 +19,7 @@ if(WIN32)
     if(NOT "${CMAKE_SIZEOF_VOID_P}" EQUAL "8")
         message(FATAL_ERROR "Only 64-bit supported on Windows")
     endif()
+    set(CMAKE_CXX_STANDARD 14)
 
     add_definitions(-DNOMINMAX)
 endif()
@@ -57,7 +58,7 @@ include(FetchContent)
 FetchContent_Declare(json URL https://github.com/nlohmann/json/releases/download/v3.11.2/json.tar.xz)
 FetchContent_Declare(googletest
     GIT_REPOSITORY https://github.com/google/googletest.git
-    GIT_TAG 471087fbfcbc3c66071189a67c313eb1d525ee51  # latest main
+    GIT_TAG a7f443b80b105f940225332ed3c31f2790092f47  # latest main
 )
 FetchContent_MakeAvailable(json googletest)
 

tests/cpp/precommit/CMakeLists.txt

@@ -12,13 +12,13 @@ if(NOT GENERATOR_IS_MULTI_CONFIG_VAR AND NOT DEFINED CMAKE_BUILD_TYPE)
     # Setting CMAKE_BUILD_TYPE as CACHE must go before project(). Otherwise project() sets its value and set() doesn't take an effect
     set(CMAKE_BUILD_TYPE Release CACHE STRING "Choose the type of build, options are: None Debug Release RelWithDebInfo MinSizeRel ...")
 endif()
-
 project(tests)
 
 if(WIN32)
     if(NOT "${CMAKE_SIZEOF_VOID_P}" EQUAL "8")
         message(FATAL_ERROR "Only 64-bit supported on Windows")
     endif()
+    set(CMAKE_CXX_STANDARD 14)
 
     add_definitions(-DNOMINMAX)
 endif()
@@ -57,7 +57,7 @@ include(FetchContent)
 FetchContent_Declare(json URL https://github.com/nlohmann/json/releases/download/v3.11.2/json.tar.xz)
 FetchContent_Declare(googletest
     GIT_REPOSITORY https://github.com/google/googletest.git
-    GIT_TAG 471087fbfcbc3c66071189a67c313eb1d525ee51  # latest main
+    GIT_TAG a7f443b80b105f940225332ed3c31f2790092f47  # latest main
 )
 FetchContent_MakeAvailable(json googletest)