search

mnoergaard / petprep_extract_tacs

PETPrep BIDS App for extracting time activity curves for PET data
Apache License 2.0
4 stars 2 forks source link

Merge Runs #9

Closed bendhouseart closed 4 months ago

bendhouseart commented 4 months ago

Should this merge the dseg and morph tsv's too? Otherwise I think it's basically ready to implement into a post pipeline step as an optional step with --merge-tacs

mnoergaard commented 4 months ago

The dseg and morphs are created from the anatomical workflow, and hence are actually equal for the separate runs. They need to be present for a given _tacs.tsv so one would need to copy one of the dseg and morphs for one of the runs and remove the run entity in the filename.

bendhouseart commented 4 months ago

I'd like to test it a bit more, but on the other hand a user can omit the --merge_runs argument and not run into any issues...

mnoergaard commented 4 months ago

@bendhouseart - just tested this locally on the PDE4B data and it works as expected!