Skip to content

This issue was moved to a discussion.

You can continue the conversation there. Go to discussion →

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

Feedback and Suggestions #5

Closed
pwt opened this issue May 9, 2019 · 5 comments
Closed

Feedback and Suggestions #5

pwt opened this issue May 9, 2019 · 5 comments
Assignees

Comments

@pwt
Copy link
Owner

pwt commented May 9, 2019

Feel free to use this issue to leave feedback and make suggestions.

@polyzen
Copy link

polyzen commented Feb 25, 2020

Hi @pwt, I'm wondering why 1000:1000:f is used as the default latency here when 1000:2000 is suggested in AirConnect's readme for Sonos.

@pwt
Copy link
Owner Author

pwt commented Feb 25, 2020

Hi @polyzen. It's just based on experience, really: I've used AirConnect extensively in multiple network environments and I find that latency settings of 750:750:f provide very reliable buffering with Sonos. I make it 1000:1000:f as the default for the Docker image just to be conservative.

You can adjust this back to the recommended defaults using -e LATENCY='1000:2000' (or whatever else you wish) on the Docker command line to start the container. See the 'Other configuration settings' section in the README.

The :f parameter just makes sure that silence frames are sent if the incoming stream is interrupted. This is to mitigate against the speaker closing its connection if there's a temporary interruption on the sending side.

@pwt pwt self-assigned this Feb 25, 2020
@jlambert121
Copy link

Just wanted to leave some feedback - thanks for this! Worked exactly as expected and I didn't have to write the Dockerfile or build the image!

@pwt
Copy link
Owner Author

pwt commented Dec 19, 2020

Just wanted to leave some feedback - thanks for this! Worked exactly as expected and I didn't have to write the Dockerfile or build the image!

Great! Glad it's useful. Thanks for the feedback.

@pwt
Copy link
Owner Author

pwt commented Apr 1, 2021

Closing this issue in favour of using Discussions instead.

@pwt pwt closed this as completed Apr 1, 2021
Repository owner locked and limited conversation to collaborators Apr 2, 2021

This issue was moved to a discussion.

You can continue the conversation there. Go to discussion →

Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants