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

Proposal for Auto-Activate New speaker function #518

Open
LordMyschkin opened this issue Apr 3, 2018 · 7 comments
Open

Proposal for Auto-Activate New speaker function #518

LordMyschkin opened this issue Apr 3, 2018 · 7 comments

Comments

@LordMyschkin
Copy link
Contributor

I just want to propose a feature maybe not only I do really miss: When playing music and changing the room, I first switch on the speaker in the new room, then open whatever remote I use and activate the newly found speaker.
Would‘nt it be generally convenient if - when playing music by forked-daapd at the moment - the speaker could be activated as soon as it is detected in the network? Maybe as an option in forked-daapd.conf?

@ejurgensen
Copy link
Member

Very good suggestion, I could also make use of that myself. I will look into it - hope it can be done easily.

@ejurgensen
Copy link
Member

ejurgensen commented Apr 14, 2018

I made an attempt at the change in the above, but in my setup it doesn't work. It seems forked-daapd ends up attempting to connect too quickly, and the speaker returns "500 Internal server error". I also see forked-daapd sometimes crashing, because during startup my speaker quickly (mdns-wise) appears, disappears and then permanently reappears again. Forked-daapd tries to connect already at the first appearance, and it then crashes while it is trying to connect to a device that briefly vanishes.

So it wasn't as easy as I hoped...

@LordMyschkin
Copy link
Contributor Author

I just want to tell you that I solved the problem for me by a small python script running in the backgound scanning for new speakers and activating them if forked-daapd is playing at that moment.
I am quite happy with the result, running on a raspberry3.
enable_speakers.zip

@ejurgensen
Copy link
Member

Thanks, good to hear. Sorry I wasn't able to build this in.

@LordMyschkin
Copy link
Contributor Author

Thank you for looking into this, anyway. May we close or should I leave that one open?

@ejurgensen
Copy link
Member

Let's keep it open, maybe I can get it done some other time.

@jshep321
Copy link

Good idea, but I think this could be a bad idea to enable by default. If speaker clients or wifi router/repeaters start dropping out temporarily, having music automagically start in certain rooms could lead to undesired behavior and quite negative... uh... marital discourse.

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

No branches or pull requests

3 participants