-
Notifications
You must be signed in to change notification settings - Fork 708
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
Fix the imports to fit in OpenVINO 2023.3 #1756
Conversation
β¦no longer available.
@adrianboguszewski, can you have a look at this please? |
Do we need to update |
OpenVINO 2023.3.0 (the current minimum supported version) also seems to support openvino.tools.ovc without issues. Since my tests worked well, updating is likely not necessary for this case. |
@prob1995, pre-commit checks are failing due to some formatting issues. Can you run the following in your terminal # Install the dev requirements
pip install -r requirements/dev.txt
# run the pre-commit checks
pre-commit run -a |
@samet-akcay , I accidentally deleted a line break. It's fixed now! Thanks. |
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
π Description
β¨ Changes
Select what type of change your PR is:
β Checklist
Before you submit your pull request, please make sure you have completed the following steps:
For more information about code review checklists, see the Code Review Checklist.