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 AMD Advanced Media Framework plugin to OBS #602

Closed
wants to merge 0 commits into from

Conversation

Xaymar
Copy link
Contributor

@Xaymar Xaymar commented Aug 26, 2016

Adds support for the AMD Advanced Media Framework through a plugin licensed under MIT. The SDK for it (AMF SDK) is also licensed under MIT.

Plugin Version: 1.3.0pre5 Binaries for Installer Bundling

@ONSBalder
Copy link

Can one of the admins verify this patch?

@Xaymar Xaymar force-pushed the master branch 8 times, most recently from e86951e to f7c1c0e Compare September 1, 2016 20:28
@ovr
Copy link

ovr commented Sep 2, 2016

@Xaymar cool job, is it needed an additional flag for cmake to compile plugin?
I see BUILD_AMF_ENCODER directive and it's false by default (or manually edit?)

I am having RX480 / Linux 4.8 / Ubuntu 16.10, I would like to test it on weekends

@Xaymar
Copy link
Contributor Author

Xaymar commented Sep 2, 2016

@ovr Only BUILD_AMF_ENCODER is needed to be enabled. Building requires the following:

Prerequisites

  • Windows® 7 (SP1 with the Platform Update), Windows® 8.1, or Windows® 10
  • Visual Studio® 2013 or Visual Studio® 2015
  • Windows 10 SDK (Version 10586)
  • AMD Radeon Software Crimson Edition 16.7.3 (16.30.2311) or newer

And of course it needs a windows system to run right now.

@ovr
Copy link

ovr commented Sep 2, 2016

@Xaymar Awwww, seems Linux is not supported GPUOpen-LibrariesAndSDKs/AMF#4 :(

@Xaymar
Copy link
Contributor Author

Xaymar commented Sep 2, 2016

As far as I've seen, gstreamer partially supports encoding on the GPU with VAAPI - my understanding of gstreamer is limited though. I hope that in the future the linux drivers will see enough updates so that this is possible there too.

@Xaymar
Copy link
Contributor Author

Xaymar commented Sep 5, 2016

Nuking this PR as i accidentally did all the changes in the master branch instead of the sub branch. I'm stupid sometimes. Edit: Also because I'm too stupid to figure out how force pushing changes

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.

3 participants