-
Notifications
You must be signed in to change notification settings - Fork 179
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
🐌 SNPE env and example fix #860
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the fixes. The lib and bin paths worked fine previously so I am not sure if it's something to do with the new snpe version or our environment differences. But it doesn't hurt to add extra paths.
It seems the shellcheck fail, do we need fix it? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
do we need fix the shellcheck failure?
Thanks! Added the Shebang info for shell script. |
Describe your changes
There are a few issues for current SNPE examples:
python3.6
need to be put visible for system for SNPE cmd runningstr(Path(f"{snpe_root}/python36-env/lib"))
underLD_LIBRARY_PATH
can fix.Checklist before requesting a review
lintrunner -a
(Optional) Issue link