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

[Runtime] INT8 inference documentation update #5949

Merged
merged 1 commit into from
Jun 22, 2021
Merged

[Runtime] INT8 inference documentation update #5949

merged 1 commit into from
Jun 22, 2021

Conversation

eshoguli
Copy link
Contributor

@eshoguli eshoguli commented Jun 1, 2021

Details:

  • INT8 inference documentation update

Tickets:

  • 56490

@eshoguli eshoguli requested a review from a team as a code owner June 1, 2021 09:06
@eshoguli eshoguli added category: CPU OpenVINO CPU plugin category: docs OpenVINO documentation category: GPU OpenVINO GPU plugin labels Jun 1, 2021
@eshoguli eshoguli added this to the 2021.4 milestone Jun 1, 2021
@MaximProshin
Copy link
Contributor

@AlexKoff88 , please also look at it

@MaximProshin
Copy link
Contributor

@dmitry-gorokhov, @AlexKoff88, @vladimir-paramuzov, can you please review it so that it becomes a part of 2021.4?
@andrew-zaytsev FYI

@ilya-lavrenov ilya-lavrenov removed this from the 2021.4 milestone Jun 11, 2021
@AlexKoff88
Copy link
Contributor

The narrative is corrupted. There is no introduction that explains the purpose of this document.
@MaximProshin, I think you should ask @andrew-zaytsev to review, first.

@MaximProshin
Copy link
Contributor

@andrew-zaytsev, can you help to improve the narrative?

@dmitry-gorokhov
Copy link
Contributor

@eshoguli @MaximProshin @AlexKoff88 Given the documentation freeze today shouldn't we merge current version? At least it removes controversial information

@MaximProshin
Copy link
Contributor

@dmitry-gorokhov, I asked @andrew-zaytsev to help with it. Hopefully it will be merged.

@MaximProshin
Copy link
Contributor

@dmitry-gorokhov , btw, as far as I understand Doc Freeze means that documentation PRs should be created before that date but they can be merged later

@azhogov azhogov merged commit 15ee515 into openvinotoolkit:master Jun 22, 2021
@eshoguli eshoguli mentioned this pull request Jun 28, 2021
Merged
rnugmanx pushed a commit to rnugmanx/openvino that referenced this pull request Aug 26, 2021
@eshoguli
[Runtime] INT8 inference documentation update (openvinotoolkit#5949)
14ad73c
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@azhogov azhogov azhogov approved these changes

Assignees

@dmitry-gorokhov dmitry-gorokhov

@vladimir-paramuzov vladimir-paramuzov

Labels
category: CPU OpenVINO CPU plugin category: docs OpenVINO documentation category: GPU OpenVINO GPU plugin
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
7 participants
@eshoguli @MaximProshin @AlexKoff88 @dmitry-gorokhov @azhogov @ilya-lavrenov @vladimir-paramuzov