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

Using YOLOv5 with Pillow-SIMD #7506

Closed
glenn-jocher opened this issue Apr 20, 2022 Discussed in #7472 · 1 comment · Fixed by #7505
Closed

Using YOLOv5 with Pillow-SIMD #7506

glenn-jocher opened this issue Apr 20, 2022 Discussed in #7472 · 1 comment · Fixed by #7505

Comments

@glenn-jocher
Copy link
Member

Discussed in #7472

Originally posted by jkocherhans April 18, 2022
Hello! I’d like to use Pillow-SIMD (a drop-in replacement for Pillow) along with YOLOv5, but as soon as I import my model with torch.hub.load, check_requirements notices that Pillow isn’t installed and installs it automatically over the top of Pillow-SIMD. I can think of a few possible ways around this, but they all involve changing YOLOv5’s code.

  1. Stop installing dependencies and print out a warning instead. Basically this would change the install=True default in check_requirements to False. In a production environment, it’s surprising to me that importing code would cause a package to be installed, but I definitely see how the current behavior is helpful for most folks.
  2. Allow users to pass install=False into torch.hub.load, and pass that down to the call to check_requirements.
  3. Special case check_requirements to check for Pillow-SIMD as a fallback for Pillow.

If any combination of those options sound like a good idea to the maintainers, I’m happy to put together a PR.

Thank you!

@glenn-jocher glenn-jocher linked a pull request Apr 20, 2022 that will close this issue
@github-actions
Copy link
Contributor

github-actions bot commented Apr 20, 2022

👋 Hello @glenn-jocher, thank you for your interest in YOLOv5 🚀! Please visit our ⭐️ Tutorials to get started, where you can find quickstart guides for simple tasks like Custom Data Training all the way to advanced concepts like Hyperparameter Evolution.

If this is a 🐛 Bug Report, please provide screenshots and minimum viable code to reproduce your issue, otherwise we can not help you.

If this is a custom training ❓ Question, please provide as much information as possible, including dataset images, training logs, screenshots, and a public link to online W&B logging if available.

For business inquiries or professional support requests please visit https://ultralytics.com or email support@ultralytics.com.

Requirements

Python>=3.7.0 with all requirements.txt installed including PyTorch>=1.7. To get started:

git clone https://github.com/ultralytics/yolov5  # clone
cd yolov5
pip install -r requirements.txt  # install

Environments

YOLOv5 may be run in any of the following up-to-date verified environments (with all dependencies including CUDA/CUDNN, Python and PyTorch preinstalled):

Status

CI CPU testing

If this badge is green, all YOLOv5 GitHub Actions Continuous Integration (CI) tests are currently passing. CI tests verify correct operation of YOLOv5 training (train.py), validation (val.py), inference (detect.py) and export (export.py) on macOS, Windows, and Ubuntu every 24 hours and on every commit.

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 a pull request may close this issue.

1 participant