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

scipy and numpy version dependency #7638

Closed
1 task done
fei4xu opened this issue Apr 29, 2022 · 2 comments · Fixed by #7646
Closed
1 task done

scipy and numpy version dependency #7638

fei4xu opened this issue Apr 29, 2022 · 2 comments · Fixed by #7646
Labels
question Further information is requested

Comments

@fei4xu
Copy link

fei4xu commented Apr 29, 2022

Search before asking

Question

Hi,

I noticed that in the requirement.txt
https://github.com/ultralytics/yolov5/blob/master/requirements.txt#L5-L10

numpy>=1.18.5
scipy>=1.4.1

and also in the scipy & numpy version compatible table at https://docs.scipy.org/doc/scipy/dev/toolchain.html#numpy

SciPy version Python versions NumPy versions
1.2 2.7, >=3.4, <=3.7 >=1.8.2, <= 1.16.x
1.4 >=3.5, <=3.8 >=1.13.3, <= 1.17.3
1.5 >=3.6, <=3.9 >=1.14.5, <= 1.19.3

if numpy>1.18, scipy should be at least 1.5, so the current version of >=1.4.1 should be change to at lease 1.5, am I right?
(I'm new to scipy and numpy)

Additional

No response

@fei4xu fei4xu added the question Further information is requested label Apr 29, 2022
@glenn-jocher
Copy link
Member

@fei4xu yes I think you are right! I'll make a PR.

@glenn-jocher
Copy link
Member

@fei4xu good news 😃! Your original issue may now be fixed ✅ in PR #7646. To receive this update:

  • Gitgit pull from within your yolov5/ directory or git clone https://github.com/ultralytics/yolov5 again
  • PyTorch Hub – Force-reload model = torch.hub.load('ultralytics/yolov5', 'yolov5s', force_reload=True)
  • Notebooks – View updated notebooks Open In Colab Open In Kaggle
  • Dockersudo docker pull ultralytics/yolov5:latest to update your image Docker Pulls

Thank you for spotting this issue and informing us of the problem. Please let us know if this update resolves the issue for you, and feel free to inform us of any other issues you discover or feature requests that come to mind. Happy trainings with YOLOv5 🚀!

tdhooghe pushed a commit to tdhooghe/yolov5 that referenced this issue Jun 10, 2022
BjarneKuehl pushed a commit to fhkiel-mlaip/yolov5 that referenced this issue Aug 26, 2022
ctjanuhowski pushed a commit to ctjanuhowski/yolov5 that referenced this issue Sep 8, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
question Further information is requested
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants