[consistency] Consider including _ToMNI substring

[tashrifbillah]

This line can have _FA_ToMNI just like this line.

We know _ToMNI was dropped from multi-shell for conciseness. Should we consider bringing it back?

[tashrifbillah]

Or do we need to drop _ToMNI from single-shell because we are actually warping to template space, not standard space?

[tashrifbillah]

Inconsistency within single-shell:
compare lines:
https://github.com/pnlbwh/dMRIharmonization/blob/master/lib/debug_fa.py#L95
versus
https://github.com/pnlbwh/dMRIharmonization/blob/master/lib/debug_fa.py#L64

One has underscore while the other does not have.

---

Edited on 4/24/2025

This is now fixed:
pnlbwh/dMRIharmonization@b45dbbd

[tashrifbillah]

because we are actually warping to template space, not standard space?

IIT template should be in MNI space. So our 4/4/2020's comment may not be right. We are probably warping to template space.

[tashrifbillah]

All these advocate for bringing back _ToMNI in multi shell:

multi-shell-dMRIharmonization/lib/debug_fa.py

Line 95 in c99736e

outPrefix = pjoin(templatePath, prefix.replace(f'_b{bshell_b}','') + '_FA')