-
Notifications
You must be signed in to change notification settings - Fork 60
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
[volspotconnect2] Status inconsistent #145
Comments
I have the exact or very similar issue. Volspotconnect 2 works great with the Windows 10 Spotify App, but doesn't sync properly with Android. The Android App can connect and dispatch commands to the Volumio player, but it doesn't always register updates back from Volumio. I can immediately see this when using the forward and back buttons on the Volumio Web Interface (volumio.local). I am using the 0.6.6 version of the plugin and Volumio 2.368. I also tried the plugin as of commit 121c6c1 (Fix sporadic seeking) just in case it had been fixed. |
Hmm, could you also provide the andorid app version you're on? There have been enough reports of this now, that I believe there is something going on that needs some digging! |
Android Nougat 7.1.1 is what I am using. |
Spotify App version 8.4.44.661 |
Unfortunately i experience the same as above..
But it seems like a bug in the app, because my Chromecast is Having the
same results
Op 14 mrt. 2018 5:36 a.m. schreef "mneedes" <notifications@github.com>:
… Spotify App version 8.4.44.661
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
<#145 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AV69TdGmzjZSEEjbNjzqD_vwgaYEdy9Fks5teJ43gaJpZM4Sj5Zq>
.
|
and the same when trying to control the spotify web player. |
reverting back to previous version of android app seems to have solved the problem |
@cortegedusage Are you by any chance on Android 8? |
@ashthespy nope android 7.1.2 but it works for me again. |
Android 8 (Oneplus 5) here, Spotify 8.4.44.661 |
@MateEke |
@cortegedusage Interesting that a downgrade solved your issue ;-.) I will try to reproduce, just upgraded (Android 8 and |
@ashthespy |
8.4.45.633 still have the issue. |
and indeed, I have to admit the issue is still comming back. |
I'm afraid the new version the Spotify 8.4.45.633 will cause us a lot of headache... |
@cortegedusage not sure, but I think it could be something to do with how we send back the current state to Spotify (inside librespot). I think its fixed with my latest librespot-armhf.tar.xz build. I am on (Android 8 and |
@ash
Tried yesterday to replace the binairie with yours, but that left me whith
a non working connect device that didn't show up in the app. No time to
investigate further yesterday(damn real life...)thanks for the reply.
Op 17 mrt. 2018 2:41 p.m. schreef "Ash" <notifications@github.com>:
@cortegedusage <https://github.com/cortegedusage> not sure, but I think it
could be something to do with how we send back the current state to Spotify
(inside librespot). I *think* its fixed with my latest
librespot-armhf.tar.xz
<https://files.gitter.im/ashthespy/cezk/librespot-armhf.tar.xz> build.
I am on (Android 8 and 8.4.44.663 armv7) and *seems* to work fine :-)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#145 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AV69TRyrhwKtK1toLzJPEuaCmp_wi5yoks5tfRJvgaJpZM4Sj5Zq>
.
|
You a to set librespot as executable. Sudo chmod +x librespot |
I believe the rights where already good. It did run grom the cli
Op zo 18 mrt. 2018 12:04 schreef balbuze <notifications@github.com>:
… You a to set librespot as executable. Sudo chmod +x librespot
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#145 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AV69TSJq7wK9YxH2iIlm94OopLNuZ2xxks5tfj9XgaJpZM4Sj5Zq>
.
|
@cortegedusage apologies, this isn't a drop in replacement - but is meant to go along with #156 We have not yet packaged it up in a neat zip yet. However - since I just was testing on my pi0, here is a small snippet that will sync you up to those changes.
Wait while the plugin helper compresses the file again 😬
Enable the plugin from UI! |
@ash yeah thougt so but couldn't find time to check.
(You might think a Little piece of code playing music is a pain in the ass.
I can assure you, real people playing music are even worse)
Will check your suggestion as soon as i can.
Op zo 18 mrt. 2018 15:06 schreef Ash <notifications@github.com>:
… @cortegedusage <https://github.com/cortegedusage> apologies, this isn't a
drop in replacement - but is meant to go along with #156
<#156> We have not yet
packaged it up in a neat zip yet.
However - since I just was testing on my pi0, here is a small snippet that
will sync you up to those changes.
PS - this is experimental/ in progress work on the UI, be warned! :-)
Uninstall the plugin from the plugin manager, then
$ wget https://files.gitter.im/ashthespy/Xrht/volspotconnect2.zip
$ mkdir ./volspotconnect2
$ miniunzip volspotconnect2.zip -d ./volspotconnect2
$ cd ./volspotconnect2
$ volumio plugin install
Wait while the plugin helper compresses the file again 😬
$ cd /data/plugins/music_service/volspotconnect2/
$ wget https://files.gitter.im/ashthespy/cezk/librespot-armhf.tar.xz
$ mv librespot librespot.bak
$ tar -xf librespot-armhf.tar.xz --strip-components 4
Enable the plugin from UI!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#145 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AV69TU3Jg6UPKEbDlbEP-0FhShB7PAbJks5tfmnrgaJpZM4Sj5Zq>
.
|
hmm followed your lead, but having the same issues as I had before uninstalling. you sure I downloaded the right binairy? |
@cortegedusage sorry, if I am being a bit dense, but if I understood your problem:
Hmm - that is strange. |
-rwxr-xr-x 1 volumio volumio 14127488 Mar 15 22:37 librespot this is the one I downloaded like you said. this is the one that I had to backup |
For this problem in Android App 8.4.45.633 , Do we know if it's a bug or intended change in the Spotify App? ios and windows 10 are working perfectly. thanks! |
@nanomesher my wife and coworkers, who are on IOS aren't experiencing this... weird |
@cortegedusage @nanomesher I am unsure, but I do believe the new binaries we are building should fix it. I don't see any issues on my end - but then I don't recall seeing this issue to begin with :-s |
@ashthespy |
Or - just to make sure -- disable the plugin and
|
In related news - I think it's time to make a separate repo for Volumio specific modifications of librespot ( Vollibrespot | VolumioSpotify | VolumioConnect | ?) and end the random binaries floating around in gitter space! |
@ashthespy good Idea I'd vote for Volibrespot apart from that. is what I'm on now same as yours running PI3 ... anyhow behaviour is the same.. cheers! |
@cortedusage I tried out the new librespot binary but I had to hack the startconnect.sh script to remove the ''--onstart" flag from the librespot command line (the new librespot doesn't support that option?). Problem is, the volspotconnect2 plugin will rewrite that script sometimes so that change won't persist, however it did allow me to temporarily test the binary. Unfortunately it didn't work for me -- but I didn't attempt to integrate #156 . Maybe I need this also? |
#156 doesnt have onstart in startconnect.sh aparently. |
@mneedes I add my changes into But, lets wait for balbuze to test out - he also reported seeing these issues with the Android client |
@ashthespy replacing it with the new one doesn't do anything. apart from that the volume normalisation flag doesn't work. something seems wrong because if I add it by hand it works as expected (volume normalization) But still having the same issues... |
on a side note I might say that in the plugin manager it says it is version 0.6.4 |
@cortegedusage I just uploaded a dev version with all the last mod. Let us know how it works. |
@balbuze & @ashthespy |
some new behaviour is that there is a 5 second delay between stopping mpd and |
Ok the delay is only there when pushing play on the app. and I'm getting occasional alsa busy errors, this is occasionally when switching from librespot to mpd, by clicking on a song. |
and apart from that the time on the app isn't always sync with the real time, it takes some time to |
@cortegedusage There are few limitations of the current hooks I've exposed in librespot. But, I have a PR over at librespot that should make the hooks exposed to Volumio more consistent and reliable. Then the UI stuff becomes trivial to solve. This limitation leads to a small check to determine if Volspotconnect is the active service before pushing the state - else the Volumio state machine rejects it leading to other issues (Alsa busy, time syncing, etc) I know it's not perfect - but we are working on it! :-) |
ah ok, well not perfect is good enough for me! |
The volspotconnect2.zip provided by ash solves my Android sync issues. |
@mneedes my zip was an unofficial quick test package, please switch to the dev version from this repo - it has more goodness in there ;-) |
Sorry to revive an old thread, but I experience this exact issue. Has the sync fix not made it into volumio distribution yet? |
If I use android phone with the newest version from volumio repo (2018-3-8) the currently played song doesn't appear on the phone, and I don't have any feedback about the play/pause state, etc. on the phone. With the windows 10 version of Spotify it seems fine.
The text was updated successfully, but these errors were encountered: