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.

Sign up for GitHub

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

Jump to bottom

Updated build documentation for Protopipe #27522

Merged
merged 10 commits into from
Dec 12, 2024
Merged

Updated build documentation for Protopipe #27522

merged 10 commits into from
Dec 12, 2024

Conversation

SeptimiuIoachimNeagaIntel
Copy link
Contributor

Details:

  • Update build documentation for protopipe

@SeptimiuIoachimNeagaIntel SeptimiuIoachimNeagaIntel requested a review from a team as a code owner November 12, 2024 11:42
@SeptimiuIoachimNeagaIntel SeptimiuIoachimNeagaIntel requested review from zKulesza and removed request for a team November 12, 2024 11:42
@github-actions github-actions bot added category: docs OpenVINO documentation category: NPU OpenVINO NPU plugin labels Nov 12, 2024
@sys-openvino-ci sys-openvino-ci added the ExternalPR External contributor label Nov 12, 2024
@TolyaTalamanov TolyaTalamanov changed the title Updated build documentation Updated build documentation for Protopipe Nov 13, 2024
@AsyaPronina
Copy link
Contributor

AsyaPronina commented Dec 6, 2024 •
edited
Loading

Hello, please fix the headings:
image

Please take in mind, that guide for G-API build was previously(link) in "Prerequisites" section, so its guide can be reused in "In-plugin build" and "Standalone build". To do something like this I suggest to make Step 5.
image
as different guide with heading as in previous "In-plugin build" but named like "Protopipe build inside of OpenVINO". And, then, "Standalone build" can be renamed to "Protopipe standalone build".

AsyaPronina
AsyaPronina reviewed Dec 11, 2024
View reviewed changes
Copy link
Contributor

@AsyaPronina AsyaPronina left a comment

Choose a reason for hiding this comment

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

Thanks a lot for the efforts!

src/plugins/intel_npu/tools/protopipe/README.md Outdated Show resolved Hide resolved
src/plugins/intel_npu/tools/protopipe/README.md Outdated Show resolved Hide resolved
src/plugins/intel_npu/tools/protopipe/README.md Outdated Show resolved Hide resolved
src/plugins/intel_npu/tools/protopipe/README.md Outdated Show resolved Hide resolved
AsyaPronina
AsyaPronina approved these changes Dec 12, 2024
View reviewed changes
Copy link
Contributor

@AsyaPronina AsyaPronina left a comment

Choose a reason for hiding this comment

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

Thanks a lot for your efforts! Lets merge it!

@AsyaPronina
Copy link
Contributor

build_jenkins

@AsyaPronina AsyaPronina added this pull request to the merge queue Dec 12, 2024
Merged via the queue into openvinotoolkit:master with commit 69256c8 Dec 12, 2024
141 checks passed
11happy pushed a commit to 11happy/openvino that referenced this pull request Dec 23, 2024
@SeptimiuIoachimNeagaIntel @AsyaPronina @11happy
Updated build documentation for Protopipe (openvinotoolkit#27522)
190de01 
### Details:
 - Update build documentation for protopipe

---------

Co-authored-by: Anastasiya(Asya) Pronina <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@AsyaPronina AsyaPronina AsyaPronina approved these changes

@zKulesza zKulesza Awaiting requested review from zKulesza zKulesza is a code owner automatically assigned from openvinotoolkit/openvino-docs-maintainers

Assignees
No one assigned
Labels
category: docs OpenVINO documentation category: NPU OpenVINO NPU plugin ExternalPR External contributor
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@SeptimiuIoachimNeagaIntel @AsyaPronina @sys-openvino-ci