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

MPV integration #12

Open
ninnghazad opened this issue Feb 7, 2019 · 3 comments · May be fixed by #62
Open

MPV integration #12

ninnghazad opened this issue Feb 7, 2019 · 3 comments · May be fixed by #62
Labels
bug Something isn't working
Milestone

Comments

@ninnghazad
Copy link

i understood this to be local version of the webgui which would allow me to play media using mpv.
it does however use whatever player is integrated with chromium it seems.
are there any additional requirements for it to use the mpv installed on the system?
using debian with minimal wm (blackbox).

@ghost
Copy link

ghost commented Feb 25, 2019

Hello @ninnghazad sorry for the late response but there quite a bit of dependency's for this application I don't have all of them written down. But hopefully here is a breakdown of your question I believe.

i understood this to be local version of the webgui which would allow me to play media using mpv.

Yes this allows you to play videos / movies and listen to music on your desktop. But its not a local version of the webgui. This just enables you to view them on your pc without a full fledged browser (if you'd call it that). This is written in electron which only loads the URL of the website and displays it through this frame on your desktop.

it does however use whatever player is integrated with chromium it seems.

That player is identical to the one in the web browser because it is the one in the web browser

are there any additional requirements for it to use the mpv installed on the system?

Yes there are quite a few. The only ones we require you to install on your own are Node JS and Npm. After that npm handles all of our deps through its package manager. It does install quite a bit. By the end of the day it is quite hefty.

@ninnghazad
Copy link
Author

thanks for the information.
but how do i actually get it to use mpv?
i have installed using npm, and can start it and watch stuff - just not using mpv.
a minimal test using the node-mpv that came as a dependency through npm works - it plays files with the correct mpv and it's configs/plugins.
it seems that whatever plugin/script/thingy make electron/chromium use the external player won't work for me. i can see mpvplayer.js getting added to plugins-option on startup, but a few console.logs thrown at the source don't show any mpv* related stuff getting actually called.

@cromefire
Copy link
Contributor

MPV is already integrated, but not working at all (like the rest of the app). I'll fix it soonish (depending on how fast the PRs get reviewed)

@anthonylavado anthonylavado added bug Something isn't working and removed enhancement New feature or request labels May 31, 2020
@cromefire cromefire added this to the 1.0.0 milestone Jun 4, 2020
@cromefire cromefire linked a pull request Aug 19, 2020 that will close this issue
@cromefire cromefire changed the title MPV ? MPV integration Aug 19, 2020
@cromefire cromefire linked a pull request Aug 19, 2020 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Development

Successfully merging a pull request may close this issue.

3 participants