-
Notifications
You must be signed in to change notification settings - Fork 86
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
Documentation: fix install error for Python 3.6 #1322
Conversation
Codecov Report
@@ Coverage Diff @@
## main #1322 +/- ##
=======================================
Coverage 99.95% 99.95%
=======================================
Files 213 213
Lines 13560 13560
=======================================
Hits 13553 13553
Misses 7 7 Continue to review full report at Codecov.
|
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.
Looks good. Just a comment on wording.
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.
Looks good to me!
docs/source/install.ipynb
Outdated
@@ -16,6 +16,8 @@ | |||
"pip install evalml\n", | |||
"```\n", | |||
"\n", | |||
"For Python versions 3.6.X you must install `pip >= 18.0` and `setuptools > 40.0.0` in order to install evalml.\n", |
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.
What happens when a user runs pip install evalml
but has pip below this version, or setuptools below this version? Should we update our requirements with this?
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.
(We don't currently list pip or setuptools as dependencies)
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.
If the user is using 3.6, they run into the shap
error that caused this issue. I figured adding it into the install docs would be easiest since pip
and setuptools
aren't in our requirements docs
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.
Not sure if this is 100% relevant, but I'd like point out that creating a virtual env with 3.6 directly means we don't run into this issue:
python3.6 -m pip install virtualenv
python3.6 -m virtualenv my-venv --python=python3.6
source my-venv/bin/activate
python3.6 -m pip install shap (or -r requirements.txt in my case)
This works fine. Could be because of the versions of pip and setuptools that are installed with the virtualenv but didn't dig too deeply.
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.
Hmm yeah that's interesting. Looking further into it, when I used @angela97lin's approach, pip == 20.2.3
and setuptools == 50.3.1
, as opposed to using pyenv to create a virtualenv for Python3. I think both approaches are valid since the versions of pip
and setuptools
are high enough. I think with the current message that I added, it should be enough to allow users to install evalml
without issues. @dsherry Do you think these two should be added to our dependencies, or would leaving/updating the message be enough?
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.
@bchen1116 so as I just wrote on the issue, when I create a pyenv with 3.6.10
and run pip install .
, it passes. But that fails with 3.6.0
.
Can we update this to say "For older Python versions (3.6.0 in particular), you must install pip>=18.0
and setuptools>=40.0.0
before installing evalml."
Great find!
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.
Approved with one comment. Thanks!
docs/source/install.ipynb
Outdated
@@ -16,6 +16,8 @@ | |||
"pip install evalml\n", | |||
"```\n", | |||
"\n", | |||
"For Python versions 3.6.X you must install `pip >= 18.0` and `setuptools > 40.0.0` in order to install evalml.\n", |
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.
@bchen1116 so as I just wrote on the issue, when I create a pyenv with 3.6.10
and run pip install .
, it passes. But that fails with 3.6.0
.
Can we update this to say "For older Python versions (3.6.0 in particular), you must install pip>=18.0
and setuptools>=40.0.0
before installing evalml."
Great find!
Fix #1242
Doc is here
Add documentation for installing EvalML for python 3.6. Avoids the
shap
error