-
Notifications
You must be signed in to change notification settings - Fork 1
Create fmriprep-DARBY-PDK_v24.1.1.yaml #3
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
|
Any ideas here? |
|
No files in the PR? |
|
Oh, yeah, this is the one I pulled into main on accident. I've updated it with some 'changes'. |
|
Needs to look like https://github.com/VUIIS/yaml_processors/blob/running_processors/fmriprep-ABIDE_v23.2.1.yaml in that section. Some stuff is missing |
|
Hmm that doesn't seem quite right either. Do you have a test output from the main command that we can review to see where files are at? |
|
@baxpr - Test COMPLETED on 047_02965_1. |
|
Oh, this is the one that wasn't quite right. NEVERMIND. |
No, what do you need exactly? This takes all day to run. |
|
I double checked and that part is actually fine (thought there were going to be extra files in the HTML output but there aren't). Do we have existing freesurfer for this? If so then in pre we need before and in main we need to add |
|
This use of |
|
Updated with freesurfer changes and the post block to match more of what I think we're needing (similarly to ABIDE processor you mentioned previously). Test running now (job 1711335). |
|
It didn't like those changes. Does that post need to be something like this: instead of The outputs section should be putting that sourcedata into the /OUTPUTS/sourcedata directory right? |
|
Ah, those would be there if we had freesurfer running internally. PR updated |
|
Still getting |
|
That's easy, I missed it. New commit |
…riprep-prep into duettwe-Darby-philips-default-k
|
@baxpr - Test COMPLETED on 047_02965_1. |
|
Do we have a bug ticket for this? |
Bug 3235 - In progress
Creating this repo so I remember where we are next week when this gets worked on again.