Installation instructions do not work for Windows machines

[aryangupta701]

Hello,

I am getting errors while using the command "pip install -e .'[all]'"
Can anyone help me to fix this please, I want to contribute to this project.

Thankyou

[tsalo]

Can you post the traceback?

[aryangupta701]

ERROR: Invalid requirement: '[all]'
I get this error

[tsalo]

I can't reproduce locally, so maybe it's a version or path issue. Let's see if we can diagnose the problem.

• Where are you running this command from? Is it from the tedana directory?
• Is your local copy of tedana up-to-date with main?
• What version of pip are you using?
• What version of Python are you using?

[aryangupta701]

I have copied this command from CONTRUBUTING.md file present in the project
yes my local copy is up to date
my pip version is latest
python version 3.8

[tsalo]

I have copied this command from CONTRIBUTING.md file present in the project

Yes, but in what location on your filesystem did you run the command?

Maybe it's an OS issue. What operating system are you using?

Lastly, can you try pip install -e .[all] (no single quotes around [all]).

[aryangupta701]

yes this worked thanks a lot.

[tsalo]

Are you using a Windows machine by any chance? I'd like to make sure our installation instructions work for all operating systems, so we may need to update them if they don't work for Windows machines.

[aryangupta701]

Yes I am using a windows 11 machine

[tsalo]

Thank you.

@ME-ICA/tedana-devs do any of you use a Windows machine? Can you reproduce the behavior @aryangupta701 reported with pip install -e .'[all]'?

[handwerkerd]

Welcome @aryangupta701 ! I don't have easy access to a Windows computer right now so I can't help directly. This sounds like a really important issue to resolve.

[eurunuela]

My colleague @vinferrer has a Windows I believe. We could have a look on Monday.

[Cristina-Tobias]

I have tried installing tedana both ways in my Windows laptop. It has only worked without the single quotes

[tsalo]

What about pip install -e '.[all]'?

[Cristina-Tobias]

It showed this error message : ERROR: .'[all]' is not a valid editable requirement. It should either be a path to a local project or a VCS URL (beginning with svn+, git+, hg+, or bzr+)

[eurunuela]

What about pip install -e '.[all]'?

That didn't work. We're gonna have to update our docs.

[tsalo]

No, not .'[all]'. '.[all]'.

EDIT: For clarification, I ask because that's the pattern that came up recently in a nilearn PR of mine: nilearn/nilearn#2953 (comment)

[Cristina-Tobias]

Sorry. I have tried it again with '.[all]' and it shows the same error message

[tsalo]

Thank you @Cristina-Tobias, that's really helpful for nilearn too.

@aryangupta701 we can follow up in #846 about how specifically we should update the installation instructions.