search

aramis-lab / clinica

Software platform for clinical neuroimaging studies
http://www.clinica.run/
Other
220 stars 74 forks source link

Logging of `run_dcm2niix` is not consistent with the conversion result #1179

Closed NicolasGensollen closed 4 months ago

NicolasGensollen commented 4 months ago

The run_dcm2niix function from the clinica.iotools.bids_utils module logs a "DICOM to BIDS conversion with dcm2niix failed" message whatever the result from the conversion:

https://github.com/aramis-lab/clinica/blob/a4984bcd2c9ff96e3696d43110955f3375a949c4/clinica/iotools/bids_utils.py#L895-L907

The second log should be something like: "DICOM to BIDS conversion with dcm2niix successful".

This inconsistency was introduced in #1029