Skip to content

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.

1 participant