Skip to content

ICA FIX implemented #42

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

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

Conversation

argyelan
Copy link

This implements ICA FIX. However in run.py line 370-375 modification was needed only for our site (was not reading this metatag correctly), so do not merge that (maybe a better way to do it).

ICA FIX was tested works fine with provided classifier on our HCP like data.

argyelan and others added 5 commits October 16, 2018 15:46
updating to current status
1. The problem was that 1. all spin Echo has to have both AP and PA as Intended for
2. For whatever reason the Phase Encoding Directin comes out incorrect in the metatag for field maps, that is why we need this modification (both "j-"
3. In the future conversion to check if files that has Intended For exists or not.
This is an attempt to implement ICA FIX with MCR in the docker. The cifti part of fix does not work (no complied version of CIFTIMATLABWriteRead comes with the fix.tar.gz) but the other part (nii) seemed to run well.
THe code runs the ZHH classifier at 1 2 5 10 20 30 40 50 threshold in addition to HCP 10 threshold.
Thsi will be build into amiklos/hcppipeline:4.11
@DavidEWarrenPhD
Copy link

This would be a nice addition!

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