Maker #27
Merged
Maker #27
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Close #15 Began with dreaded cv2 not found. Having earned agony earlier in this project, I knew that the problem was most likely with running setupvars and init OpenVINO env. Log in with a shell (as root, which prompted the run_root addition) and running 'source /opt/intel/openvino/bin/setupvars.sh' allowed me to then run the classification sample. Actually, I had to unearth some dependencies first, such as libsm6, libgtk... all the ones now included in the first RUN in Dockerfile. The 'source' command in bash essentially makes variables and things set when the script is executed available to later processes. Simply calling setupvars.sh didn't solve the problem because then nothing was changed when I ran inference. However, source is not included in sh, which is Docker default. Fortunately, '.' served enough of the same purpose. I opted to switch to . instead of using SHELL in the Dockerfile to switch the default. I also added a '-l' to the CMD. This is to make sure that .bashrc gets sourced when going into an interactive shell. fatal: Invalid path '/opt/intel/openvino_2019.2.242/bin/setupvars.sh allowed me to then run the classification sample. Actually, I had to unearth some dependencies first, such as libsm6, libgtk... all the ones now included in the first RUN in Dockerfile. The source command in bash essentially makes variables and things set when the script is executed available to later processes. Simply calling setupvars.sh didn't solve the problem because then nothing was changed when I ran inference. However, source is not included in sh, which is Docker default. Fortunately, . served enough of the same purpose. I opted to switch to . instead of using SHELL in the Dockerfile to switch the default. I also added a -l to the CMD. This is to make sure that .bashrc gets sourced when going into an interactive shell.
This reduces the size of docker context, which reduces build times.
|
Close #25 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Manual testing with makefile on CPU works. Will move to Raspberry Pi after this merge.