nipreps / sdcflows

Susceptibility Distortion Correction (SDC) workflows for EPI MR schemes
https://www.nipreps.org/sdcflows
Apache License 2.0
32 stars 26 forks source link

CLI run requires mriqc to be installed #468

Open miltoncamacho opened 4 hours ago

miltoncamacho commented 4 hours ago

What happened?

When running sdcflows CLI command, I get an error No module named 'mriqc'.

If I run the container interactively and install the mriqc using pip install mriqc the issue is resolved. I am sure there is an excellent reason why mriqc is necessary, so perhaps this needs to be added to the requirements.txt

What command did you use?

sdcflows /bids-input /bids-output participant --debug

What version of the software are you running?

2.10.0

How are you running this software?

Docker

Is your data BIDS valid?

Yes

Are you reusing any previously computed results?

No

Please copy and paste any relevant log output.

(sdcflows) root@91f133b4405b:/tmp# sdcflows /bids-input /bids-output participant --debug
241030-17:10:41,542 cli IMPORTANT:
     Running SDCFlows 2.10.0:
             * BIDS dataset path: /bids-input.
             * Output folder: /bids-output.
             * Analysis levels: ['participant'].

Traceback (most recent call last):
  File "/opt/conda/envs/sdcflows/bin/sdcflows", line 8, in <module>
    sys.exit(main())
  File "/opt/conda/envs/sdcflows/lib/python3.10/site-packages/sdcflows/cli/main.py", line 178, in main
    "plugin": MultiProcPlugin(
  File "/opt/conda/envs/sdcflows/lib/python3.10/site-packages/niworkflows/engine/plugin.py", line 416, in __init__
    from mriqc import config
ModuleNotFoundError: No module named 'mriqc'

Additional information / screenshots

No response

effigies commented 4 hours ago

Can you upgrade niworkflows?

effigies commented 4 hours ago

Sorry, just to be more explicit, this bug was fixed in https://github.com/nipreps/niworkflows/pull/876 and released in niworkflows 1.11.0.

miltoncamacho commented 3 hours ago

Thanks

see #469