-
Notifications
You must be signed in to change notification settings - Fork 615
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
Fix documentation of CropMirrorNormalize dtype argument #3439
Conversation
Signed-off-by: Joaquin Anton <janton@nvidia.com>
@@ -163,8 +163,7 @@ class CropMirrorNormalize : public Operator<Backend> { | |||
void SetupCommonImpl(const workspace_t<Backend> &ws) { | |||
const auto &input = ws.template InputRef<Backend>(0); | |||
input_type_ = input.type(); | |||
if (output_type_ == DALI_NO_TYPE) | |||
output_type_ = input_type_; | |||
assert(output_type_ != DALI_NO_TYPE); |
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.
Isn't it a breaking change? Or it is float by default?
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.
It is float by default, it is never DALI_NO_TYPE.
!build |
CI MESSAGE: [3244213]: BUILD STARTED |
CI MESSAGE: [3244213]: BUILD PASSED |
Signed-off-by: Joaquin Anton <janton@nvidia.com>
Signed-off-by: Joaquin Anton <janton@nvidia.com>
Signed-off-by: Joaquin Anton <janton@nvidia.com>
Signed-off-by: Joaquin Anton <janton@nvidia.com>
Signed-off-by: Joaquin Anton <janton@nvidia.com>
Signed-off-by: Joaquin Anton <janton@nvidia.com>
Signed-off-by: Joaquin Anton <janton@nvidia.com>
Signed-off-by: Joaquin Anton <janton@nvidia.com>
Signed-off-by: Joaquin Anton <janton@nvidia.com>
Signed-off-by: Joaquin Anton <janton@nvidia.com>
Signed-off-by: Joaquin Anton <janton@nvidia.com>
Signed-off-by: Joaquin Anton <janton@nvidia.com>
Signed-off-by: Joaquin Anton <janton@nvidia.com>
Signed-off-by: Joaquin Anton <janton@nvidia.com>
Signed-off-by: Joaquin Anton <janton@nvidia.com>
Signed-off-by: Joaquin Anton <janton@nvidia.com>
Signed-off-by: Joaquin Anton janton@nvidia.com
Description
What happened in this PR
Update CropMirrorNormalize's dtype argument documentation. CropMirrorNormalize default dtype is FLOAT, not the input type.
Fixes #3437
Additional information
Fixed wrong description of dtype argument in CropMirrorNormalize
NA
Checklist
Tests
Documentation
DALI team only
Requirements
REQ IDs: N/A
JIRA TASK: N/A