-
Notifications
You must be signed in to change notification settings - Fork 39
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 openvino workflows #30
Conversation
Signed-off-by: Xin Liu <[email protected]>
Signed-off-by: Xin Liu <[email protected]>
Hello, I am a code review bot on flows.network. Here are my reviews of code commits in this PR. Summary: The patch includes updates to the installation script for OpenVINO and the workflows. The key changes involve updating the OpenVINO version and modifying the PATH environment variable. There are several potential problems identified in the patch. Firstly, there is no error handling or validation for the wget command used for downloading the GPG key file. This may lead to undetected issues during the download process. Secondly, there is no check or handling for the success of the "apt-key add" command, which may cause problems if the GPG key is not added successfully. Additionally, there is no verification step for the success of the "apt update" command, potentially leading to issues with package repository updates. Finally, there is no explanation provided for why the PATH environment variable is modified to include "/usr/lib/x86_64-linux-gnu", which should be clarified. Two important findings in the patch are the removal of OPENVINO_VERSION and OPENVINO_YEAR variables from the "Install OpenVINO" step, with no clear explanation provided. This may require further investigation. Additionally, the updated VERSION environment variable in the "Install WasmEdge with Wasi-NN OpenVINO plugin" step should be evaluated for compatibility and any potential breaking changes. In summary, it is recommended to address the potential problems identified, provide clarification on the reasons for the changes, and ensure compatibility and stability in the updated workflow. DetailsCommit 83f3f2a666f53e31cbf010907df7231b8224a52aKey changes in the patch:
Potential problems:
These are the most important findings in the patch. It is recommended to address the potential problems mentioned above to ensure a smooth and reliable installation process for OpenVINO. Commit 8cb98bf3618f76ecc22b543c7df73d6b9460765dKey Changes:
Potential Problems:
Overall, further clarification is needed on the reasons for the changes and any potential impact they may have on the project. |
@q82419 Could you please help review this PR? Thanks a lot! |
In this PR, the
build_openvino_mobilenet
andbuild_openvino_road_seg_adas
workflows are updated to useWasmEdge 0.13.4
. In addition, theinstall_openvino
script is improved.