Skip to content

[OV JS] Conditional enabling of JS API#22139

Merged
ilya-lavrenov merged 5 commits intoopenvinotoolkit:masterfrom
vishniakov-nikolai:fix/disable-jsapi-for-vcpkg
Jan 13, 2024
Merged

[OV JS] Conditional enabling of JS API#22139
ilya-lavrenov merged 5 commits intoopenvinotoolkit:masterfrom
vishniakov-nikolai:fix/disable-jsapi-for-vcpkg

Conversation

@vishniakov-nikolai
Copy link
Copy Markdown
Contributor

Details:

  • item1
  • ...

Tickets:

  • ticket-id

@vishniakov-nikolai vishniakov-nikolai requested a review from a team as a code owner January 12, 2024 15:21
@github-actions github-actions bot added category: build OpenVINO cmake script / infra category: JS API OpenVino JS API Bindings labels Jan 12, 2024
@vishniakov-nikolai vishniakov-nikolai requested a review from a team as a code owner January 12, 2024 15:38
@github-actions github-actions bot added category: CI OpenVINO public CI github_actions Pull requests that update GitHub Actions code labels Jan 12, 2024
@vishniakov-nikolai vishniakov-nikolai changed the title [OV JS] Disable js api building for vcpkg [OV JS] Disable js api building by default Jan 12, 2024
@ilya-lavrenov ilya-lavrenov added the port to 2023.3 Need port from master to 2023.3 LTS label Jan 12, 2024
@ilya-lavrenov ilya-lavrenov added this to the 2024.0 milestone Jan 12, 2024
@vishniakov-nikolai vishniakov-nikolai requested a review from a team as a code owner January 12, 2024 16:24
@github-actions github-actions bot removed category: CI OpenVINO public CI github_actions Pull requests that update GitHub Actions code labels Jan 12, 2024
@ilya-lavrenov ilya-lavrenov enabled auto-merge (squash) January 12, 2024 16:43
@vishniakov-nikolai vishniakov-nikolai changed the title [OV JS] Disable js api building by default [OV JS] Conditional enabling of JS API Jan 12, 2024
@ilya-lavrenov ilya-lavrenov merged commit 0a8f138 into openvinotoolkit:master Jan 13, 2024
@vishniakov-nikolai vishniakov-nikolai deleted the fix/disable-jsapi-for-vcpkg branch January 13, 2024 21:32
@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: JS API OpenVino JS API Bindings

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants