-
Notifications
You must be signed in to change notification settings - Fork 607
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
Presteps needed by image_decoder.h APIs #4277
Conversation
17c45e1
to
fd67eeb
Compare
!build |
CI MESSAGE: [5992476]: BUILD STARTED |
fd67eeb
to
847abdb
Compare
CI MESSAGE: [5992476]: BUILD PASSED |
} catch (...) { | ||
res.exception = std::current_exception(); | ||
res.success = false; | ||
auto decoded_image = jpeg::Uncompress(encoded_data, data_size, flags); |
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.
removed try/catch (handled one layer up)
res.intermediate_buffer.resize(volume(ctx.shape) * type_size); | ||
decode_out = {res.intermediate_buffer.data(), ctx.shape, ctx.pixel_type}; | ||
} | ||
auto decode_out = out; |
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.
removed try/catch (handled one layer up)
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.
My concern is that when a try/catch is removed, there is a short period between the merges of PRs, that this error is not handled properly. Please consider this.
The other try/catch is already in place. This one is redundant at this point. |
!build |
CI MESSAGE: [6012177]: BUILD STARTED |
CI MESSAGE: [6012177]: BUILD FAILED |
!build |
CI MESSAGE: [6012360]: BUILD STARTED |
CI MESSAGE: [6012360]: BUILD FAILED |
result.exception = std::current_exception(); | ||
if (is_processing_needed) { | ||
auto multiplier = calc_bpp_adjustment_multiplier(ctx.bpp, ctx.pixel_type); | ||
Convert(out, "HWC", opts.format, decode_out, "CHW", format, ctx.cuda_stream, {}, multiplier); |
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.
Outdated?
Signed-off-by: Joaquin Anton <janton@nvidia.com>
4e9d0b5
to
8993f9c
Compare
CI MESSAGE: [6013162]: BUILD STARTED |
CI MESSAGE: [6013162]: BUILD PASSED |
Signed-off-by: Joaquin Anton janton@nvidia.com
Category:
Refactoring
Description:
Additional information:
Affected modules and functionalities:
imgcodec
Key points relevant for the review:
NA
Tests:
Checklist
Documentation
DALI team only
Requirements
REQ IDs: N/A
JIRA TASK: N/A