-
Notifications
You must be signed in to change notification settings - Fork 15
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
Move derivatives from spine-generic-processed #123
Conversation
@mguaypaq , could you take a look at the PR. Thank you! |
Sure, I'm pretty busy today but I'll take a look tomorrow. |
I had some extra modifications to add, but I was unable to push to git annex:
|
@sandrinebedard please try
The wiki docs seem to have had a bad merge at some point (this point). The last step, The lab manual docs have the same instructions but without the final step. I've updated the wiki to match. |
Thanks @kousu ! It seams t have worked! |
@mguaypaq I will need finally to modify some extra data. I think you can wait to check this PR until then. I'll ping you when it's ready! |
Hello @mguaypaq ! I've finished the extra modification, you can now take a look, thank you! |
It looks like the following 45 files are missing from amazon:
The file metadata already exists on your branch
|
OK I've added them to amazon! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Since data validation passes, and this (giant) PR is mostly about moving files from another dataset back into this repository, I'm going to approve and merge.
Context
Currenlty, we have a processed version of this dataset (
spine-generic-processed
) where we have derivatives, but some are duplicated. See #121.This PR moves the derivatives from
spine-generic-processed
to data-multi-subjects.The derivatives are now BIDS compatible, they inlcude:
Related issues
fixes #121
What is missing
/derivatives/labels
to explain what are the processing steps and link to the processing script Modify README.txt in derivatives to explain processing #124spine-generic-processed