-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Add channel_wise option to RandShiftIntensity and RandScaleIntensity #6629
Comments
Hi @philnovv, I think for |
Hi @KumoLiu, the latter is what I mean. The option to randomly select and apply a factor/offset for each channel would be (I think) more closely aligned with standard practice. |
BTW this applies to all intensity-based augmentation transforms. For example we may want to apply random blurring with an independent blurring kernel for each channel. This makes sense as an augmentation strategy because the different contrasts in real-world multi-contrast images are often acquired with very different resolutions/slice thicknesses etc... |
Part of #6629 . ### Description Add `channel_wise` in `RandScaleIntensity` and `RandScaleIntensityd`. ### Types of changes <!--- Put an `x` in all the boxes that apply, and remove the not applicable items --> - [x] Non-breaking change (fix or new feature that would not break existing functionality). - [ ] Breaking change (fix or new feature that would cause existing functionality to change). - [ ] New tests added to cover the changes. - [ ] Integration tests passed locally by running `./runtests.sh -f -u --net --coverage`. - [ ] Quick tests passed locally by running `./runtests.sh --quick --unittests --disttests`. - [ ] In-line docstrings updated. - [ ] Documentation updated, tested `make html` command in the `docs/` folder. --------- Signed-off-by: KumoLiu <yunl@nvidia.com>
Signed-off-by: KumoLiu <yunl@nvidia.com>
Fixes #6629. ### Description Add `channel_wise` in `RandShiftIntensity` and `RandShiftIntensityd`. ### Types of changes <!--- Put an `x` in all the boxes that apply, and remove the not applicable items --> - [x] Non-breaking change (fix or new feature that would not break existing functionality). - [ ] Breaking change (fix or new feature that would cause existing functionality to change). - [ ] New tests added to cover the changes. - [ ] Integration tests passed locally by running `./runtests.sh -f -u --net --coverage`. - [ ] Quick tests passed locally by running `./runtests.sh --quick --unittests --disttests`. - [ ] In-line docstrings updated. - [ ] Documentation updated, tested `make html` command in the `docs/` folder. --------- Signed-off-by: KumoLiu <yunl@nvidia.com>
It is very common practice to independently augment intensities of separate modalities. Yet currently RandShiftIntensity and RandScaleIntensity do not have this option, and instead randomly shift/scale along all channels at once.
It is noteworthy that RandStdShiftIntensity has a "channel_wise" option to do just this. I suggest we add a similar option to RandShiftIntensity and RandScaleIntensity.
The text was updated successfully, but these errors were encountered: