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

feat: Add AMD / Radeon Transcoding Support #42

Closed
wants to merge 1 commit into from

Conversation

sammcj
Copy link

@sammcj sammcj commented Jun 12, 2023

Add libraries required to enable AMD (Radeon / APU) transcoding.

@Adam-Ant
Copy link
Member

Adam-Ant commented Jun 14, 2023

Thanks for the PR! We're going to discuss internally whether the transcoding support should live on its own branch/tag, or if its small and side-effect free enough into include in the base image. NVidia support should probably also be looked at when we've worked things out, but that can be in a later PR... Bear with us on this, unfortunately real-life gets in the way of open source sometimes... 😁

In the meantime, I can see that the CI for the PR is failing. The error message it gives is clear as mud, but all it means is the container is exiting before the tests can be ran against it - i.e. it's crashing out straight away. Please feel free to add commits & rerun the CI for debugging if you aren't seeing this on a local build of the image, the boxes that runs the CI have decent usage limits. If you run into any walls with the CI side, drop a message here and we can take a look.

@sammcj
Copy link
Author

sammcj commented Jun 16, 2023

I've had an idea, another PR will be incoming to replace this :)

@sammcj sammcj closed this Jun 16, 2023
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

2 participants