Skip to content
This repository has been archived by the owner on Aug 6, 2024. It is now read-only.

Adds gstreamer-vaapi backends #278

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

zzoon
Copy link

@zzoon zzoon commented May 21, 2017

  • Uses vaapisink.
  • Consumes CPU less than gstreamer-opengl.
  • Note that it's not working on Wayland.

}
case GST_MESSAGE_ELEMENT:
{
#ifdef GDK_WINDOWING_X11
Copy link
Contributor

Choose a reason for hiding this comment

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

You also have to check if the current backend is x11 too.

Choose a reason for hiding this comment

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

@TingPing do you know how to do that ?

@@ -0,0 +1,295 @@
/*
* This file is part of GNOME Twitch - 'Enjoy Twitch on your GNU/Linux desktop'
* Copyright © 2017 Vincent Szolnoky <vinszent@vinszent.com>
Copy link
Contributor

Choose a reason for hiding this comment

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

You should add your own name in the copyright notice, since (presumably) you have written this file. Same for the other files you have written that contain a copyright notice.

@zzoon zzoon force-pushed the upstream2 branch 2 times, most recently from 38f73ef to fa058f0 Compare May 21, 2017 14:14
- Using vaapisink
- Consumes CPU less than gstreamer-opengl.
- Note that it's not working on Wayland.
@vinszent
Copy link
Owner

According to the docs, it should be possible to use the vaapidecode element and pipe it into a OpenGL element without much performance loss. Basically what the gstreamer-opengl player backend does but using vaapi instead of libav to do the decoding. This should allow it to work on Wayland as well.

HumanG33k added a commit to WildTurtles/gnome-twitch that referenced this pull request Jan 4, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants