Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Make time && stress tests independent from IEDeveloperPackage #7411

Merged
merged 9 commits into from
Sep 16, 2021

Conversation

ishariko
Copy link
Contributor

@ishariko ishariko commented Sep 7, 2021

Details:

  • Get rid of InferenceEngineDeveloperPackage_DIR in order to provide possibility to build tests on installed OpenVINO. To achieve it, include gflags and gtest building into stress tests build

Tickets:

  • 49280

@ishariko ishariko marked this pull request as ready for review September 9, 2021 20:32
@ishariko ishariko requested a review from a team September 9, 2021 20:32
@ishariko
Copy link
Contributor Author

ishariko commented Sep 9, 2021

This MR need merge together with CI PR.

@ishariko ishariko requested a review from a team as a code owner September 9, 2021 20:47
@ishariko ishariko requested review from andrew-zaytsev and removed request for a team September 9, 2021 20:47
@ishariko ishariko added the category: IE Tests OpenVINO Test: plugins and common label Sep 9, 2021
tests/stress_tests/README.md Outdated Show resolved Hide resolved
tests/stress_tests/README.md Show resolved Hide resolved
tests/time_tests/README.md Show resolved Hide resolved
@@ -12,7 +19,8 @@ endif()
# Define directory where artifacts will be placed
set(OUTPUT_ROOT ${CMAKE_CURRENT_SOURCE_DIR})

find_package(InferenceEngineDeveloperPackage REQUIRED)
# Search OpenVINO Inference Engine installed
find_package(InferenceEngine REQUIRED)
Copy link
Contributor

Choose a reason for hiding this comment

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

Maybe can we start to use new OpenVINO cmake config?

@@ -15,21 +15,11 @@ the commands below:

1. Build tests:
``` bash
source <OpenVINO_install_dir>/bin/setupvars.sh
Copy link
Contributor

Choose a reason for hiding this comment

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

-> <OpenVINO_install_dir>/setupvars.sh

mkdir -p build && cd build && \
cmake -DInferenceEngineDeveloperPackage_DIR=$OPENVINO_BUILD_DIR .. && make -j$(nproc) \
)
source <OpenVINO_install_dir>/bin/setupvars.sh
Copy link
Contributor

Choose a reason for hiding this comment

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

bin/setupvars.sh -> setupvars.sh

@azhogov
Copy link

azhogov commented Sep 16, 2021

@azhogov azhogov merged commit 5847b35 into openvinotoolkit:master Sep 16, 2021
akuporos pushed a commit to akuporos/openvino that referenced this pull request Sep 29, 2021
…notoolkit#7411)

* Specify own CXX configuration for time && stress tests

* Included os_flags for build gflags

* Updated README

* Updated commands for build tests in README

* Use new cmake config
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
category: IE Tests OpenVINO Test: plugins and common
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants