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

AMD Media - Fix FFmpeg swscale error when using hwdec for amd_media #1310

Merged
merged 1 commit into from
Mar 20, 2024

Conversation

rrawther
Copy link
Contributor

The error was most likely triggered when updating FFmpeg version.
For VAAPI- ffmpeg returns wrong pix_format which was causing failure in swscale.

@kiritigowda kiritigowda changed the base branch from master to develop March 20, 2024 06:00
@kiritigowda kiritigowda changed the title fix FFmpeg swscale error when using hwdec for amd_media AMD Media - Fix FFmpeg swscale error when using hwdec for amd_media Mar 20, 2024
@kiritigowda
Copy link
Collaborator

For #1305

@kiritigowda
Copy link
Collaborator

Test Passing with pr

runvx 1.0.0
OK: using AMD OpenVX 1.3.0
OK: loaded 2 kernels from libvx_amd_media.so
VAAPI device created for device /dev/dri/renderD128 and stream 0
INFO: reading 2048x1024 into slice#0 from /home/kiriti/develop/mivisionx-kiriti/MIVisionX/data/videos/AMD_driving_virtual_20.mp4
csv,HEADER ,STATUS, COUNT,cur-ms,avg-ms,min-ms,clenqueue-ms,clwait-ms,clwrite-ms,clread-ms
csv,OVERALL,  PASS,     1,      , 50.37, 50.37,  0.00,  0.00,  0.00,  0.00 (median 50.370)
> total elapsed time:   0.05 sec
Average Decode Time per frame (ms): 0.792564
Average Transfer Time per frame (ms): 12.6994

@kiritigowda kiritigowda merged commit 86ff17b into ROCm:develop Mar 20, 2024
10 of 18 checks passed
kiritigowda added a commit that referenced this pull request Mar 28, 2024
* Bump rocm-docs-core[api_reference] from 0.34.0 to 0.34.2 in /docs/sphinx (#1286)

Bumps [rocm-docs-core[api_reference]](https://github.com/RadeonOpenCompute/rocm-docs-core) from 0.34.0 to 0.34.2.
- [Release notes](https://github.com/RadeonOpenCompute/rocm-docs-core/releases)
- [Changelog](https://github.com/ROCm/rocm-docs-core/blob/develop/CHANGELOG.md)
- [Commits](ROCm/rocm-docs-core@v0.34.0...v0.34.2)

---
updated-dependencies:
- dependency-name: rocm-docs-core[api_reference]
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Bump cryptography from 42.0.0 to 42.0.2 in /docs/sphinx (#1289)

Bumps [cryptography](https://github.com/pyca/cryptography) from 42.0.0 to 42.0.2.
- [Changelog](https://github.com/pyca/cryptography/blob/main/CHANGELOG.rst)
- [Commits](pyca/cryptography@42.0.0...42.0.2)

---
updated-dependencies:
- dependency-name: cryptography
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Bump rocm-docs-core[api_reference] from 0.34.2 to 0.35.0 in /docs/sphinx (#1291)

Bumps [rocm-docs-core[api_reference]](https://github.com/RadeonOpenCompute/rocm-docs-core) from 0.34.2 to 0.35.0.
- [Release notes](https://github.com/RadeonOpenCompute/rocm-docs-core/releases)
- [Changelog](https://github.com/ROCm/rocm-docs-core/blob/develop/CHANGELOG.md)
- [Commits](ROCm/rocm-docs-core@v0.34.2...v0.35.0)

---
updated-dependencies:
- dependency-name: rocm-docs-core[api_reference]
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Docs - Bump cryptography from 42.0.0 to 42.0.4 in /docs/sphinx (#1290)

Bumps [cryptography](https://github.com/pyca/cryptography) from 42.0.0 to 42.0.4.
- [Changelog](https://github.com/pyca/cryptography/blob/main/CHANGELOG.rst)
- [Commits](pyca/cryptography@42.0.0...42.0.4)

---
updated-dependencies:
- dependency-name: cryptography
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Kiriti Gowda <[email protected]>

* CMakeLists - Updates (#1287)

* Docs - Bump rocm-docs-core[api_reference] from 0.35.0 to 0.35.1 in /docs/sphinx (#1293)

Bumps [rocm-docs-core[api_reference]](https://github.com/RadeonOpenCompute/rocm-docs-core) from 0.35.0 to 0.35.1.
- [Release notes](https://github.com/RadeonOpenCompute/rocm-docs-core/releases)
- [Changelog](https://github.com/ROCm/rocm-docs-core/blob/develop/CHANGELOG.md)
- [Commits](ROCm/rocm-docs-core@v0.35.0...v0.35.1)

---
updated-dependencies:
- dependency-name: rocm-docs-core[api_reference]
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* NN Inference Setup - Use ONNX 1.12.0 (#1294)

* Bump rocm-docs-core[api_reference] from 0.35.1 to 0.36.0 in /docs/sphinx (#1296)

Bumps [rocm-docs-core[api_reference]](https://github.com/RadeonOpenCompute/rocm-docs-core) from 0.35.1 to 0.36.0.
- [Release notes](https://github.com/RadeonOpenCompute/rocm-docs-core/releases)
- [Changelog](https://github.com/ROCm/rocm-docs-core/blob/develop/CHANGELOG.md)
- [Commits](ROCm/rocm-docs-core@v0.35.1...v0.36.0)

---
updated-dependencies:
- dependency-name: rocm-docs-core[api_reference]
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* README - Updates and cleanup

* Readme - Updates (#1299)

* Readme - Updates

* Updates - Readme

* Copyright - Updated to 2024 (#1300)

* Docs - Update Doxyfile (#1301) (#1302)

* Docs - Update Doxyfile (#1301)

* Update .Doxyfile

---------

Co-authored-by: Lisa <[email protected]>

* Sample - mv_object README.md updates

* Deprecate - rocAL (#1304)

* Deprecate - rocAL

* Docker - Updates

* Readme - Cleanup

* README.md - rocAL fix

* CMakeLists - Build for advanced GPUs fix (#1307)

* MIVisionX - BUILD_WITH_AMD_ADVANCE

* Half - msg for required

* Readme & Changelog - Updates

* Readme - Cleanup

* Setup - Exit without error

* Setup - Cleanup & Fix

* Fix Copyright

* Loom - Off for HIP

* CMakeList - Fix msg

* Bump rocm-docs-core[api_reference] from 0.36.0 to 0.37.0 in /docs/sphinx (#1309)

Bumps [rocm-docs-core[api_reference]](https://github.com/RadeonOpenCompute/rocm-docs-core) from 0.36.0 to 0.37.0.
- [Release notes](https://github.com/RadeonOpenCompute/rocm-docs-core/releases)
- [Changelog](https://github.com/ROCm/rocm-docs-core/blob/develop/CHANGELOG.md)
- [Commits](ROCm/rocm-docs-core@v0.36.0...v0.37.0)

---
updated-dependencies:
- dependency-name: rocm-docs-core[api_reference]
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Documentation - Fix broken links (#1308)

* Fix broken links

* remove unused

* restored conf.py

* Update NOTE format

* fix other broken links

* Update README.md

---------

Co-authored-by: Kiriti Gowda <[email protected]>

* AMD Media - Fix swscale error when using hwdec for amd_media (#1310)

* Enable Static Library Target Generation st1

* Static packaging Update

* Extending static option to targets

* Enabling Multiple target static lib in openvx cmake

* Review Comment to avoid packaging binaries with external dependency

* Review Comment Update

---------

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Kiriti Gowda <[email protected]>
Co-authored-by: Hansel Yang <[email protected]>
Co-authored-by: Lisa <[email protected]>
Co-authored-by: randyh62 <[email protected]>
Co-authored-by: Kiriti Gowda <[email protected]>
Co-authored-by: Rajy Rawther <[email protected]>
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.

2 participants