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

Sklearn pip installation broke because of issues with setuptools #24992

Closed
victorosorioparis opened this issue Nov 20, 2022 · 2 comments
Closed
Labels
Bug Needs Triage Issue requires triage

Comments

@victorosorioparis
Copy link

Describe the bug

Apparently because of this other issue pypa/setuptools#3693

Installation of sklearn breaks with pip install right now.

File "/tmp/pip-build-env-mmiaw08q/overlay/lib/python3.10/site-packages/numpy/distutils/log.py", line 4, in <module>
           from distutils.log import Log as old_Log
       ImportError: cannot import name 'Log' from 'distutils.log' (/tmp/pip-build-env-mmiaw08q/overlay/lib/python3.10/site-packages/setuptools/_distutils/log.py)

I've read some other teams sticking to setuptools 65.5.1 for the moment, scikit learn seems to be aiming at 60+

Steps/Code to Reproduce

pip install scikit_learn

Expected Results

No error should be thrown

Actual Results

File "/tmp/pip-build-env-mmiaw08q/overlay/lib/python3.10/site-packages/numpy/distutils/log.py", line 4, in <module>
           from distutils.log import Log as old_Log
       ImportError: cannot import name 'Log' from 'distutils.log' (/tmp/pip-build-env-mmiaw08q/overlay/lib/python3.10/site-packages/setuptools/_distutils/log.py)

Versions

all versions depending on setuptools 60+
@victorosorioparis victorosorioparis added Bug Needs Triage Issue requires triage labels Nov 20, 2022
@eli-schwartz
Copy link

Since #24563 this shouldn't be an issue anymore as numpy.distutils isn't used.

@victorosorioparis
Copy link
Author

Yep, sorry guys, this works if I update the sklearn version to one that includes #24563 - the problem is probably only affecting previous versions but I guess that's fair.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Needs Triage Issue requires triage
Projects
None yet
Development

No branches or pull requests

2 participants