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

Feature/azaytsev/g api docs #3731

Merged
merged 11 commits into from
Dec 25, 2020
Merged

Feature/azaytsev/g api docs #3731

merged 11 commits into from
Dec 25, 2020

Conversation

andrew-zaytsev
Copy link
Contributor

No description provided.

@andrew-zaytsev andrew-zaytsev requested a review from a team as a code owner December 24, 2020 14:13
@andrew-zaytsev andrew-zaytsev added the category: docs OpenVINO documentation label Dec 24, 2020
ilya-lavrenov
ilya-lavrenov reviewed Dec 24, 2020
View reviewed changes
docs/gapi/gapi_intro.md
@@ -0,0 +1,52 @@
# Introduction to OpenCV Graph API (G-API) {#openvino_docs_gapi_gapi_intro}
Copy link
Contributor

Choose a reason for hiding this comment

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

why do we need all this stuff in our docs? It's OpenCV module and all documentation must be there

Copy link
Contributor

Choose a reason for hiding this comment

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

CC @dmatveev @alalek

Copy link
Contributor Author

Choose a reason for hiding this comment

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

The G-API documentation was integrated by request from Dmitry @dmatveev .

Copy link
Contributor

@ilya-lavrenov ilya-lavrenov Dec 24, 2020 •
edited
Loading

Choose a reason for hiding this comment

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

ok, question is redirected to @dmatveev - why do we need this docs inside OpenVINO? Such intro must be a part of OpenCV docs, we can just refer to it, I suppose

Otherwise we can have custom intro into opencv_core, opencv_imgproc and so. Why G-API is an exception?

Copy link
Contributor

Choose a reason for hiding this comment

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

Customers asked for some starting point at the OpenVINO docs as G-API was exposed and served there under the OpenVINO umbrella.

OpenCV docs have this already & the operation/kernel reference will remain there. The OpenVINO-side part will be more Intel/OpenVINO-specific as

  • more *PU support comes in the future;
  • more OMZ samples get ported on G-API.

Comparison with core/imgproc is not very correct as those parts are quite stable, well known, and not evolve that fast as we do.

@ilya-lavrenov ilya-lavrenov added this to the 2021.2 milestone Dec 25, 2020
@evgenytalanin-intel evgenytalanin-intel merged commit b373761 into openvinotoolkit:releases/2021/2 Dec 25, 2020
maxim-kurin pushed a commit to maxim-kurin/openvino that referenced this pull request Jan 26, 2021
@andrew-zaytsev @maxim-kurin
Feature/azaytsev/g api docs (openvinotoolkit#3731)
03e953c 
* Initial commit

* Added content

* Added new content for g-api documentation. Removed obsolete links through all docs

* Fixed layout

* Fixed layout

* Added new topics

* Added new info

* added a note

* Removed redundant .svg
andrew-zaytsev added a commit to andrew-zaytsev/openvino that referenced this pull request Jan 27, 2021
@andrew-zaytsev
Feature/azaytsev/g api docs (openvinotoolkit#3731)
2a6137f 
* Initial commit

* Added content

* Added new content for g-api documentation. Removed obsolete links through all docs

* Fixed layout

* Fixed layout

* Added new topics

* Added new info

* added a note

* Removed redundant .svg
# Conflicts:
#	docs/get_started/get_started_dl_workbench.md
andrew-zaytsev added a commit to andrew-zaytsev/openvino that referenced this pull request Jan 27, 2021
@andrew-zaytsev
Feature/azaytsev/g api docs (openvinotoolkit#3731)
1c1e124 
* Initial commit

* Added content

* Added new content for g-api documentation. Removed obsolete links through all docs

* Fixed layout

* Fixed layout

* Added new topics

* Added new info

* added a note

* Removed redundant .svg
andrew-zaytsev added a commit that referenced this pull request Feb 2, 2021
@andrew-zaytsev @dtrawins
@ntyukaev @aalborov
Feature/azaytsev/cherry picks from 2021 2 (#4069)
235cd56 
* Added info on DockerHub CI Framework

* Feature/azaytsev/change layout (#3295)

* Changes according to feedback comments

* Replaced @ref's with html links

* Fixed links, added a title page for installing from repos and images, fixed formatting issues

* Added links

* minor fix

* Added DL Streamer to the list of components installed by default

* Link fixes

* Link fixes

* ovms doc fix (#2988)

* added OpenVINO Model Server

* ovms doc fixes

Co-authored-by: Trawinski, Dariusz <[email protected]>

* Updated openvino_docs.xml

* Added Intel® Iris® Xe Dedicated Graphics, naming convention info (#3523)

* Added Intel® Iris® Xe Dedicated Graphics, naming convention info

* Added GPU.0 GPU.1

* added info about Intel® Iris® Xe MAX Graphics drivers

* Feature/azaytsev/transition s3 bucket (#3609)

* Replaced https://download.01.org/ links with https://storage.openvinotoolkit.org/

* Fixed links
# Conflicts:
#	inference-engine/ie_bridges/java/samples/README.md

* Benchmarks 2021 2 (#3590)

* Initial changes

* Updates

* Updates

* Updates

* Fixed graph names

* minor fix

* Fixed link

* Implemented changes according to the review changes

* fixed links

* Updated Legal_Information.md according to review feedback

* Replaced  Uzel* UI-AR8 with Mustang-V100-MX8

* Feature/azaytsev/ovsa docs (#3627)

* Added ovsa_get_started.md

* Fixed formatting issues

* Fixed formatting issues

* Fixed formatting issues

* Fixed formatting issues

* Fixed formatting issues

* Fixed formatting issues

* Fixed formatting issues

* Updated the GSG topic, added a new image

* Formatting issues fixes

* Formatting issues fixes

* Formatting issues fixes

* Formatting issues fixes

* Formatting issues fixes

* Formatting issues fixes

* Formatting issues fixes

* Formatting issues fixes

* Formatting issues fixes

* Formatting issues fixes

* Formatting issues fixes

* Formatting issues fixes

* Formatting issues fixes

* Revert "Formatting issues fixes"

This reverts commit c6e6207.

* Replaced to Security section

* doc fixes (#3626)

Co-authored-by: Nikolay Tyukaev <[email protected]>
# Conflicts:
#	docs/IE_DG/network_state_intro.md

* fix latex formula (#3630)

Co-authored-by: Nikolay Tyukaev <[email protected]>

* fix comments ngraph api 2021.2 (#3520)

* fix comments ngraph api

* remove whitespace

* fixes

Co-authored-by: Nikolay Tyukaev <[email protected]>

* Feature/azaytsev/g api docs (#3731)

* Initial commit

* Added content

* Added new content for g-api documentation. Removed obsolete links through all docs

* Fixed layout

* Fixed layout

* Added new topics

* Added new info

* added a note

* Removed redundant .svg
# Conflicts:
#	docs/get_started/get_started_dl_workbench.md

* [Cherry-pick] DL Workbench cross-linking (#3488)

* Added links to MO and Benchmark App

* Changed wording

* Fixes a link

* fixed a link

* Changed the wording

* Links to WB

* Changed wording

* Changed wording

* Fixes

* Changes the wording

* Minor corrections

* Removed an extra point

* cherry-pick

* Added the doc

* More instructions and images

* Added slide

* Borders for screenshots

* fixes

* Fixes

* Added link to Benchmark app

* Replaced the image

* tiny fix

* tiny fix

* Fixed a typo

* Feature/azaytsev/g api docs (#3731)

* Initial commit

* Added content

* Added new content for g-api documentation. Removed obsolete links through all docs

* Fixed layout

* Fixed layout

* Added new topics

* Added new info

* added a note

* Removed redundant .svg

* Doc updates 2021 2 (#3749)

* Change the name of parameter tensorflow_use_custom_operations_config to transformations_config

* Fixed formatting

* Corrected MYRIAD plugin name

* Installation Guides formatting fixes

* Installation Guides formatting fixes

* Installation Guides formatting fixes

* Installation Guides formatting fixes

* Installation Guides formatting fixes

* Installation Guides formatting fixes

* Installation Guides formatting fixes

* Installation Guides formatting fixes

* Installation Guides formatting fixes

* Fixed link to Model Optimizer Extensibility

* Fixed link to Model Optimizer Extensibility

* Fixed link to Model Optimizer Extensibility

* Fixed link to Model Optimizer Extensibility

* Fixed link to Model Optimizer Extensibility

* Fixed formatting

* Fixed formatting

* Fixed formatting

* Fixed formatting

* Fixed formatting

* Fixed formatting

* Fixed formatting

* Fixed formatting

* Fixed formatting

* Fixed formatting

* Fixed formatting

* Updated IGS, added links to Get Started Guides

* Fixed links

* Fixed formatting issues

* Fixed formatting issues

* Fixed formatting issues

* Fixed formatting issues

* Move the Note to the proper place

* Removed optimization notice
# Conflicts:
#	docs/ops/detection/DetectionOutput_1.md

* minor fix

* Benchmark updates (#4041)

* Link fixes for 2021.2 benchmark page  (#4086)

* Benchmark updates

* Fixed links

Co-authored-by: Trawinski, Dariusz <[email protected]>
Co-authored-by: Nikolay Tyukaev <[email protected]>
Co-authored-by: Nikolay Tyukaev <[email protected]>
Co-authored-by: Alina Alborova <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@dmatveev dmatveev dmatveev left review comments

@ntyukaev ntyukaev ntyukaev approved these changes

@ilya-lavrenov ilya-lavrenov ilya-lavrenov approved these changes

Assignees
No one assigned
Labels
category: docs OpenVINO documentation
Projects
None yet
Milestone
2021.2
Development

Successfully merging this pull request may close these issues.
5 participants
@andrew-zaytsev @dmatveev @ilya-lavrenov @ntyukaev @evgenytalanin-intel