Skip to content

Conversation

@avelad
Copy link
Member

@avelad avelad commented Dec 2, 2025

This is part to remove dontChooseCodecs config in the future.

@avelad avelad requested a review from joeyparrish December 2, 2025 09:41
@avelad avelad added type: bug Something isn't working correctly priority: P2 Smaller impact or easy workaround component: streaming The issue involves the streaming engine or media source engine implementation labels Dec 2, 2025
@avelad avelad added this to the v5.0 milestone Dec 2, 2025
@avelad
Copy link
Member Author

avelad commented Dec 2, 2025

@joeyparrish this is related to our chat conversation about track filtering

@shaka-bot
Copy link
Collaborator

Incremental code coverage: 100.00%

@avelad avelad requested review from TAhub, matvp91 and tykus160 December 3, 2025 07:20
@avelad avelad merged commit 2c1ab30 into shaka-project:main Dec 3, 2025
84 of 90 checks passed
@avelad avelad deleted the audio-filtering branch December 3, 2025 09:31
@joeyparrish
Copy link
Member

I don't understand the thinking behind this PR, and it is causing a merge conflict with another one I'm working on. (#9452)

Why would we not pick the best supported track out of a set of redundant ones? (Same sample rate, language, role, etc, but different codec and/or bitrate)

@avelad
Copy link
Member Author

avelad commented Dec 4, 2025

I don't understand the thinking behind this PR, and it is causing a merge conflict with another one I'm working on. (#9452)

Why would we not pick the best supported track out of a set of redundant ones? (Same sample rate, language, role, etc, but different codec and/or bitrate)

Apps should be able to choose

avelad added a commit that referenced this pull request Dec 15, 2025
…ead (#9457)

This is part to remove dontChooseCodecs config in the future.
@shaka-bot shaka-bot added the status: archived Archived and locked; will not be updated label Feb 1, 2026
@shaka-project shaka-project locked as resolved and limited conversation to collaborators Feb 1, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

component: streaming The issue involves the streaming engine or media source engine implementation priority: P2 Smaller impact or easy workaround status: archived Archived and locked; will not be updated type: bug Something isn't working correctly

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants