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
Use pip install -e spinalcordtoolbox to gain flexibility #1892
Conversation
…1702) ... and fixup update_bin.sh to be less quirky
Ran tests on this, the installation worked, check_dependencies seem to be fine. |
install_sct
Outdated
@@ -528,7 +520,7 @@ if [ ${SCT_DEV_MODE} ]; then | |||
pip install -e ${SCT_SOURCE} | |||
e_status=$? | |||
else | |||
pip install ${SCT_SOURCE} | |||
pip install -e ${SCT_DIR} |
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.
do we still need to distinguish dev vs. non-dev mode? i.e., maybe we don't need the -z
flag anymore?
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.
If nobody uses it, it could be removed. The only justification for its existence, now, is that with it, we can perform out-of-place installs and edit files that are not in the installation folder (but I have never done that).
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.
I think we can remove it.
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.
OK done.
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.
Thanks!
install: use pip install -e spinalcordtoolbox to gain flexibility Former-commit-id: dd5b094
This PR brings a "solution" for #1702, whereby users can tinker with the running code in all cases:
pip install --upgrade
needed to be ran).Bonus: