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

[CI] [GHA] Refactor Win pipeline: split the Build job, remove unnecessary requirements installation #20320

Merged

Conversation

akashchi
Copy link
Contributor

@akashchi akashchi commented Oct 9, 2023

Tickets:

  • 121357

@akashchi akashchi added category: CI OpenVINO public CI github_actions Pull requests that update GitHub Actions code labels Oct 9, 2023
@akashchi akashchi added this to the 2023.2 milestone Oct 9, 2023
@akashchi akashchi requested a review from a team as a code owner October 9, 2023 13:13
@akashchi akashchi added the WIP work in progress label Oct 9, 2023
.github/workflows/windows.yml Outdated Show resolved Hide resolved
.github/workflows/windows.yml Outdated Show resolved Hide resolved
.github/workflows/windows.yml Outdated Show resolved Hide resolved
.github/workflows/windows.yml Outdated Show resolved Hide resolved
.github/workflows/windows.yml Outdated Show resolved Hide resolved
.github/workflows/windows.yml Show resolved Hide resolved
-DENABLE_CPPLINT=OFF `
-DENABLE_ONEDNN_FOR_GPU=OFF `
-DCMAKE_CXX_STANDARD=14 `
Copy link
Contributor

Choose a reason for hiding this comment

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

proper fix is there opencv/ade#40

@akashchi akashchi removed the WIP work in progress label Oct 10, 2023
@ilya-lavrenov ilya-lavrenov enabled auto-merge (squash) October 10, 2023 09:07
@ilya-lavrenov ilya-lavrenov merged commit fca34c6 into openvinotoolkit:master Oct 10, 2023
36 of 41 checks passed
alvoron pushed a commit to alvoron/openvino that referenced this pull request Nov 6, 2023
…essary requirements installation (openvinotoolkit#20320)

* split build job; rm unnecessary requirements installation

* add missing OV Contrib build

* align requirements install

* rm incorrectly place vsvars, apply comments: add missing tests

* rm build and source options in the cmake OV Contrib command

* add missing build directory

* add logs

* Update windows.yml

Fixed rebuild from scratch with OpenVINO Contrib

* verbose archive extraction

* Fixed path for extraction of openvino_tests package

* Use windows-latest-8-cores for samples

* Update windows.yml

* Update .github/workflows/windows.yml

* use filename for wheel; build ov contrib with ov

---------

Co-authored-by: Ilya Lavrenov <[email protected]>
allnes pushed a commit to allnes/openvino that referenced this pull request Nov 23, 2023
…essary requirements installation (openvinotoolkit#20320)

* split build job; rm unnecessary requirements installation

* add missing OV Contrib build

* align requirements install

* rm incorrectly place vsvars, apply comments: add missing tests

* rm build and source options in the cmake OV Contrib command

* add missing build directory

* add logs

* Update windows.yml

Fixed rebuild from scratch with OpenVINO Contrib

* verbose archive extraction

* Fixed path for extraction of openvino_tests package

* Use windows-latest-8-cores for samples

* Update windows.yml

* Update .github/workflows/windows.yml

* use filename for wheel; build ov contrib with ov

---------

Co-authored-by: Ilya Lavrenov <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
category: CI OpenVINO public CI github_actions Pull requests that update GitHub Actions code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants