Skip to content

Correct vbm longitudinal#54

Merged
AGrigis merged 2 commits intobrainprepdesk:devfrom
AymericGaudin:vbm_long
Apr 22, 2026
Merged

Correct vbm longitudinal#54
AGrigis merged 2 commits intobrainprepdesk:devfrom
AymericGaudin:vbm_long

Conversation

@AymericGaudin
Copy link
Copy Markdown
Contributor

The vbm longitudinal pipeline was breaking because the session folders were not created. This PR corrects the issue.

@AymericGaudin
Copy link
Copy Markdown
Contributor Author

AymericGaudin commented Apr 21, 2026

I notice that it now outputs an error at the end of the pipeline. The mwp1 output is not found because of the extra prefix before the image name (brainprep is looking for mwp1sub-[...], but something like mwp1rsub-[...] is created)

@AymericGaudin
Copy link
Copy Markdown
Contributor Author

Brainprep is now checking for mwp1rsub- files when longitudinal pipeline is chosen. This fix is not robust to other prefixes (usub-, rusub-), but I don't if they are supposed to appear and in which context.

@AGrigis AGrigis merged commit 9a44366 into brainprepdesk:dev Apr 22, 2026
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants