-
Notifications
You must be signed in to change notification settings - Fork 7.2k
[ROCM] Add rocjpeg support for GPU image decoding #9342
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
Open
xytpai
wants to merge
27
commits into
pytorch:main
Choose a base branch
from
xytpai:xyt/rocjpeg_upstream
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
+306
−65
Open
Changes from all commits
Commits
Show all changes
27 commits
Select commit
Hold shift + click to select a range
f4fc92e
add rocjpeg support
xytpai a371c3e
update rocjpeg utils
xytpai e4c4fd0
rm cout
xytpai 3d9041c
refine code
xytpai 1d29986
rm unused file
xytpai 15d8f11
refine code 2
xytpai fb2f9fc
Merge branch 'main' into xyt/rocjpeg_upstream
xytpai bc8c702
Merge branch 'main' into xyt/rocjpeg_upstream
zy1git 173c23d
Merge branch 'main' into xyt/rocjpeg_upstream
zy1git 09c589d
Merge branch 'main' into xyt/rocjpeg_upstream
xytpai 8d4f6ff
Merge branch 'main' into xyt/rocjpeg_upstream
xytpai b68f0ef
full format support
xytpai e113fcc
remove stream dependency
xytpai 85b55f1
make batch-size dynamic
xytpai dd23f0e
resolve remaining comments
xytpai 722a4af
[ROCm] Clean up rocJPEG decode and share GPU JPEG scaffolding (#2)
jeffdaily a319739
refine IMAGE_READ_MODE_UNCHANGED
xytpai 4b71908
rm dead code & refine comment
xytpai 7ce968f
recover nv path
xytpai 248894c
resolve comments
xytpai 802cac2
apply clang-format
xytpai d942228
Separate rocJPEG and nvJPEG setup blocks
xytpai 7581393
add _ suffix for private class members
xytpai a4073b0
just return padded tensor in its original layout
xytpai a2572c8
rm unnecessary sync
xytpai b413e54
refine code
xytpai 0fe060a
add rocjpeg doc link
xytpai File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
Can we just leave the previous
if USE_NVJPEG and (torch.cuda.is_available() or FORCE_CUDA):block exactly like it was, and just have a separate (indepentent) ROCm-specific block below it? They should be mutually exclusive?