-
Notifications
You must be signed in to change notification settings - Fork 7
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
chore(deps): update dependencies #261
Conversation
Confirmed
this would not show if |
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.
Why can't the library itself install these dependencies?
I have learned what they are! They are called library extras, when they're installed as like Dustin filed a feature request before: pypa/setuptools#1503, but tl;dr is that with current state of python packaging, it is not possible to ask for users to install all or some extras as default when you simply install |
I guess I'm wondering if the In other words, if this was a third-party library, it wouldn't make sense for it to require |
Ah, they can! I'm just adding support for this in the mega generator in case I have to regenerate all the docs and didn't want to miss out on |
Will you add a comment explaining that? Perhaps that regeneration script could install those dependencies separately? |
Done! Please take a look again. |
Adding specific installation for
aiplatform[prediction]
. Also making therequirements.in
file look nicer :)Fixes #260.