HCPPipelines icon indicating copy to clipboard operation
HCPPipelines copied to clipboard

HCP Pipelines Improvements (ICAFIX, PostFix, RestingStateStats, TaskfMRIAnalysis, londitudinal/cross-sectional logic)

Open tjhendrickson opened this issue 7 years ago • 5 comments

This pull request extends the existing bids apps HCP Pipeline by adding on ICAFIX, PostFix, and RestingStateStats.

The TaskfMRIAnalysis portion has been started but has yet to be completed.

Logic was added in order to account for a bids format that meets either a standard cross sectional format such as: sub-01 anat func And also a longitudinal format: sub-01 ses-01 anat func ses-02 anat func

tjhendrickson avatar Jun 21 '18 20:06 tjhendrickson

Thanks for sending the PR!

Could you remove the .idea folder?

Also could you remove the TaskfMRIAnalysis folder since this work is not finished and those scripts are already part of HCP Pipelines installed in the docker container?

chrisgorgo avatar Jun 21 '18 21:06 chrisgorgo

I can certainly remove the .idea and TaskfMRIAnalysis folders. Is there a way that can be done without deleting it on my fork and resending the pull request (sorry this is my first pull request!)?

tjhendrickson avatar Jun 21 '18 21:06 tjhendrickson

You just need to delete it on the branch from this PR and puszh it to GitHub. The PR will update automatically.

chrisgorgo avatar Jun 21 '18 22:06 chrisgorgo

I'll also correct merge conflicts within Dockerfile

tjhendrickson avatar Jun 21 '18 22:06 tjhendrickson

Yes I am learning my lesson that PRs should be small. I will definitely create small PRs in the future.

tjhendrickson avatar Jun 22 '18 14:06 tjhendrickson