python-mpv-jsonipc icon indicating copy to clipboard operation
python-mpv-jsonipc copied to clipboard

Number arguments to mpv are not parsed correctly

Open alfhj opened this issue 1 year ago • 0 comments

Number arguments to the MPV constructor are converted to "yes"/"no" when they should't be.

Example:

extra_args = {
    "video-scale-x": 1
}
MPV(**extra_args)

Expected Behavior

MPV starts

Current Behavior

python_mpv_jsonipc.MPVError: MPV process retry limit reached.

Possible Solution

Change MPVProcess_mpv_fmt() to if data is True: and elif data is False

Also: is it possible to include the mpv error in the traceback?

alfhj avatar Apr 18 '24 12:04 alfhj