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

AESinkAudioTrack: Help broken firmwares to make kodi ignore broken delay #24597

Merged
merged 1 commit into from Feb 3, 2024

Conversation

fritsch
Copy link
Member

@fritsch fritsch commented Jan 28, 2024

Kodi can supervise the sink for being stuck, which means: if you open a sink with 160 ms buffer and it does not move while eating more than twice of the buffer's audio data, it is considered a bug. Sadly - there is a whole lot of broken firmwares out there, one of the leards: Ugoos. Which is the reason I make this opt-in. That means that FireTV Cube 3rd Gen and others would need to enable this - once again - via an advanced settings.

<advancedsettings>
<audio>
<superviseaudiodelay>true</superviseaudiodelay> 
</audio>
</advancedsettings>

See forum report: https://forum.kodi.tv/showthread.php?tid=376090 - I briefly informed in Slack some weeks ago and just waited to see more broken devices. Yeah, seen now the Ugoos twice ...

For now it's only the Ugoos, while I did not get other reports. The alternative to this change would be a "negative list" .... which we don't want to maintain. Sad thing is, that FireTV Cube 3rd Gen and the other boxes that need this are a bigger number.

Open for suggestions.

@fritsch fritsch force-pushed the brokendelay branch 2 times, most recently from 6fc5510 to 77a825b Compare January 28, 2024 18:58
@fritsch fritsch added Type: Fix non-breaking change which fixes an issue Wiki: Needed v21 Omega labels Jan 28, 2024
@fritsch fritsch added this to the Omega 21.0 Beta 3 milestone Jan 28, 2024
@fritsch fritsch removed this from the Omega 21.0 Beta 3 milestone Jan 28, 2024
@fritsch
Copy link
Member Author

fritsch commented Jan 28, 2024

Let's keep that just here ... want to see if it's really needed or could be inversed ... after next beta lots of additional testers will appear ...

@fritsch fritsch added this to the Omega 21.0 Beta 3 milestone Jan 28, 2024
Copy link
Member

@thexai thexai left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approved conditioned to improve code comment a bit

Kodi can supervise the sink for being stuck, which means: if you open a sink
with 160 ms buffer and it does not move while eating more than twice of the
buffer's audio data, it is considered a bug. Sadly - there is a whole
lot of broken firmwares out there, one of the leards: Ugoos. Which is the
reason I make disabled by default. That means that FireTV Cube 3rd Gen and
others would need to enable this - once again - via an advanced settings.

<advancedsettings>
<audio>
<superviseaudiodelay>true</superviseaudiodelay>
<audio>
</advancedsettings>

Co-authored-by: thexai <58434170+thexai@users.noreply.github.com>
@fritsch
Copy link
Member Author

fritsch commented Jan 31, 2024

jenkins build this please

@fuzzard fuzzard merged commit 264b44d into xbmc:master Feb 3, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants