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

device_name="MYRIAD" for Intel NCS2 comment #8327

Merged
merged 1 commit into from
Jun 24, 2022
Merged

Conversation

glenn-jocher
Copy link
Member

@glenn-jocher glenn-jocher commented Jun 24, 2022

Display device_name="MYRIAD" for Intel NCS2 option per user question in #8154

πŸ› οΈ PR Summary

Made with ❀️ by Ultralytics Actions

🌟 Summary

Improved argument compatibility in OpenVINO model compilation for YOLOv5.

πŸ“Š Key Changes

  • πŸ›  Removed the unnecessary model= keyword argument during the model compilation call for OpenVINO.
  • 🏷 Added a comment to guide users for Intel Neural Compute Stick 2 (NCS2) usage.

🎯 Purpose & Impact

  • πŸ’‘ Purpose: The simplification of the code makes it cleaner and potentially prevents confusion about the named argument usage when compiling models with OpenVINO.
  • πŸ” Impact: Users running YOLOv5 models on different devices, especially those using Intel's NCS2, may benefit from a smoother setup process with this refined guide comment. Minimal impact on actual performance, but improved code maintainability and user experience.

Display device_name="MYRIAD" for Intel NCS2 option per user question in #8154
@glenn-jocher glenn-jocher self-assigned this Jun 24, 2022
@glenn-jocher glenn-jocher linked an issue Jun 24, 2022 that may be closed by this pull request
1 task
@glenn-jocher glenn-jocher merged commit 724d5b2 into master Jun 24, 2022
@glenn-jocher glenn-jocher deleted the update/ncs2 branch June 24, 2022 12:19
ctjanuhowski pushed a commit to ctjanuhowski/yolov5 that referenced this pull request Sep 8, 2022
Display device_name="MYRIAD" for Intel NCS2 option per user question in ultralytics#8154
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.

YOLOv5 Real Time Detection With OpenVINO
1 participant