celluloid-player / celluloid

A simple GTK+ frontend for mpv
https://celluloid-player.github.io
GNU General Public License v3.0
1.08k stars 88 forks source link

GLSL-Shaders not working #780

Open gsantner opened 2 years ago

gsantner commented 2 years ago

Overview Description:

The issue is that glsl-shaders do not work in celluloid/gnome-mpv, but do in normal mpv.

Steps to Reproduce:

  1. Use glsl-shaders= in mpv-options, in mpv.conf or via lua scripts
  2. it doesn't work, no quality improvement

Version:

Latest Debian version (bullseye 11)

grafik

gnome-mpv commented 2 years ago

CTRL+4 no-osd change-list glsl-shaders set "~~/shaders/ .... show-text "Mode A+A (HQ)

~~ doesn't work in Celluloid right now due to #687. It should work if you use absolute paths instead.

gsantner commented 2 years ago

Can WorkingDirectory in .desktop maybe set to something so it resolves correctly? (i.e. workaround)

gnome-mpv commented 2 years ago

Maybe. I guess it could work if you set WorkingDirectory to Celluloid's config dir.

RobotRoss commented 1 year ago
gnome-mpv commented 1 year ago

@RobotRoss Do you see any errors in the log?

chermnyx commented 6 months ago

I can't reproduce it with absolute path to the shaders and vo=gpu-next, the shaders get loaded and look as expected

RobotRoss commented 5 months ago

I decided to revisit this as I've just installed a new system.

Seems like part of the issue is that the flatpak permissions are not set to allow access to the home directory. Using flatseal to manually grant access to the home directory allowed the shader to load.

This was using an absolute path - I haven't tried a relative path.

As an aside, if a shader fails to load (e.g. for the above permissions issue) it'd be very helpful if an error message popped up informing the user of the issue instead of silently failing.