Skip to content
This repository has been archived by the owner on Nov 17, 2023. It is now read-only.

refine Nano setup directions #15524

Merged
merged 2 commits into from
Jul 18, 2019
Merged

Conversation

aaronmarkham
Copy link
Contributor

@aaronmarkham aaronmarkham commented Jul 12, 2019

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

@aaronmarkham
Copy link
Contributor Author

Nudging CI again...

Copy link
Contributor

@IvyBazan IvyBazan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Added minor edits

docs/install/install-jetson.md Outdated Show resolved Hide resolved
docs/install/install-jetson.md Outdated Show resolved Hide resolved
Copy link

@darlaker darlaker left a 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.

docs/install/install-jetson.md Show resolved Hide resolved
export MXNET_HOME=$(pwd)
echo "export PYTHONPATH=$MXNET_HOME/python:$PYTHONPATH" >> ~/.rc
source ~/.rc
sudo rm /usr/local/cuda

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. :)

Copy link
Contributor Author

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.

docs/install/install-jetson.md Outdated Show resolved Hide resolved
@aaronmarkham aaronmarkham merged commit dd14c81 into apache:master Jul 18, 2019
anirudhacharya pushed a commit to anirudhacharya/mxnet that referenced this pull request Aug 20, 2019
* refine directions

* resolving feedback on instructions
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants