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

[volspotconnect-futurdev] adding ReplayGain Support #119

Closed
cortegedusage opened this issue Jan 11, 2018 · 21 comments
Closed

[volspotconnect-futurdev] adding ReplayGain Support #119

cortegedusage opened this issue Jan 11, 2018 · 21 comments

Comments

@cortegedusage
Copy link

Hi Balbuze,

did you notice Herrernst implemented ReplayGain support.

https://github.com/herrernst/librespot/tree/volume-normalization/src

This would truly be an addition to volspotconnect2

@balbuze
Copy link
Owner

balbuze commented Jan 13, 2018

Hi!
thanks, I'll have a look and if possible add it in the plugin.

@balbuze
Copy link
Owner

balbuze commented Jan 15, 2018

I'm compiling now a version of librespot with ReplayGain... It fails... Retry tomorrow..
But do you think I should add a switch in the webUI to enable/disable it ?

@cortegedusage
Copy link
Author

cortegedusage commented Jan 15, 2018 via email

@balbuze
Copy link
Owner

balbuze commented Jan 15, 2018

ok I have a working version for x64...
Tomorrow I'll try to compil for amrv6 and add the switch !

@balbuze
Copy link
Owner

balbuze commented Jan 16, 2018

@cortegedusage
Copy link
Author

Wow, that's great.

cheers!

@cortegedusage
Copy link
Author

I've been using it today, and it sounds very nice.
but, it seems to crash whenever I stop a song.
It looks active in the app but when i resume
librespot restarts.

play the song:
Jan 17 20:33:21 woonkamer volumio[10197]: INFO:librespot::player: Loading track "Too Late"
Jan 17 20:33:21 woonkamer volumio[10197]: INFO:librespot::player: Track gain: -3.2399979db
Jan 17 20:33:21 woonkamer volumio[10197]: INFO:librespot::player: Track peak: 0.86110026
Jan 17 20:33:21 woonkamer volumio[10197]: INFO:librespot::player: Album gain: -5.0699997db
Jan 17 20:33:21 woonkamer volumio[10197]: INFO:librespot::player: Album peak: 1.0085515
Jan 17 20:33:21 woonkamer volumio[10197]: INFO:librespot::player: Track "Too Late" loaded
Jan 17 20:33:21 woonkamer volumio[10197]: INFO:librespot::player: metadata:{"ARTIST":"Augustus Pablo","TITLE":"Too Late"}

Push pause
Jan 17 20:33:26 woonkamer volumio[10197]: thread '' panicked at 'invalid state', src/player.rs:178
Jan 17 20:33:26 woonkamer volumio[10197]: note: Run with RUST_BACKTRACE=1 for a backtrace.

try to resume

Jan 17 20:33:32 woonkamer volumio[10197]: thread 'main' panicked at 'called Result::unwrap() on an Err value: "SendError(..)"', /checkout/src/libcore/result.rs:859
Jan 17 20:33:32 woonkamer systemd[1]: volspotconnect2.service: main process exited, code=exited, status=101/n/a
Jan 17 20:33:32 woonkamer systemd[1]: Unit volspotconnect2.service entered failed state.
Jan 17 20:33:34 woonkamer systemd[1]: volspotconnect2.service holdoff time over, scheduling restart.
Jan 17 20:33:34 woonkamer systemd[1]: Stopping Volspotconnect2 Daemon...
Jan 17 20:33:34 woonkamer systemd[1]: Starting Volspotconnect2 Daemon...
Jan 17 20:33:34 woonkamer systemd[1]: Started Volspotconnect2 Daemon.
Jan 17 20:33:34 woonkamer volumio[12740]: INFO:librespot: librespot de2b4cc (2017-12-07). Built on 2018-01-15. Build ID: JxrhZJUc
Jan 17 20:33:34 woonkamer volumio[12740]: WARN:mdns: Failed to register IPv6 receiver: Error { repr: Os { code: 19, message: "No such device" } }
Jan 17 20:33:34 woonkamer volumio[12740]: INFO:librespot_core::session: Connecting to AP "gew1-accesspoint-b-k4kn.ap.spotify.com:4070"
Jan 17 20:33:35 woonkamer volumio[12740]: INFO:librespot_core::session: Authenticated as "*********" !
Jan 17 20:33:35 woonkamer volumio[12740]: INFO:librespot::audio_backend::alsa: Using alsa sink
Jan 17 20:33:35 woonkamer volumio[12740]: INFO:librespot_core::session: Country: "NL"

do you know how to run with backtrace?

@balbuze
Copy link
Owner

balbuze commented Jan 19, 2018

Hi,
Sorry for this late answers...I've got some problem with my home network...
for backtrace, just enter "export RUST_BACKTRACE=1" via ssh and restart librespot. But not sure it will help...

@balbuze
Copy link
Owner

balbuze commented Jan 19, 2018

I found something maybe wrong in the code... I'm recompiling librespot to see if better...

@balbuze
Copy link
Owner

balbuze commented Jan 24, 2018

outch ! After a long fight my network problem is solved... So I uploaded a "regular" version of volspotconnect2 including all last changes/improvements. Hope it will be fine....

@cortegedusage
Copy link
Author

did you manage to get the stop error out of it, or is it still present?

I've also encountered some issues with the implementation it seems to clip with very quiet tracks.
I gave this back to herrernst, maybe he can fix this...

happy listening!

@balbuze
Copy link
Owner

balbuze commented Jan 24, 2018

Yes it seems to be fixed.by the code I modified.. I'm using it for several days and didn't experienced any problems....

@cortegedusage
Copy link
Author

cortegedusage commented Jan 24, 2018 via email

@cortegedusage
Copy link
Author

cortegedusage commented Jan 24, 2018 via email

@balbuze
Copy link
Owner

balbuze commented Jan 24, 2018

Do you use volspotconnect2, not volspotconnect-futurdev?

@balbuze
Copy link
Owner

balbuze commented Jan 24, 2018

Or I made a mistake? Can't check now..

@cortegedusage
Copy link
Author

yeah you made a mistake in the install script,
it still points at the futeredev repo on github, so it downloads the old binary.

the new binairy works perfectly, thank you again!!!

you know people get paid to do this sort of things :)

good night

@balbuze
Copy link
Owner

balbuze commented Jan 25, 2018

Damn!
Stupid error...
I can't change it before tonight...

@balbuze
Copy link
Owner

balbuze commented Jan 25, 2018

done!

@cortegedusage
Copy link
Author

cortegedusage commented Jan 25, 2018 via email

@cortegedusage
Copy link
Author

Closing this, thank you for your work...
adding metadata to the ui is the sugar on the cream.

cheers!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants