fixes behav in singl-session-subjects in long study

bids-apps / freesurfer

BIDS app wrapping recon-all from FreeSurfer

Apache License 2.0

40 stars 35 forks source link

fixes behav in singl-session-subjects in long study #22

Closed fliem closed 7 years ago

fliem commented 7 years ago

In longitudinal studies (studies in which at least one subject has more than one session) and --multiple_sessions='longitudinal') subjects with only one session should undergo the entire longitudinal stream. The changes in #11 broke that behavior. The present changes should fix it.

fliem commented 7 years ago

Thanks. We need two loops here. The first determines if the study is longitudinal. For that it needs to loop through all subjects and look for sessions. The second loop starts the recon-all.

But you're right, I've missed the sessions variables. I've added them.
I also ~~added longitudinal tests &~~ (they time-out because they take longer than 2h) made the tests more thorough by checking if a recon-all.done file exists.

fliem commented 7 years ago

There still is something wrong here. I'll have a go at it again in the next days.