Skip to content

Update video decoder & reader supported format list#6377

Merged
JanuszL merged 1 commit into
NVIDIA:mainfrom
JanuszL:supported_codecs_docs
May 29, 2026
Merged

Update video decoder & reader supported format list#6377
JanuszL merged 1 commit into
NVIDIA:mainfrom
JanuszL:supported_codecs_docs

Conversation

@JanuszL
Copy link
Copy Markdown
Contributor

@JanuszL JanuszL commented May 29, 2026

  • Removes references to unsupported CPU codecs
  • Updates operator documentation to reflect changes in supported formats

Category:

Other (e.g. Documentation, Tests, Configuration)

Description:

Additional information:

Affected modules and functionalities:

  • video reader and decoder

Key points relevant for the review:

  • NA

Tests:

  • Existing tests apply
  • New tests added
    • Python tests
    • GTests
    • Benchmark
    • Other
  • N/A

Checklist

Documentation

  • Existing documentation applies
  • Documentation updated
    • Docstring
    • Doxygen
    • RST
    • Jupyter
    • Other
  • N/A

DALI team only

Requirements

  • Implements new requirements
  • Affects existing requirements
  • N/A

REQ IDs: N/A

JIRA TASK: DALI-4712

- Removes references to unsupported CPU codecs
- Updates operator documentation to reflect changes in supported formats

Signed-off-by: Janusz Lisiecki <jlisiecki@nvidia.com>
@review-notebook-app
Copy link
Copy Markdown

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@JanuszL JanuszL force-pushed the supported_codecs_docs branch from bcf5d9d to c2c0e8f Compare May 29, 2026 08:52
@JanuszL
Copy link
Copy Markdown
Contributor Author

JanuszL commented May 29, 2026

!build

@dali-automaton
Copy link
Copy Markdown
Collaborator

CI MESSAGE: [53041683]: BUILD STARTED

@greptile-apps
Copy link
Copy Markdown
Contributor

greptile-apps Bot commented May 29, 2026

Greptile Summary

This PR updates operator docstrings for the video decoder and video reader operators to correctly reflect that H.264/AVC and H.265/HEVC are no longer supported on the CPU backend, as a follow-up to #6352.

  • decoders__Video docstring: H.264/AVC and H.265/HEVC moved from "supported by both CPU and Mixed backends" to "Mixed backend only".
  • experimental__readers__Video docstring: the same codecs moved from "supported by both CPU and GPU backends" to "GPU backend only".

Confidence Score: 5/5

Documentation-only change with no logic modifications; both operator docstrings are updated consistently.

Both changed files touch only docstrings. The codec lists in the two operators are updated identically and are internally consistent with each other. No runtime behavior is altered.

No files require special attention.

Important Files Changed

Filename Overview
dali/operators/video/decoder/video_decoder_cpu.cc Docstring updated: H.264/AVC and H.265/HEVC moved from "CPU and Mixed" section to "Mixed backend only" to reflect removal of CPU codec support (follow-up to #6352).
dali/operators/video/reader/video_reader_decoder_op.cc Docstring updated: H.264/AVC and H.265/HEVC moved from "CPU and GPU" section to "GPU backend only", consistent with the decoder operator change.

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    subgraph Before["Before (Incorrect)"]
        B_CPU["CPU + Mixed/GPU backends"]
        B_CPU --> B_H264["H.264/AVC"]
        B_CPU --> B_H265["H.265/HEVC"]
        B_CPU --> B_VP8["VP8"]
        B_CPU --> B_VP9["VP9"]
        B_CPU --> B_MJPEG["MJPEG"]
        B_GPU["Mixed/GPU only"]
        B_GPU --> B_AV1["AV1"]
        B_GPU --> B_MPEG4["MPEG-4"]
    end

    subgraph After["After (Correct)"]
        A_CPU["CPU + Mixed/GPU backends"]
        A_CPU --> A_VP8["VP8"]
        A_CPU --> A_VP9["VP9"]
        A_CPU --> A_MJPEG["MJPEG"]
        A_GPU["Mixed/GPU only"]
        A_GPU --> A_AV1["AV1"]
        A_GPU --> A_MPEG4["MPEG-4"]
        A_GPU --> A_H264["H.264/AVC"]
        A_GPU --> A_H265["H.265/HEVC"]
    end
Loading

Reviews (2): Last reviewed commit: "Update video decoder & reader supported ..." | Re-trigger Greptile

@JanuszL
Copy link
Copy Markdown
Contributor Author

JanuszL commented May 29, 2026

!build

@dali-automaton
Copy link
Copy Markdown
Collaborator

CI MESSAGE: [53044242]: BUILD STARTED

@dali-automaton
Copy link
Copy Markdown
Collaborator

CI MESSAGE: [53044242]: BUILD PASSED

@dali-automaton
Copy link
Copy Markdown
Collaborator

CI MESSAGE: [53044242]: BUILD PASSED

@dali-automaton
Copy link
Copy Markdown
Collaborator

CI MESSAGE: [53041683]: BUILD PASSED

@JanuszL JanuszL merged commit b1a2cd9 into NVIDIA:main May 29, 2026
7 checks passed
@JanuszL JanuszL deleted the supported_codecs_docs branch May 29, 2026 12:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants