Skip to content
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

Add MPEG-4 Part2 decoding support #4361

Open
wants to merge 12 commits into
base: main
Choose a base branch
from

Conversation

awolant
Copy link
Contributor

@awolant awolant commented Oct 15, 2022

Adds MPEG-4 Part2 decoding support for FramesDecoder and FramesDecoderGpu for all usage patterns (regular, in-memory, no index).

Category:

Description:

Based on #4327 by @JanuszL

Requires NVIDIA/DALI_deps#61 deps PR and NVIDIA/DALI_extra#109 DALI_extra PR.

Additional information:

Affected modules and functionalities:

Key points relevant for the review:

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: N/A

Signed-off-by: Albert Wolant <awolant@nvidia.com>
Signed-off-by: Albert Wolant <awolant@nvidia.com>
Signed-off-by: Albert Wolant <awolant@nvidia.com>
Signed-off-by: Albert Wolant <awolant@nvidia.com>
Signed-off-by: Albert Wolant <awolant@nvidia.com>
Signed-off-by: Albert Wolant <awolant@nvidia.com>
Signed-off-by: Albert Wolant <awolant@nvidia.com>
Signed-off-by: Albert Wolant <awolant@nvidia.com>
Signed-off-by: Albert Wolant <awolant@nvidia.com>
@awolant awolant mentioned this pull request Oct 15, 2022
18 tasks
@awolant awolant marked this pull request as ready for review October 15, 2022 13:39
Signed-off-by: Albert Wolant <awolant@nvidia.com>
Signed-off-by: Albert Wolant <awolant@nvidia.com>
class FramesDecoderGpuMpeg4Test : public FramesDecoderGpuTest {
void AssertFrame(uint8_t *frame, int index, TestVideo& ground_truth) override {
MemCopy(FrameDataCpu(), frame, ground_truth.FrameSize());
SaveFrame(FrameDataCpu(), index, 0, 0, "/home/awolant/Downloads/frames/", ground_truth.Width(), ground_truth.Height());
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please fix.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done

@dali-automaton
Copy link
Collaborator

CI MESSAGE: [6216597]: BUILD STARTED

@dali-automaton
Copy link
Collaborator

CI MESSAGE: [6216597]: BUILD FAILED

Signed-off-by: Albert Wolant <awolant@nvidia.com>
@dali-automaton
Copy link
Collaborator

CI MESSAGE: [6225403]: BUILD STARTED

@dali-automaton
Copy link
Collaborator

CI MESSAGE: [6225403]: BUILD FAILED

@dali-automaton
Copy link
Collaborator

CI MESSAGE: [6226244]: BUILD STARTED

@dali-automaton
Copy link
Collaborator

CI MESSAGE: [6226244]: BUILD FAILED

@JanuszL
Copy link
Contributor

JanuszL commented Oct 24, 2022

I think you need to update conda/third_party/dali_ffmpeg/recipe/build.sh and https://github.com/NVIDIA/DALI/blame/main/docs/compilation.rst#L205

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.

None yet

5 participants