-
Notifications
You must be signed in to change notification settings - Fork 42
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
stats.threshold usage is confusing #304
Comments
We should talk about this and simplify and consolidate. Just took a peak at this and one thing I'm concerned about with
My guess is that this was added to deal with nans and to keep the shape the same, but this will be an issue depending on what the values of the map you are trying to threshold. This option assumes that values will be centered at zero, which will not always be the case.
@ejolly I'm not quite sure why it would be changing the shape of Brain_Data though. Any ideas? Actually, this might be a bug where This is related to #310. |
@ljchang I think it's changing the shape for exactly the reason you suggested. The I guess the broader decision is, that there are two different kinds of "masking" operations a user wants to perform and we don't cleanly differentiate between the two:
|
One possible delineation is this:
How does something like that sound? |
Ugh getting burned by this again. We should really standardize behavior across
|
@ljchang Few issues with students at MIND encountering this again. I'd strongly advise to replace Is there a situation I'm missing where we want |
Thresholding method on
Brain_Data
objects vs thethreshold
function behave differently. @ljchang is this intended behavior? For example, a common workflow is to:However,
threshold
changes the shape ofBrain_Data
rather than simply masking out non-surviving voxels like theBrain_Data.threshold
method. This make it impossible to append images to each other for storage or other use.The text was updated successfully, but these errors were encountered: