Skip to content

[CI] [GHA] Introduce JS API as a part of the existing workflows#21898

Merged
ilya-lavrenov merged 21 commits intoopenvinotoolkit:masterfrom
akashchi:ci/gha/include-js-api-into-existing-workflows
Jan 8, 2024
Merged

[CI] [GHA] Introduce JS API as a part of the existing workflows#21898
ilya-lavrenov merged 21 commits intoopenvinotoolkit:masterfrom
akashchi:ci/gha/include-js-api-into-existing-workflows

Conversation

@akashchi
Copy link
Copy Markdown
Contributor

Details:

  • 120634

@akashchi akashchi added category: CI OpenVINO public CI WIP work in progress github_actions Pull requests that update GitHub Actions code labels Dec 28, 2023
@akashchi akashchi added this to the 2024.0 milestone Dec 28, 2023
@github-actions github-actions bot added category: build OpenVINO cmake script / infra category: packaging OpenVINO packaging / distribution labels Jan 3, 2024
@akashchi akashchi marked this pull request as ready for review January 4, 2024 10:15
@akashchi akashchi requested review from a team as code owners January 4, 2024 10:15
@akashchi akashchi removed the WIP work in progress label Jan 4, 2024
cmake \
-DCPACK_GENERATOR=NPM \
-DENABLE_SYSTEM_TBB=OFF -UTBB* \
-DOV_CPU_ARM_TARGET_ARCH="armv8.2-a" \
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I suppose we can remove explicit OV_CPU_ARM_TARGET_ARCH, because armv8.2-a is a default value

-DENABLE_SYSTEM_TBB=OFF -UTBB* \
-DOV_CPU_ARM_TARGET_ARCH="armv8.2-a" \
-DENABLE_INTEL_GPU=OFF \
-DENABLE_MLAS_FOR_CPU=ON \
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

ENABLE_MLAS_FOR_CPU is ON by default.

@ilya-lavrenov ilya-lavrenov merged commit f0f1e7a into openvinotoolkit:master Jan 8, 2024
@ilya-lavrenov ilya-lavrenov added the port to 2023.3 Need port from master to 2023.3 LTS label Jan 8, 2024
@ilya-lavrenov ilya-lavrenov removed the port to 2023.3 Need port from master to 2023.3 LTS label Jan 19, 2024
ilya-lavrenov added a commit that referenced this pull request Jan 19, 2024
* [CI] [GHA] Introduce JS API as a part of the existing workflows (#21898)

* add js api to linux

* try inside the ov repo

* use rel path

* use a separate job for js api

* correct command formatting

* add missing var

* use spacing

* mv js building

* add node installing

* add to windows

* check pwsh and cmd running npm

* add smart CI conditions; disable for win

* use node version as env var

* extract js job into a separate workflow, add to other *nix

* fix input name

* Activate js bindings tests for arm64

* upload ov js package

* correct formatting

* add missing syntax

---------

Co-authored-by: Vishniakov Nikolai <nikolai.vishniakov@intel.com>

* Cmake Python build option flags should be added to the command in step #3 not step #4. I fixed the typo (#21993)

* [CI] [GHA] [JS API] Remove explicit default values settings in Linux ARM64 `cmake` (#22019)

* rm explicit default values settings

* Activate mac arm64 js api check

* Specify test run

---------

Co-authored-by: Vishniakov Nikolai <nikolai.vishniakov@intel.com>

* [OV JS] Activate validation for mac x86 (#22035)

* Extend validation for mac x86

* Remove extra params

* fixed broken doc links (#22088)

Co-authored-by: Przemyslaw Wysocki <przemyslaw.wysocki@intel.com>

* [GHA] Update MO deps (#22130)

* [GHA] Update MO deps

Signed-off-by: Kazantsev, Roman <roman.kazantsev@intel.com>

* Update .github/components.yml

---------

Signed-off-by: Kazantsev, Roman <roman.kazantsev@intel.com>

* Avoid DOWNLOAD_EXTRACT_TIMESTAMP warning (#22135)

* Avoid DOWNLOAD_EXTRACT_TIMESTAMP warning

* Change applying policy condition

Co-authored-by: Ilya Lavrenov <ilya.lavrenov@intel.com>

---------

Co-authored-by: Ilya Lavrenov <ilya.lavrenov@intel.com>

* Fixed API validator search (#22136)

* [OV JS] Conditional enabling of JS API (#22139)

* Disable js api building for vcpkg

* Disable JS API by default

* Add disable JS API conditions in features.cmake

* Update cmake/features.cmake

* Update src/bindings/js/CMakeLists.txt

---------

Co-authored-by: Ilya Lavrenov <ilya.lavrenov@intel.com>

* Fixed GHSA-h5c8-rqwp-cp95 (#22159)

* [PyOV][SAMPLES] Fix bugbear issue B038 (#22183)

* Fixed compilation on GHA CI

* Decrease number of workers for ONNX Model tests to prevent OOM kills (#22243)

* Decrease number of workers for ONNX Model tests to prevent OOM kills

* Try to use "-n auto" also

---------

Signed-off-by: Kazantsev, Roman <roman.kazantsev@intel.com>
Co-authored-by: Andrei Kashchikhin <andrey.kashchikhin@intel.com>
Co-authored-by: Vishniakov Nikolai <nikolai.vishniakov@intel.com>
Co-authored-by: fredrickomondi <omondifredrick@gmail.com>
Co-authored-by: Santhosh Mamidisetti <92091342+SANTHOSH-MAMIDISETTI@users.noreply.github.com>
Co-authored-by: Przemyslaw Wysocki <przemyslaw.wysocki@intel.com>
Co-authored-by: Roman Kazantsev <roman.kazantsev@intel.com>
Co-authored-by: Jan Iwaszkiewicz <jan.iwaszkiewicz@intel.com>
Co-authored-by: Andrey Babushkin <andrey.babushkin@intel.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

category: build OpenVINO cmake script / infra category: CI OpenVINO public CI category: packaging OpenVINO packaging / distribution github_actions Pull requests that update GitHub Actions code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants