-
Notifications
You must be signed in to change notification settings - Fork 262
ENH: Adding vox2ras_tkr #164
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
Changes from all commits
Commits
Show all changes
11 commits
Select commit
Hold shift + click to select a range
95e6b75
ENH: Adding vox2ras_tkr
larsoner d5f8c07
FIX(?): Add zooms test
larsoner 39ab09f
FIX: Deal with borked test dataset
larsoner 57f1089
FIX: Match MRIRead.m
larsoner 6724c78
FIX?: Order
larsoner 4e0d3ea
RF: make header update only if changed affine
matthew-brett 13e87c0
BF: with new save affine algorithm, zooms=1
matthew-brett 1ab91f0
Merge pull request #1 from matthew-brett/setting-mgh-affines
larsoner d5588bf
FIX: Reverting to mri_info test compat
larsoner 2b0ecbe
TST: add test for cosine vector order
matthew-brett 06a0a23
Merge pull request #2 from matthew-brett/mgh-cosine-vector-order
larsoner File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
These are the lines I'm unclear about (not that there are so many to choose from). I expected the 1's in the upper left 3x3 of this matrix to be given by the values in
get_zooms()
, but using those values makes the test script fail (since the zooms are not unity, butmri_info --vox2ras-tkr
on the test .mgz file has only +/-1's in the upper left 3x3...There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Using
mri_info
to get the resolution in each direction gives:However, in nibabel getting the deltas I get:
So, clearly, get_zooms() is doing something else that I mis-interpreted. I'm not sure where to get the equivalent information in nibabel, though...