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.

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

Update image-classification benchmark setup #273

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

rahulchaphalkar
Copy link

Addresses #267
Combines 2-step setup into 1-step, now only source ./benchmarks/image-classification/setup.sh is needed.
Updated Readme to reflect this.
@jlb6740

@jlb6740
Copy link
Collaborator

jlb6740 commented Aug 15, 2024

@rahulchaphalkar When I tried to run this, I got this error:

"[setupvars.sh] ERROR: Unsupported Python version. Please install one of Python 3.6 - 3.9 (64-bit) from https://www.python.org/downloads/"

This is coming from the OpenVino download where I usually just go in and comment out the lines myself. But it got me thinking if this is the latest version of OpenVino that will work here?

@rahulchaphalkar
Copy link
Author

But it got me thinking if this is the latest version of OpenVino that will work here?

The openvino version here is old, 2022.x, as is the wasi-nn bindings version, 0.2.1. I haven't really updated anything in this PR, just tried to smooth over some setup script issues with openvino.

"[setupvars.sh] ERROR: Unsupported Python version. Please install one of Python 3.6 - 3.9

As you pointed out this seems to be coming from setupvars.sh script from openvino. I think this should pop up irrespective of the changes in this PR right? Seems like it just needs an updated python to set up libopenvino.so path and stuff.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants