-
Notifications
You must be signed in to change notification settings - Fork 6
resliceToiImage - interpolation type suggestion #22
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
Conversation
[ENH] update from bids-matlab
to nearest neighbour
I might be wrong but here is a bit more thought on it. |
If you are re-slicing a multi-label atlas to different resolution and you want to preserve the same labels, then the interpolation method that should be used is the nearest neighbor (interpolation == 1). |
inserted back the value 4 for tmaps. For atlases, we need to use interp = 1
ok, if I understood @mohmdrezk right, then we would need a if case here. Please correct me if I'm wrong.
But I'm guessing in atlas masks, this repo does not do reslicing. I haven't figure it out why/how though. WIP. Again correct me if I'm wrong/missing a step:
|
interp = 4 % for neurosynth (having zscores), tmaps, already extracted ROI from an atlas (not carrying mutliple labels)
For the already extracted ROI from an atlas (not carrying multiple
labels), I think it's better to do nearest neighbour while reslicing
instead of doing interpolation and then binarizing.
… if multilabelAtlas or binaryMap is used
interp = 1
Or any ROI that you intend to use as a mask.
|
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.
Will merge but this just contains only typo fixes now.
This is a suggestion of interpolation type change for reslicing images.
The suggestion was due to @mohmdrezk warning. I want to ask of this is what he meant + if someone else has an objection to it or not.