Skip to content
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

Remove OpenVINO ONNX opset<=12 check #7398

Merged
merged 1 commit into from
Apr 12, 2022
Merged

Remove OpenVINO ONNX opset<=12 check #7398

merged 1 commit into from
Apr 12, 2022

Conversation

glenn-jocher
Copy link
Member

@glenn-jocher glenn-jocher commented Apr 12, 2022

No longer needed.

πŸ› οΈ PR Summary

Made with ❀️ by Ultralytics Actions

🌟 Summary

Simplification of export conditions in export.py.

πŸ“Š Key Changes

  • Removed the line of code that forces the OpenVINO model export to use ONNX opset version 12.

🎯 Purpose & Impact

  • The goal is to streamline the export process by not explicitly limiting the OpenVINO export to a specific ONNX opset.
  • Impact to users includes potentially greater flexibility and compatibility with various versions of OpenVINO without being restricted to opset 12.
  • Developers may see a simplified codebase, making maintenance and further development easier.
  • Users may need to ensure their OpenVINO setup is compatible with the opset version used by YOLOv5 for successful model deployment.

@glenn-jocher glenn-jocher merged commit 5333b55 into master Apr 12, 2022
@glenn-jocher glenn-jocher deleted the update/export branch April 12, 2022 12:57
@glenn-jocher glenn-jocher self-assigned this Apr 12, 2022
BjarneKuehl pushed a commit to fhkiel-mlaip/yolov5 that referenced this pull request Aug 26, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant