-
Notifications
You must be signed in to change notification settings - Fork 168
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 majority replacement option for Reduce #5670
base: dev
Are you sure you want to change the base?
Conversation
The build and test suite have started for your pull request. To view your build log, please reference the build with source version: "PR_5670". Additionally, check the latest "dev" source version to identify existing test failures. Please note that you are not responsible for the test failures that exist on both your PR and the dev branch. |
7d323f8
to
c9a6065
Compare
The build and test suite have started for your pull request. To view your build log, please reference the build with source version: "PR_5670". Additionally, check the latest "dev" source version to identify existing test failures. Please note that you are not responsible for the test failures that exist on both your PR and the dev branch. |
@AustinSanders Would it be possible to get a couple screenshots of the new mode in use? |
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.
Overall looks good! @AustinSanders just need to update the CHANGELOG and I can approve this
The build and test suite have started for your pull request. To view your build log, please reference the build with source version: "PR_5670". Additionally, check the latest "dev" source version to identify existing test failures. Please note that you are not responsible for the test failures that exist on both your PR and the dev branch. |
Description
Add a vper_replace option for majority replacement. When alg=AVERAGE, the user can select vper_replace=MAJORITY to replace the special pixel with the most commonly occurring special pixel in the processing window.
Related Issue
Fixes #5101
How Has This Been Validated?
Types of changes
Checklist:
Licensing
This project is mostly composed of free and unencumbered software released into the public domain, and we are unlikely to accept contributions that are not also released into the public domain. Somewhere near the top of each file should have these words: