Skip to content

Commit

Permalink
do not use sccache, use \0 for termination
Browse files Browse the repository at this point in the history
  • Loading branch information
akashchi committed Feb 6, 2024
1 parent 0e65932 commit f8af793
Show file tree
Hide file tree
Showing 2 changed files with 24 additions and 26 deletions.
46 changes: 22 additions & 24 deletions .github/workflows/linux.yml
Original file line number Diff line number Diff line change
Expand Up @@ -61,12 +61,12 @@ jobs:
DEBIAN_FRONTEND: noninteractive # to prevent apt-get from waiting user input
CMAKE_BUILD_TYPE: 'Release'
CMAKE_GENERATOR: 'Ninja Multi-Config'
CMAKE_CXX_COMPILER_LAUNCHER: sccache
CMAKE_C_COMPILER_LAUNCHER: sccache
SCCACHE_IGNORE_SERVER_IO_ERROR: 1
SCCACHE_SERVER_PORT: 35555
SCCACHE_ERROR_LOG: /__w/openvino/sccache_log.txt
SCCACHE_LOG: warn
# CMAKE_CXX_COMPILER_LAUNCHER: sccache
# CMAKE_C_COMPILER_LAUNCHER: sccache
# SCCACHE_IGNORE_SERVER_IO_ERROR: 1
# SCCACHE_SERVER_PORT: 35555
# SCCACHE_ERROR_LOG: /__w/openvino/sccache_log.txt
# SCCACHE_LOG: warn
GITHUB_WORKSPACE: '/__w/openvino/openvino'
OPENVINO_REPO: /__w/openvino/openvino/openvino
OPENVINO_CONTRIB_REPO: /__w/openvino/openvino/openvino_contrib
Expand All @@ -75,7 +75,7 @@ jobs:
INSTALL_TEST_DIR: /__w/openvino/openvino/tests_install
DEVELOPER_PACKAGE_DIR: /__w/openvino/openvino/developer_package_install
BUILD_DIR: /__w/openvino/openvino/openvino_build
SCCACHE_AZURE_KEY_PREFIX: ubuntu20_x86_64_Release
# SCCACHE_AZURE_KEY_PREFIX: ubuntu20_x86_64_Release
ONNX_RUNTIME_UTILS: /__w/openvino/openvino/openvino/src/frontends/onnx/tests/ci_utils/onnxruntime
if: "!needs.smart_ci.outputs.skip_workflow"

Expand Down Expand Up @@ -119,10 +119,10 @@ jobs:
# default-jdk - Java API
apt install --assume-yes --no-install-recommends default-jdk
- name: Install sccache
uses: mozilla-actions/[email protected]
with:
version: "v0.7.5"
# - name: Install sccache
# uses: mozilla-actions/[email protected]
# with:
# version: "v0.7.5"

- name: Setup Python ${{ env.PYTHON_VERSION }}
uses: ./openvino/.github/actions/setup_python
Expand Down Expand Up @@ -167,20 +167,18 @@ jobs:
-DCMAKE_VERBOSE_MAKEFILE=ON \
-DCPACK_GENERATOR=TGZ \
-DCMAKE_COMPILE_WARNING_AS_ERROR=ON \
-DCMAKE_CXX_COMPILER_LAUNCHER=${{ env.CMAKE_CXX_COMPILER_LAUNCHER }} \
-DCMAKE_C_COMPILER_LAUNCHER=${{ env.CMAKE_C_COMPILER_LAUNCHER }} \
-DENABLE_SANITIZER=ON \
-S ${OPENVINO_REPO} \
-B ${BUILD_DIR}
- name: Clean sccache stats
run: ${SCCACHE_PATH} --zero-stats
# - name: Clean sccache stats
# run: ${SCCACHE_PATH} --zero-stats

- name: Cmake build - OpenVINO
run: cmake --build ${BUILD_DIR} --parallel --config ${{ env.CMAKE_BUILD_TYPE }}

- name: Show sccache stats
run: ${SCCACHE_PATH} --show-stats
# - name: Show sccache stats
# run: ${SCCACHE_PATH} --show-stats

- name: Cmake install - OpenVINO
run: |
Expand Down Expand Up @@ -243,13 +241,13 @@ jobs:
#
# Upload build artifacts and logs
#
- name: Upload build logs
uses: actions/upload-artifact@v3
if: always()
with:
name: build_logs
path: ${{ env.SCCACHE_ERROR_LOG }}
if-no-files-found: 'ignore'
# - name: Upload build logs
# uses: actions/upload-artifact@v3
# if: always()
# with:
# name: build_logs
# path: ${{ env.SCCACHE_ERROR_LOG }}
# if-no-files-found: 'ignore'

- name: Upload openvino package
if: ${{ always() }}
Expand Down
4 changes: 2 additions & 2 deletions src/inference/src/dev/converter_utils.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -441,9 +441,9 @@ class IInferRequestInternalWrapper : public InferenceEngine::IInferRequestIntern
old_info.execution_index = static_cast<unsigned>(i);
old_info.realTime_uSec = info.real_time.count();
strncpy(old_info.exec_type, info.exec_type.c_str(), sizeof(old_info.exec_type));
old_info.exec_type[sizeof(old_info.exec_type) - 1] = 0;
old_info.exec_type[sizeof(old_info.exec_type) - 1] = '\0';
strncpy(old_info.layer_type, info.node_type.c_str(), sizeof(old_info.layer_type));
old_info.layer_type[sizeof(old_info.layer_type) - 1] = 0;
old_info.layer_type[sizeof(old_info.layer_type) - 1] = '\0';
switch (info.status) {
case ov::ProfilingInfo::Status::EXECUTED:
old_info.status = InferenceEngine::InferenceEngineProfileInfo::EXECUTED;
Expand Down

0 comments on commit f8af793

Please sign in to comment.