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

suggested tedana conda environmentl yml #1111

Closed
rcwelsh opened this issue Jul 18, 2024 · 5 comments · Fixed by #1113
Closed

suggested tedana conda environmentl yml #1111

rcwelsh opened this issue Jul 18, 2024 · 5 comments · Fixed by #1113

Comments

@rcwelsh
Copy link

rcwelsh commented Jul 18, 2024

I posted this on neurostars and someone (Steven Meisler, Penn Med postdoc) suggested I drop on tedana github as well. I did not change the distribution, so not going to do a PR, but just open as an issue. I would highly suggest this going into the readdoc.

name: tedana_nox
dependencies:
  - python>=3.8
  - pip
  - mdp
  - numpy
  - scikit-learn
  - scipy
  - pip:
    - mapca
    - nilearn
    - nibabel
    - tedana
@tsalo
Copy link
Member

tsalo commented Jul 18, 2024

The list of dependencies and required versions are in the pyproject.toml file, so I'd prefer not to include a separate list of requirements unless absolutely necessary. What about including information about creating an environment in the installation instructions instead? Something like:

conda create -n tedenv python=3.10 pip
conda activate tedenv
pip install tedana

@rcwelsh
Copy link
Author

rcwelsh commented Jul 18, 2024

Then the readthedocs needs to be corrected, because the things I list above in the yml (except for mdp which is from an older version of tedana) are in the online instructions, those say the user will need to install the packages:

https://tedana.readthedocs.io/en/stable/installation.html

It will be helpful to end-users (such as me) for the documentation to reflect what is actually needed then.

I'd also suggest a downloadable yml as well as what you have above.

@tsalo
Copy link
Member

tsalo commented Jul 18, 2024

Absolutely, we should remove the (possibly outdated) list of dependencies from the installation instructions and simply link to the pyproject.toml file for the list of dependencies and any version restrictions that might apply.

EDIT: I disagree that compiling an independent list would be beneficial. It introduces an opportunity for the different lists to drift from one another (e.g., if the devs forget to update the documentation- especially given that dependabot regularly opens pull requests that affect the pyproject.toml, but not other files).

@rcwelsh
Copy link
Author

rcwelsh commented Jul 18, 2024

What ever it takes so that end-users can install and use without too much hassle.

Good software is only as good as the usability and success of the package is measured (in part) by the size of the end-user population. Removing as many barriers to use will keep it on that path. Not all users are python programmers.

@handwerkerd
Copy link
Member

@rcwelsh We should definitely update the documentation, but I do want to confirm that the three lines @tsalo mentions above ( #1111 (comment) ) works for you and lets you install tedana. If you are installing tedana in an existing python environment, then just pip install tedana should be all someone needs. If so, that's what we'll put in the documentation.

End-users shouldn't need to worry about dependencies because pip should install any missing ones.

Do you have suggestions for making this simpler?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants