-
Notifications
You must be signed in to change notification settings - Fork 354
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
docs: update dependency version in contributing readme #8776
docs: update dependency version in contributing readme #8776
Conversation
✅ Deploy Preview for determined-ui canceled.
|
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #8776 +/- ##
==========================================
- Coverage 47.44% 47.44% -0.01%
==========================================
Files 1046 1046
Lines 166888 166888
Branches 2239 2239
==========================================
- Hits 79186 79184 -2
- Misses 87544 87546 +2
Partials 158 158
Flags with carried forward coverage won't be shown. Click here to find out more. |
@@ -118,7 +118,7 @@ source $HOME/.profile | |||
cd determined | |||
python3 -m venv $HOME/.virtualenvs/determined | |||
. $HOME/.virtualenvs/determined/bin/activate | |||
$HOME/.virtualenvs/determined/bin/python3.7 -m pip install --upgrade pip | |||
$HOME/.virtualenvs/determined/bin/python3.9 -m pip install --upgrade pip |
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.
is 3.9 our current min?
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.
It's 3.7 (as listed). The previous instructions have users installing python 3.9 ... whichever version is used I think it should be consistent.
thanks for fixing these! 🚀 |
Description
Webui requires node 20.* to compile. The docs listed multiple version of python; updating it so be consistent.
Test Plan
Run
make all
with the specified dependency versions.Checklist
docs/release-notes/
.See Release Note for details.