Closed vikyi closed 7 months ago
Thanks for letting me know of this issue. I've double checked and can confirm that there is a problem with running MPC-HC (and MPC-BE) without any file.
The MPC-HC code is some of the oldest code in Syncplay and I had very little to do with its development. As such, I'm not entirely sure why the error is happening.
However, having had quick look over the code my best guess is that the issue arises from the code having been written for Python 2, then ported into Python 3, and then we moved to a newer version of Python 3. Also, originally people were generally expected to run Syncplay with the file to be played which was how things worked before shared playlists.
If someone can figure out how to get it to work and makes a pull request containing a fix then we can continue to support MPC-HC and MPC-BE for the timebeing.
Otherwise, it might be time for Syncplay to drop MPC-HC and MPC-BE support and encourage people to use mpv or mpv.net instead as those work far better with Syncplay (not least because they support chat messages and Syncplay commands to be written straight into the player UI).
I think I've fixed the issue. You can download the latest development build with the patch at https://github.com/Syncplay/syncplay/actions/runs/7732159464 and let me know if it works
P.S. As there is no official Syncplay maintainer for MPC-HC support my recommendation is that people move to mpv/mpv.net (which in any case has better Syncplay integration/support).
Thanks for your reply. I have tried this 1.7.2 development, but the bug is still exist. Now I always add an arbitrary file path when I startup the Syncplay client, so that I can use MPC-HC as usual. (However, the filename after username will be unreadable) If Syncplay update to 1.7.2 stable and drop MPC-HC support. At that time I will choose mpv.
@vikyi I've hopefully fixed your MPC-HC issue with https://github.com/Syncplay/syncplay/actions/runs/7820071508 - if you still have the issue then tell me precisely which version of MPC-HC you are using. I am using mpc-hc.2.1.3.x64 for testing although just checked https://github.com/clsid2/mpc-hc/releases and there is now a version 2.1.4.
@Et0h I've test this new version and everything works properly. Thanks for your fix.
Describe the bug When i add file to sharedplaylist, Syncplay will loop try to open the file, then disconnect from server and retry. And MPC-HC can't open the file correctly.
To Reproduce Steps to reproduce the behavior:
Expected behavior MPC-HC open the file correctly and Syncplay run as normal, wait everyone ready
Screenshots the loop message like this![Screenshot](https://github.com/Syncplay/syncplay/assets/53124987/cc00a94c-53c4-4418-aa4a-891a4129991f)
Version and platform:
Additional context MPC-HC using madvr and lav splitter Only if i set the empty file path will see this bug, like this screenshot
While i give it a filepath, the chatbox will still show warnings, but then i can use Syncplay correctly.
![sc2](https://github.com/Syncplay/syncplay/assets/53124987/d0e7f0d4-5de5-4632-888c-37c85321db10)
the chatbox warning message
.syncplay.log