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

Couple of suggestions #9

Open
spinktvis opened this issue Oct 21, 2017 · 2 comments
Open

Couple of suggestions #9

spinktvis opened this issue Oct 21, 2017 · 2 comments

Comments

@spinktvis
Copy link

spinktvis commented Oct 21, 2017

You asked for some feedback and input, so here i am. These are of course my personal preferences.

  • Add the following to input.conf, i just can't live without the mouse scroller changing volume:
    AXIS_UP add volume +2 ; show_text "Volume: ${volume}"
    AXIS_DOWN add volume -2 ; show_text "Volume: ${volume}"

  • In mpv.conf, this is to centre the mpv screen in the middle of the monitor/tv each time it opens:
    geometry=50%:50%

  • Add the autoload.lua in your scripts folder, so that mpv automatically plays the next file inside the folder. Since i binge watch a lot of series and anime, i am to lazy to each time open the next file manually:
    https://github.com/mpv-player/mpv/blob/master/TOOLS/lua/autoload.lua

  • Update the lua scripts with the latest versions, most of them are getting out of date and some started showing warnings when I run mpv from shell.

  • Not sure how feasible the following is, but your auto options script, can it be made to detect the GPU? I'm asking because I've got a Intel GPU and I need to adjust some of the options within the script because for example the hwdec auto-copy doesn't work at all.

  • Would it also be possible to drop the acpitool dependency and use acpid instead? Since acpitool doesn't seem te be developed anymore. And acpid is in arch community hence no need for me to build it from scratch.

@ahjolinna
Copy link
Owner

sorry for the late response, about to your suggestions...

  • yes, I can add those settings to my conf (also autoload.lua) when the next "stable" MPV release has arrived. (why then? it's my policy)

  • I know "some" of the lua scripts are out of date, I will try remember to get them up to date when the next mpv release is out.

  • yeah, currently my script doesn't detect the gpu in use, it only checks if you are using battery or AC power, I would like to add that "gpu detection" feature to the script. ...BUT...
    as there has been so changes in mpv & ffmpeg in a year or so and I'm currently waiting for cuvid hwdec support to arrive (...which is the nvidia's vdpau replacement, and because vdpau has been deprecated I'm not sure if AMD will stay with VAAPI or will they do something new...again.) ...so I really haven't had any enthusiasm to add that "gpu detect" support to my script...yet (maybe when ffmpeg 3.5/4.0 has arrived)

  • The "intel gpu /auto-copy issue", well unfortunately intel drivers are little bit hit and miss ...and the "-copy" option should be better...well in general (maybe try to check if it's a mpv/ffmpeg or driver problem?). I really can't test this myself as I only have Nvidia GTX970, this oc means that my focus will be on features that benefits nvidia, but I'm always open to suggestion.

  • acpitool thing, well it's not needed anymore as I went for basic acpi : commit


btw. if you don't know you can add the conf file to /home/<username>/.config/mpv/ instead of /etc/mpv/ to make your own edits ...you can have both, but mpv will use conf's in the "home folder" ...if I remember correctly

@spinktvis
Copy link
Author

Thanks for the reply. Looking forward to the new release. Yes, i have all the configs in ./, much easier to configure them like that.

About the auto-copy, when running mpv from the command line it actually gives a warning to not use auto-copy and instead specify the GPU renderer directly in the config file. So i am using vaapi-copy which works perfect.

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

No branches or pull requests

2 participants