Skip to content

fixed mcflirt mean image output #776

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

Merged
merged 1 commit into from
Jan 31, 2014

Conversation

juhuntenburg
Copy link
Contributor

No description provided.

@mwaskom
Copy link
Member

mwaskom commented Jan 27, 2014

Can you explain a little bit more about what the problem was here?

@juhuntenburg
Copy link
Contributor Author

I found the mean image that is output by -stats to be different from the one output by -meanvol. When you specify both options FSL actually outputs two files ('mean_reg' and 'meanvol'), where meanvol is missing top and bottom slices. It would be an alternative to adopt this but I found it even more confusing to output two slightly different mean images, one of which is corrupted anyhow.

@coveralls
Copy link

Coverage Status

Changes Unknown when pulling 6b23666 on juhuntenburg:fix/mcflirt_mean into * on nipy:master*.

@juhuntenburg
Copy link
Contributor Author

Maybe I should add that my original problem was that nipype, as opposed to FSL, at the moment outputs no mean image at all when only the -mean_vol option is specified but not the -stats_imgs option. I'm still in two minds whether it wouldn't be better to just go the FSL way and output the two mean images independently for -stats and -meanvol. One reason against it is that the description of the -stats doesn't even mention a mean image.

@satra
Copy link
Member

satra commented Jan 29, 2014

@juhuntenburg - how does this relate to matthew webster's comment on the fsl mailing list. there seems to be some fix that happened? did this get fixed in fsl?

@juhuntenburg
Copy link
Contributor Author

Hej Satra, took me a while to disentangle but I believe it's two different problems. The motion parameters substantially differ between 4.1.9 and 5.0.6, but the mean images seem to be the same and the issue described above remains. But I will bring this up on the FSL mailing list, too.

chrisgorgo added a commit that referenced this pull request Jan 31, 2014
@chrisgorgo chrisgorgo merged commit b209e4c into nipy:master Jan 31, 2014
@juhuntenburg juhuntenburg deleted the fix/mcflirt_mean branch September 5, 2014 11:27
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.

5 participants