-
Notifications
You must be signed in to change notification settings - Fork 6.8k
Conversation
Nudging CI again... |
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.
Added minor edits
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.
This looks great! Added some suggestions from the walkthrough, but feel free to disregard since the places where I got stuck should not impact users with more Linux expertise.
export MXNET_HOME=$(pwd) | ||
echo "export PYTHONPATH=$MXNET_HOME/python:$PYTHONPATH" >> ~/.rc | ||
source ~/.rc | ||
sudo rm /usr/local/cuda |
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.
Maybe add a line here that describes how these commands will remove the directory and then replace it by creating a symlink. But Linux users may not need this called out. :)
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 was always a symlink, not an actual folder. I just briefly freaked out when I thought I deleted the cuda files.
* refine directions * resolving feedback on instructions
Description
After another run-through on a fresh Nano, plus trying out these new pip wheels, we found some refinements that should make the setup go a little more cleanly.
Thanks for the assist @darlaker