-
Notifications
You must be signed in to change notification settings - Fork 608
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
Reintroduce "Move old ImageDecoder to legacy module and make the nvImageCodec based ImageDecoder the default" #5470
Reintroduce "Move old ImageDecoder to legacy module and make the nvImageCodec based ImageDecoder the default" #5470
Conversation
…vImageCodec based ImageDecoder the default (NVIDIA#5445)"" This reverts commit a1b4fbd. Signed-off-by: Joaquin Anton <[email protected]>
Signed-off-by: Joaquin Anton <[email protected]>
a887fbb
to
bd18094
Compare
a710ce5
to
9c6cedb
Compare
Signed-off-by: Joaquin Anton <[email protected]>
9c6cedb
to
c010602
Compare
!build |
CI MESSAGE: [15065473]: BUILD FAILED |
CI MESSAGE: [15066134]: BUILD STARTED |
CI MESSAGE: [15066134]: BUILD PASSED |
Signed-off-by: Joaquin Anton <[email protected]>
CI MESSAGE: [15089873]: BUILD STARTED |
CI MESSAGE: [15089873]: BUILD PASSED |
@@ -1,4 +1,4 @@ | |||
# Copyright (c) 2019-2024, NVIDIA CORPORATION & AFFILIATES. All rights reserved. | |||
# Copyright (c) 2019-2023, NVIDIA CORPORATION & AFFILIATES. All rights reserved. |
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 is just reverting previous changes
Signed-off-by: Joaquin Anton <[email protected]>
st->out_shape = st->parsed_sample.dali_img_info.shape; | ||
st->out_shape[2] = NumberOfChannels(format_, st->out_shape[2]); | ||
if (use_orientation_ && | ||
((st->parsed_sample.nvimgcodec_img_info.orientation.rotated / 90) & 1)) { |
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.
(Nitpick) This &
is quite non-obvious here. We just want to know if it's rotated by 90 or 270 degrees, why not just
((st->parsed_sample.nvimgcodec_img_info.orientation.rotated / 90) & 1)) { | |
(st->parsed_sample.nvimgcodec_img_info.orientation.rotated % 180 != 0) { |
CI MESSAGE: [15192159]: BUILD STARTED |
Signed-off-by: Joaquin Anton <[email protected]>
6f616c2
to
ffa5fb0
Compare
CI MESSAGE: [15196770]: BUILD STARTED |
CI MESSAGE: [15196770]: BUILD FAILED |
CI MESSAGE: [15224234]: BUILD STARTED |
CI MESSAGE: [15224234]: BUILD FAILED |
CI MESSAGE: [15235949]: BUILD STARTED |
5978608
to
ffa5fb0
Compare
!build |
CI MESSAGE: [15236466]: BUILD STARTED |
CI MESSAGE: [15236466]: BUILD FAILED |
Category:
Other Replaces default ImageDecoders with the nvImageCodec based implementations.
Description:
legacy
moduleAdditional information:
Affected modules and functionalities:
Key points relevant for the review:
Tests:
Checklist
Documentation
DALI team only
Requirements
REQ IDs: N/A
JIRA TASK: N/A