MPVNet Support

[seniorm0ment]

Would love to have MPVNet support. It just errors when trying to use it.

[Et0h]

I can confirm that it gives the following error:

Exception

version: MPV_ERROR_PROPERTY_NOT_FOUND

System.Exception: version: MPV_ERROR_PROPERTY_NOT_FOUND
   at mpvnet.mp.set_property_string(String name, String value, Boolean throwOnException) in D:\Projekte\CS\mpv.net\mpv.net\mpv\mp.cs:line 420
   at mpvnet.mp.ProcessCommandLine() in D:\Projekte\CS\mpv.net\mpv.net\mpv\mp.cs:line 544

I think if you want MPVNet to support Syncplay then you will need to discuss this with the MPVNet developers.

[stax76]

mpv.net has no switch called --version

System.Exception: version: MPV_ERROR_PROPERTY_NOT_FOUND

[stax76]

@Et0h

There is a property called 'mpv-version' you can get via json ipc.

[Et0h]

@stax76 For the response I gave my first post I disabled the version check code, and I got the error message noted above. Syncplay does not communicate with mpv using JSON IPC, it uses the regular STDIN/STDOUT system.

[stax76]

I knew mpv reacts to keys pressed in a console window but that you can send input commands to stdin is new to me. It's hard to tell how hard it would be to replicate this in mpv.net. You seem to send --script, there would be a property called 'scripts' but it can only be set before libmpv initialization and mpv.net parses the command line after initialization. My python knowledge being only very basic don't help either, there might not be an easy way for us to make this work.

[stax76]

well, I figured there is input-terminal

            if (App.IsStartedFromTerminal)
            {
                set_property_string("terminal", "yes");
                set_property_string("input-terminal", "yes");
                set_property_string("msg-level", "osd/libass=fatal");
            }

maybe it's not that hard, we'll see.

[stax76]

I got this working... Just don't invoke an instance to get the version and try my next release.

[stax76]

https://www.dropbox.com/sh/4ctl2y928xkak4f/AAADEZj_hFpGQaNOdd3yqcAHa?dl=0

https://1drv.ms/f/s!ArwKS_ZUR01gySQIE1QCL2i9F_uR

[Et0h]

@stax76 Nice one - it works! :)

mpvnet support can now be added to Syncplay.

[Et0h]

Okay, initial support should be added by 187c8d8 which you can test out at https://bintray.com/syncplay/Syncplay/Syncplay/v1.6.5#files

[Et0h]

@Z3ROGRAV1TYx Can you please test the test build of mpvnet with the test build of Syncplay and let me know if it works for you? Seems to be working fine for me.

[fmohican]

It's work! Smooth!!!
Tested with last version of MPV.NET (5.2.1, from github https://github.com/stax76/mpv.net/releases/tag/5.2.1 ) && V1.6.5 Dev.