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

TST: Test DSI with b0s. #753

Merged
merged 10 commits into from Nov 16, 2015

Conversation

Projects
None yet
3 participants
@arokem
Member

arokem commented Oct 29, 2015

Is this a problem?

cc: @Garyfallidis

@samuelstjean

This comment has been minimized.

Contributor

samuelstjean commented Oct 29, 2015

DSI is an acquisition scheme in which you have usually only one b0, not sure it does make sense to have more than one conceptually just because of that, but ask mic, he's the dsi expert.

@arokem

This comment has been minimized.

Member

arokem commented Oct 29, 2015

Yep. This is something @Garyfallidis mentioned to me in passing as a
potential bug, so I am floating it here.

The problem in this case seems to be that even just one b0 raises an error
(the dsi_voxels don't have a b0. Smallest b-value in that set is 15).

If people do run into this with multiple b0's being the issue, they can
always get around it by 'preprocessing' the data to average all the b0
measurements into one b0 that gets passed in.

On Thu, Oct 29, 2015 at 9:45 AM, Samuel St-Jean notifications@github.com
wrote:

DSI is an acquisition scheme in which you have usually only one b0, not
sure it does make sense to have more than one conceptually just because of
that, but ask mic, he's the dsi expert.


Reply to this email directly or view it on GitHub
#753 (comment).

@Garyfallidis

This comment has been minimized.

Member

Garyfallidis commented Nov 13, 2015

Seems working 👍

Garyfallidis added a commit that referenced this pull request Nov 16, 2015

Merge pull request #753 from arokem/dsi-with-b0s
TST: Test DSI with b0s.

@Garyfallidis Garyfallidis merged commit e9f4053 into nipy:master Nov 16, 2015

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment