PennLINC / xcp_d

Post-processing of fMRIPrep, NiBabies, and HCP outputs
https://xcp-d.readthedocs.io
BSD 3-Clause "New" or "Revised" License
78 stars 26 forks source link

Stop appending pipeline name to output directory #1050

Closed tsalo closed 8 months ago

tsalo commented 9 months ago

Summary

Other BIDS Apps are moving away from hardcoded output folder names (e.g., fMRIPrep, ASLPrep), so it makes sense to do this for XCP-D, although it will be a breaking change.

Additional details

It will bring XCP-D in line with other BIDS Apps and will also make it easier for users to run multiple versions of XCP-D without extra work, as they could customize the output folder.

mattcieslak commented 9 months ago

Check with hbcd to make sure they don't start missing outputs

tsalo commented 8 months ago

👍 from @madisoth and @tjhendrickson

tsalo commented 8 months ago

I think I'll open a separate PR for this, but also implement it in the Nipreps-style Config PR (#1040). I should probably wait until after the next release to merge #1040, so there aren't any surprises for HBCD.