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

[Doc] Update README for PIP install with correct validation command and not… #4791

Merged
merged 1 commit into from
Mar 29, 2021
Merged

[Doc] Update README for PIP install with correct validation command and not… #4791

merged 1 commit into from
Mar 29, 2021

Conversation

helena-intel
Copy link
Contributor

Discussed with @ryanloney and @mryzhov .

  • Change command to verify openvino-dev installation from python -c "pot -h" to pot -h
  • Add note that Mac/Linux users may need to type python3 instead of python (this is not always the case - it depends on how they configured Python).
  • Made "Activate Virtual Environment" a separate step, so that there is 1 step per step, and you don't accidentally miss a sub-step.

@helena-intel helena-intel requested a review from a team as a code owner March 15, 2021 18:25
@openvino-pushbot openvino-pushbot added the category: docs OpenVINO documentation label Mar 15, 2021
@ryanloney
Copy link
Contributor

Thank you @helena-intel. This looks good.

@mryzhov
Copy link
Contributor

mryzhov commented Mar 16, 2021

@andrew-zaytsev we are waiting for your review :)
@helena-intel could you please also create PR for the release branch (2021.3)

@helena-intel helena-intel mentioned this pull request Mar 16, 2021
Closed
@helena-intel
Copy link
Contributor Author

@helena-intel could you please also create PR for the release branch (2021.3)

Done: #4819

@mryzhov mryzhov added this to the 2021.3 milestone Mar 16, 2021
@mryzhov mryzhov changed the title Update README for PIP install with correct validation command and not… [Doc] Update README for PIP install with correct validation command and not… Mar 16, 2021
@helena-intel helena-intel mentioned this pull request Mar 16, 2021
Merged
@ilya-lavrenov ilya-lavrenov modified the milestones: 2021.3, 2021.4 Mar 24, 2021
@ryanloney
Copy link
Contributor

When is this getting merged?

@ababushk ababushk merged commit 8ad8b6b into openvinotoolkit:master Mar 29, 2021
luo-cheng2021 pushed a commit to luo-cheng2021/openvino that referenced this pull request Apr 7, 2021
@helena-intel @luo-cheng2021
Update README for PIP install with correct validation command and not…
7beb12c 
…e about python/python3 (openvinotoolkit#4791)
mryzhov pushed a commit to mryzhov/openvino that referenced this pull request Apr 23, 2021
@helena-intel @mryzhov
Update README for PIP install with correct validation command and not…
01a98db 
…e about python/python3 (openvinotoolkit#4791)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mryzhov mryzhov mryzhov approved these changes

@andrew-zaytsev andrew-zaytsev andrew-zaytsev approved these changes

Assignees

@mryzhov mryzhov

@andrew-zaytsev andrew-zaytsev

Labels
category: docs OpenVINO documentation
Projects
None yet
Milestone
2021.4
Development

Successfully merging this pull request may close these issues.
7 participants
@helena-intel @ryanloney @mryzhov @andrew-zaytsev @ilya-lavrenov @ababushk @openvino-pushbot