-
Notifications
You must be signed in to change notification settings - Fork 32
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
Add update/reinstall & training documentation #788
base: master
Are you sure you want to change the base?
Conversation
Link to documentation that was added: https://axondeepseg--788.org.readthedocs.build/en/788/documentation.html#updating-axondeepseg |
Pull Request Test Coverage Report for Build 9194145887Details
💛 - Coveralls |
|
||
From the parent folder of the AxonDeepSeg directory::: | ||
|
||
rm -rf axondeepseg |
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.
For Windows users, the command will be rmdir axondeepseg
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 that on the powershell? Would the command be the same on the Anaconda terminal?
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.
Would the command be the same on the Anaconda terminal?
Yes
|
||
From the parent folder of the AxonDeepSeg directory::: | ||
|
||
rm -rf axondeepseg |
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.
Here's the instruction box I suggest using
# Mac and Linux
rm -rf axondeepseg
# Windows
rmdir axondeepseg /s
Checklist
Description
Our old update/reinstall instructions would occasionally lead to users having broken installs (see: #784), and so asking the users (note: not devs) to simply delete the repo and conda env and start over is likely the cleanest instructions we can provide.
I've also updated the Training section with two notices: one that IVADOMED is being discontinued, and another informing people that ADS is able to train models with their data (with conditions)
Linked issues