fix: OpenVINO optimization notebook deployment flow in 003 notebook#184
Conversation
maxxgx
left a comment
There was a problem hiding this comment.
Thanks for the PR.
Do you find these tutorials helpful? I was thinking we could reduce them, as they have quite a bit of overlap. For example, the collect_train_deploy notebook covers most of the topics.
Also, I believe it will be hard to keep this branch up to date, since breaking changes are quite regular while we're still in early development.
| "Clone the OpenVINO Physical AI repo and install a pinned revision.\n", | ||
| "\n", | ||
| "> **Why pin the revision?** The OpenVINO Physical AI APIs are evolving quickly. This notebook depends on a tested `physicalai` runtime revision, so the install cell below checks out a known-good commit before installing the package in editable mode. Update `PHYSICALAI_COMMIT` only after re-validating the rest of this notebook." |
There was a problem hiding this comment.
I think these tutorial were written before release. Now there is no need to clone the physicalai repo since it's already available on pypi, so you could pin a released version
There was a problem hiding this comment.
Yes, just found the formal release version. Will modify the commit number to the released version.
Thank you for your comments! |
Thank you for your feedback. I think we still improvements before merging it to main branch:
|
Thanks for the feedback. I updated 003 to use the released PyPI package flow end-to-end instead of relying on a cloned local physicalai repo. The hardware extras are now installed from the pinned release as well. I also replaced the personal Hugging Face repos with the official OpenVINO Physical AI collection model ( For the broader 004 cleanup and browser-based simulation direction, I'll submit new PR to fix the issues. For the 002 notebook suggestion, yes, I think we could remove it and point users to documentation as you suggested. But for now, I think we need to show the tutorials repo first and could improve it step by step, so maybe this notebook could be removed later? |
Summary
Fixed issues in 003_OpenVINO_Optimization.ipynb, including: