-
Notifications
You must be signed in to change notification settings - Fork 1k
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
Automatically install dependencies when package is installed #276
Conversation
@@ -12,4 +18,10 @@ | |||
keywords=['computer vision', 'image processing', 'opencv', 'matplotlib'], | |||
classifiers=[], | |||
scripts=['bin/range-detector'], | |||
install_requires=[ | |||
'numpy', | |||
'scipy', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I have raised a PR to remove scipy dependency. If that gets approved then you might have one less dependency to install.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@farqis When your PR is merged, please update that here and resolve this conversation.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@farqis Was your PR merged (for removing scipy dependency)?
@jrosebr1 Please review my pull request - it will make installing |
@jrosebr1 Did you have a chance to review my PR yet? |
@sohang3112 I no longer maintain this library. Please contact PyImageSearch support. Thank you. |
Who is the current maintainer?
I filled this contact form of PyImageSearch: https://pyimagesearch.com/contact/ . Should I contact them in some other way also? |
@sohang3112 Yes, use the contact form. Thanks for submitting the PR, I hope it's merged 🙏 |
@jrosebr1 I filled the contact form, but I got this email in response:
I'm not going to buy their course just to get my PR merged 🙂 |
Closing this PR, since @jrosebr1 no longer maintains the repo and PyImageSearch did not respond at all. |
All dependencies (NumPy, SciPy, Matplotlib, OpenCV) will be installed automatically when
imutils
is installed.This is done using
setuptools
, sincedistutils
doesn't support automatic installation of dependencies.