Skip to content
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

Introduction spinalcordtoolbox python setup file #1243

Merged
merged 4 commits into from
Apr 7, 2017
Merged

Conversation

peristeri
Copy link
Contributor

Spinalcordtoolbox will be added to the python's system path by running

python setup.py install

or for setting up the development environment by running

python setup.py develop

This does not change the installation process of spinalcordtoolbox

Spinalcordtoolbox will be added to the python's system path by running

    python setup.py install

or for setting up the development environment by running

    python setup.py develop

This does not change the installation process of spinalcordtoolbox
@peristeri peristeri added fix:patch installation category: install_sct or pip/setup.py labels Apr 7, 2017
@jcohenadad
Copy link
Member

I successfully ran python setup.py develop. It used my default Python though (and it updated other modules, including scipy, which might be annoying for people). I guess the future plan is NOT to interact with the user's native Python but to use SCT's own Python?

setup.py Outdated

setup(
name='spinalcordtoolbox',
description='Library of analysis tools for the MRI of the spinal cord',
Copy link
Member

Choose a reason for hiding this comment

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

Library of analysis tools for MRI of the spinal cord

setup.py Outdated
'Programming Language :: Python :: 2',
'Programming Language :: Python :: 2.7',
],
keywords='Image MRI spinal cord',
Copy link
Member

Choose a reason for hiding this comment

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

keywords='Magnetic Resonance Imaging MRI spinal cord analysis template'

@peristeri
Copy link
Contributor Author

The wording of my message might be misleading. I assumed that the user activated the conda python that was installed with SCT. What is planned in the future is the complete integration of SCT in a python environment.

@peristeri peristeri merged commit c27a70f into master Apr 7, 2017
@peristeri peristeri deleted the sct-setup branch April 7, 2017 20:11
@peristeri peristeri changed the title Introduction to spinalcordtoolbox setup Introduction spinalcordtoolbox python setup file Apr 20, 2017
@peristeri peristeri added this to the Release v3.0.2 milestone Apr 20, 2017
@peristeri peristeri added the enhancement category: improves performance/results of an existing feature label Apr 21, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement category: improves performance/results of an existing feature installation category: install_sct or pip/setup.py
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants