-
Notifications
You must be signed in to change notification settings - Fork 605
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 support for UINT8 and INT8 outputs in CMN + scale and shift arguments #2458
Conversation
…ents. Signed-off-by: Michał Zientkiewicz <mzient@gmail.com>
!build |
CI MESSAGE: [1793821]: BUILD STARTED |
CI MESSAGE: [1793821]: BUILD PASSED |
In test_operator_crop_mirror_normalize.py:test_cmn_vs_numpy you can provide more types than just FLOAT. The only thing left there is to add type conversion in crop_mirror_normalize_func, but should not be that difficult. |
I'll need to cut reduce the number of tests elsewhere - it's already taking very long. |
Reduce number of tests for CMN by randomly selecting paramters that don't need full combination sweep. Signed-off-by: Michał Zientkiewicz <mzient@gmail.com>
!build |
CI MESSAGE: [1796073]: BUILD STARTED |
CI MESSAGE: [1796073]: BUILD PASSED |
@@ -58,6 +58,14 @@ If not set, the input type is used.)code", DALI_FLOAT) | |||
.AddOptionalArg("std", | |||
R"code(Standard deviation values for image normalization.)code", | |||
std::vector<float>{1.0f}) | |||
.AddOptionalArg("scale", R"(The value by which the result is multiplied. | |||
|
|||
This argument is useful when using integer outputs to improve dynamic range utiliztion.)", |
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.
This argument is useful when using integer outputs to improve dynamic range utiliztion.)", | |
This argument is useful when using integer outputs to improve dynamic range utilization.)", |
1.0f) | ||
.AddOptionalArg("shift", R"(The value added to the (scaled) result. | ||
|
||
This argument is useful when using unsigned integer outputs to improve dynamic range utiliztion.)", |
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.
This argument is useful when using unsigned integer outputs to improve dynamic range utiliztion.)", | |
This argument is useful when using unsigned integer outputs to improve dynamic range utilization.)", |
Signed-off-by: Michał Zientkiewicz <mzient@gmail.com>
!build |
CI MESSAGE: [1796429]: BUILD STARTED |
Signed-off-by: Michał Zientkiewicz mzient@gmail.com
Why we need this PR?
Pick one, remove the rest
What happened in this PR?
Fill relevant points, put NA otherwise. Replace anything inside []
JIRA TASK: N/A