Skip to content
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

Consider "EPImean": using the mean across all tasks/runs to do EPI -> T1 coregistrations #253

Closed
chrisgorgo opened this issue Dec 8, 2016 · 7 comments
Milestone

Comments

@chrisgorgo
Copy link
Contributor

chrisgorgo commented Dec 8, 2016

This would help with cases such as rewardBeast sub-317 where half of the runs have bad distortion and signal dropout, but the other half are fine. Such cases are rare though.

@oesteban oesteban added this to the 1.0.0 milestone Jul 17, 2017
@effigies
Copy link
Member

Does this still seem like a good idea, and, just to be clear, the mechanics of this would be to move to an (EPI -> mean EPI) FLIRT alignment + (mean EPI -> T1w)?

@chrisgorgo
Copy link
Contributor Author

Correct.

@effigies
Copy link
Member

Thinking about what's involved in this:

  • Single workflow that takes all BOLD series
  • Each series needs to be SDC unwarped
  • Resulting reference files then merged with StructuralReference, and (EPI -> mean EPI) transform saved
  • Mean EPI aligned to T1w image

This will involve a fairly major refactor from the current, isolated processing of each BOLD series. One problem with this approach is that you can get different results based on whether your analyze all tasks or just a subset in the same process.

However, if we took an EPInorm approach, then there's a common space to which all EPIs are registered that doesn't depend on which subset of tasks have been specified.

WDYT?

@oesteban
Copy link
Member

+1 to EPInorm approach. And we have the paper to support the decision.

I would leave that for the future though, beyond 1.0.0.

@effigies
Copy link
Member

Opened an issue for EPInorm, if that's the route we decide to go.

@chrisgorgo
Copy link
Contributor Author

@oscar - from what I know EPInorm has not been compared with syn based SDC. It actually might not provide any improvement.

@effigies indeed this would be the case, but I don't think it's a big issue

@effigies effigies modified the milestones: Future, 1.0.0 Aug 1, 2017
@oesteban oesteban changed the title Consider using the mean across all tasks/runs to do EPI -> T1 coregistrations Consider "EPImean": using the mean across all tasks/runs to do EPI -> T1 coregistrations Aug 1, 2017
@oesteban
Copy link
Member

Duplicate of #450 - I'll copy one of @effigies' comments there because it is relevant.

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

No branches or pull requests

3 participants