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

[DOCS] Mac install added enter directory command #17897

Merged
merged 2 commits into from
Jun 8, 2023
Merged

[DOCS] Mac install added enter directory command #17897

merged 2 commits into from
Jun 8, 2023

Conversation

tsavina
Copy link
Contributor

@tsavina tsavina commented Jun 6, 2023 •
edited by ilya-lavrenov
Loading

Details:

  • item1
  • ...

Tickets:

@tsavina tsavina added the category: docs OpenVINO documentation label Jun 6, 2023
@tsavina tsavina requested a review from ilya-lavrenov June 6, 2023 09:11
@tsavina tsavina requested a review from a team as a code owner June 6, 2023 09:11
@tsavina tsavina requested review from zKulesza and removed request for a team June 6, 2023 09:11
ilya-lavrenov
ilya-lavrenov reviewed Jun 6, 2023
View reviewed changes
docs/install_guides/installing-openvino-from-archive-macos.md
@@ -93,6 +93,7 @@ You must update several environment variables before you can compile and run Ope

.. code-block:: sh

cd /opt/intel/openvino_2023
Copy link
Contributor

Choose a reason for hiding this comment

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

it's not required since you use absolute paths below.

Copy link
Contributor

@totoka-intel totoka-intel Jun 6, 2023 •
edited
Loading

Choose a reason for hiding this comment

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

On MacOS 10.15 the setupvars.sh script throws warnings when called from other directory (please see the
case 112739):

[setupvars.sh] WARNING: Can not find OpenVINO Python binaries by path /Users/test/python
[setupvars.sh] WARNING: OpenVINO Python environment does not set properly
[setupvars.sh] OpenVINO environment initialized

The easiest workaround is to enter to the openvino_2023 directory as done in this PR.

Copy link
Contributor

Choose a reason for hiding this comment

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

ok, but can we fix the original issue?

Copy link
Contributor

Choose a reason for hiding this comment

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

We could check the reason of this behavior - I have observed this on MacOS 10.15 only (It does not reproduces on MacOS 12), so it seems that it is OS related. But I will agree - the best solution would be to provide fix for setupvars.sh

ilya-lavrenov
ilya-lavrenov approved these changes Jun 8, 2023
View reviewed changes
Copy link
Contributor

@ilya-lavrenov ilya-lavrenov left a comment

Choose a reason for hiding this comment

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

Approved, but don't close the ticket.
We need to find a proper solution.

@ilya-lavrenov ilya-lavrenov added this to the 2023.0 milestone Jun 8, 2023
@tsavina tsavina merged commit 41b4fd1 into openvinotoolkit:releases/2023/0 Jun 8, 2023
tsavina added a commit to tsavina/openvino that referenced this pull request Jun 12, 2023
@tsavina
add enter dir (openvinotoolkit#17897)
cab09b3
tsavina added a commit to tsavina/openvino that referenced this pull request Jun 13, 2023
@tsavina
add enter dir (openvinotoolkit#17897)
4d5c728
tsavina added a commit that referenced this pull request Jun 13, 2023
@tsavina
[DOCS] Port docs (#18032)
ec2db81 
* add enter dir (#17897)

* add tabs (#18007)
t-jankowski pushed a commit to t-jankowski/openvino that referenced this pull request Jun 13, 2023
@tsavina @t-jankowski
[DOCS] Port docs (openvinotoolkit#18032)
b5af227 
* add enter dir (openvinotoolkit#17897)

* add tabs (openvinotoolkit#18007)
alvoron pushed a commit to alvoron/openvino that referenced this pull request Jun 21, 2023
@tsavina @alvoron
[DOCS] Port docs (openvinotoolkit#18032)
7c41c41 
* add enter dir (openvinotoolkit#17897)

* add tabs (openvinotoolkit#18007)
@tsavina tsavina deleted the mac_install branch December 5, 2023 16:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@totoka-intel totoka-intel totoka-intel left review comments

@ilya-lavrenov ilya-lavrenov ilya-lavrenov 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
Projects
None yet
Milestone
2023.0
Development

Successfully merging this pull request may close these issues.
3 participants
@tsavina @ilya-lavrenov @totoka-intel