Skip to content

Scope torchvision.utils.draw_bounding_boxes import #8915

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.

Sign up for GitHub

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

Jump to bottom
Merged
glenn-jocher merged 2 commits into from
Aug 9, 2022
Merged

Scope torchvision.utils.draw_bounding_boxes import #8915

glenn-jocher merged 2 commits into from
Aug 9, 2022

Conversation

@glenn-jocher
Copy link
Member

@glenn-jocher glenn-jocher commented Aug 9, 2022 •
edited by UltralyticsAssistant
Loading

Quick fix for #8898 (comment) until a more permanent fix is implemented

🛠️ PR Summary

Made with ❤️ by Ultralytics Actions

🌟 Summary

Update to ClearML logger dependency in YOLOv5 for bounding box visualization.

📊 Key Changes

  • Moved the import statement for draw_bounding_boxes from torchvision.utils inside the try-except block.

🎯 Purpose & Impact

  • 🔁 The change ensures compatibility with specific versions of the torchvision package (enforcing a minimum version of 0.9.0).
  • ✅ Users will benefit from more reliable bounding box visualizations when using ClearML with YOLOv5 as this prevents import errors related to version mismatches.
  • ⏩ Developers can ensure that the necessary prerequisites are met for proper logger functionality, reducing potential issues with dependencies.

@glenn-jocher
Scope torchvision.utils.draw_bounding_boxes import
7c6df2f 
Quick fix for #8898 (comment) until a more permanent fix is implemented
@glenn-jocher glenn-jocher self-assigned this Aug 9, 2022
@glenn-jocher glenn-jocher merged commit 7ab1f50 into master Aug 9, 2022
@glenn-jocher glenn-jocher deleted the fix/clearml_torchvision branch August 9, 2022 18:29
@glenn-jocher glenn-jocher linked an issue Aug 9, 2022 that may be closed by this pull request
ImportError: cannot import name 'draw_bounding_boxes' from 'torchvision.utils' #8898
Closed
2 tasks
@glenn-jocher glenn-jocher mentioned this pull request Aug 9, 2022
Closed
2 tasks
ctjanuhowski pushed a commit to ctjanuhowski/yolov5 that referenced this pull request Sep 8, 2022
@glenn-jocher
Scope torchvision.utils.draw_bounding_boxes import (ultralytics#8915)
243536e 
* Scope `torchvision.utils.draw_bounding_boxes` import

Quick fix for ultralytics#8898 (comment) until a more permanent fix is implemented

* Update clearml_utils.py
@Hojland Hojland mentioned this pull request Oct 17, 2022
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

@glenn-jocher glenn-jocher

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

ImportError: cannot import name 'draw_bounding_boxes' from 'torchvision.utils'

2 participants

@glenn-jocher