Add update/reinstall & training documentation #788
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
mathieuboudreau
added
the
documentation
mathieuboudreau
requested review from
Stoyan-I-A,
ArthurBoschet,
hermancollin and
Nish228
|
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 |
Stoyan-I-A
reviewed
Feb 16, 2024
|
|
||
| From the parent folder of the AxonDeepSeg directory::: | ||
|
|
||
| rm -rf axondeepseg |
There was a problem hiding this comment.
For Windows users, the command will be rmdir axondeepseg
There was a problem hiding this comment.
Is that on the powershell? Would the command be the same on the Anaconda terminal?
There was a problem hiding this comment.
Would the command be the same on the Anaconda terminal?
Yes
mathieuboudreau
changed the title
Stoyan-I-A
requested changes
Apr 14, 2024
|
|
||
| From the parent folder of the AxonDeepSeg directory::: | ||
|
|
||
| rm -rf axondeepseg |
There was a problem hiding this comment.
Here's the instruction box I suggest using
# Mac and Linux
rm -rf axondeepseg
# Windows
rmdir axondeepseg /s
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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