Do not clean up FA for degenerate tensors within DTI node

[Oscar Esteban]

What happened?

We are immediately cleaning up the FA:

mriqc/mriqc/interfaces/diffusion.py

Lines 472 to 473 in 61b59e0

 fa_data[np.isnan(fa_data)] = 0 
 fa_data = np.clip(fa_data, 0, 1) 

and that precludes that we calculate the percentage of degenerate tensors as an IQM or plot them highlighted for visual QA/QC.

Methods that require a "massaged" FA (e.g., to calculate the CC mask) should take these two easy steps on their own.

/cc @arokem @teresamg

What command did you use?

-m dwi

What version of the software are you running?

master

How are you running this software?

Local installation ("bare-metal")

Is your data BIDS valid?

Yes

Are you reusing any previously computed results?

No

Please copy and paste any relevant log output.

No response

Additional information / screenshots

No response

[Ariel Rokem]

What's the point of the FA file? Where is it used subsequent to saving?

[Oscar Esteban]

Generating the reports, mostly.

…

On Fri, Mar 22, 2024, 21:07 Ariel Rokem ***@***.***> wrote: What's the point of the FA file? Where is it used subsequent to saving? — Reply to this email directly, view it on GitHub <#1215 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AAESDRXOC3YNNDDHNXL3Q6LYZSFQHAVCNFSM6AAAAABFCWPDFOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAMJVHAZDKOJWG4> . You are receiving this because you authored the thread.Message ID: ***@***.***>