-
Notifications
You must be signed in to change notification settings - Fork 639
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
Don't require scipy for regular use #948
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -26,11 +26,13 @@ def read(fname): | |
license="MIT", | ||
keywords="gpu optimizers optimization 8-bit quantization compression", | ||
url="https://github.com/TimDettmers/bitsandbytes", | ||
install_requires=['scipy'], | ||
packages=find_packages(), | ||
package_data={"": libs}, | ||
install_requires=['torch', 'numpy', 'scipy'], | ||
extras_require={'benchmark': ['pandas', 'matplotlib']}, | ||
install_requires=['torch', 'numpy'], | ||
extras_require={ | ||
'benchmark': ['pandas', 'matplotlib'], | ||
'test': ['scipy'], | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Is There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. The only call to the function is in a test (and it's mentioned in a couple copy-pasted comments too):
I can't find other relevant references to it across all of GitHub either – and anyway, the function is still there for downstream users who might need it, they'll just need |
||
}, | ||
long_description=read("README.md"), | ||
long_description_content_type="text/markdown", | ||
classifiers=[ | ||
|
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.
Are these runtime dependencies at all? The seem to be used only in scripts and are not part of the wheel? Maybe the
benchmark
extra is not needed at all?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.
The idea is that when desiring to run the benchmark scripts, you'd do
to get the regular requirements plus the things required for benchmarking, or if you want that and the test deps,
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.
Doesn’t this also make it part of the public metadata, e.g “pip install bitsandbytes[benchmark]” ?
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.
Yes, it does.