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] Introduce GHA Linux Conditional Compilation Pipeline #19341

Merged
merged 8 commits into from
Sep 15, 2023

Conversation

akashchi
Copy link
Contributor

@akashchi akashchi commented Aug 22, 2023

Details:

This PR introduces the Linux Conditional Compilation pipeline in the GitHub Actions environment. It is mirroring the Azure Linux Conditional Compilation pipeline.

Tickets:

  • 117538

@akashchi akashchi added category: CI OpenVINO public CI WIP work in progress github_actions Pull requests that update GitHub Actions code labels Aug 22, 2023
@akashchi akashchi added this to the 2023.2 milestone Aug 22, 2023
@akashchi akashchi requested a review from a team as a code owner August 22, 2023 13:53
@akashchi akashchi removed the WIP work in progress label Aug 22, 2023
Copy link
Contributor

@mryzhov mryzhov left a comment

Choose a reason for hiding this comment

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

Don't see log collection step, is it expected?

@akashchi
Copy link
Contributor Author

Don't see log collection step, is it expected?

There is no log collection step in the Azure counterpart and I believe that there are no test stages that would generate artifacts of some sort to collect.

python3 ${{ env.OPENVINO_REPO }}/thirdparty/itt_collector/runtool/sea_runtool.py \
--bindir ${{ env.OPENVINO_REPO }}/bin/intel64/Release -o ${{ env.BUILD_DIR }}/itt_stat ! \
${{ env.OPENVINO_REPO }}/bin/intel64/Release/benchmark_app -niter 1 -nireq 1 \
-m ${{ env.MODELS_PATH }}/models/test_model/test_model_fp32.xml -d CPU
Copy link
Contributor

Choose a reason for hiding this comment

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

I recommend to use several real models instead of dummy test_modelt
We can download models from public model hubs (TF, others) and use it here.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Is there any way we could download such model(s) w/o building and installing the Python wheels for model download and convertation? Download IRs directly from some storage?

Or do you suggest extending the workflow beyond what is in Azure with Python setup, dependencies setup, wheels building, and model downloading and convertation?

Copy link
Contributor

Choose a reason for hiding this comment

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

@ilya-lavrenov , I would suggest we move the pipelines without modifications first, and improve after it's working.

.github/workflows/linux_conditional_compilation.yml Outdated Show resolved Hide resolved
@akashchi akashchi added the WIP work in progress label Sep 11, 2023
@akashchi akashchi removed the WIP work in progress label Sep 14, 2023
@ilya-lavrenov ilya-lavrenov merged commit e090e37 into openvinotoolkit:master Sep 15, 2023
alvoron pushed a commit to alvoron/openvino that referenced this pull request Nov 6, 2023
…vinotoolkit#19341)

* introduce linux cc

* disable some triggers

* check dirs

* use another model

* return model

* rm triggers

---------

Co-authored-by: Mikhail Ryzhov <[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.

4 participants