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

normalize fails to run #185

Closed
mgxd opened this issue Sep 26, 2016 · 5 comments
Closed

normalize fails to run #185

mgxd opened this issue Sep 26, 2016 · 5 comments
Assignees

Comments

@mgxd
Copy link
Contributor

mgxd commented Sep 26, 2016

Copy of a crashfile

"Too many samples map outside moving image buffer. There are only 38344 valid points out of 1801501 total points."

I'm running into this error on about half of subjects in my dataset - any thoughts?

@oesteban
Copy link
Member

Hi @mgxd this is a problem related to ANTs normalization. We are pretty aware of this problem, and for this reason, we have recently added a new normalization interface that will try new settings if the default ones fail.

Basically, the T1w image of your subject sub-voice968 failed to be registered to the MNI template. This is particular of that case, so if you could share it with me, I could search for more appropriate settings. I will incorporate those settings into mriqc as a fallback.

For now, you can also try the following: use current master (this fallback thing was added last Friday) and give it a try. Maybe the quick and dirty settings I wrote work for you in this subject.

@oesteban oesteban added the bug label Sep 26, 2016
@oesteban oesteban self-assigned this Sep 26, 2016
@oesteban
Copy link
Member

Sorry, I see you are using the docker image.

Please confirm me if you could share that T1 image with me (or provide me with ANTs settings that work for you) and I will build a new mriqc image with those settings.

@mgxd
Copy link
Contributor Author

mgxd commented Sep 27, 2016

@oesteban thanks for the reply - I have a T1 image I can share with you, just need your email.

@oesteban
Copy link
Member

@mgxd I just copied you image, thanks for sharing. Sorry for my silence, I'm on it.

@oesteban
Copy link
Member

Hi @mgxd, the PR just merged (see above) fixed the problem with this subject. It also requires to update niworkflows to the latest version (0.0.3a14).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants