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

Enchanced audio selection #4287

Closed

Conversation

hikkamorii
Copy link

@hikkamorii hikkamorii commented Nov 23, 2023

This change should address issues #4235 and #2513

To be exact:

  • When listening to audio in listening mode, codec and quality are specified in the list (i.e. "opus @ 64k", "mp4 @ 128k")
  • When viewing video in DASH mode, audio quality can be selected based on the bitrate (it's not necessary to have codec listed, as, at least for now, support for Opus is not enabled) (i.e. "64k", "128k", etc.)

Daniil Hikkamorii added 2 commits November 23, 2023 19:35
With this change, instdead of showing bitrate in bps to the user, user
will see pure bitrate in kbps or codec and bitrate for when listening
mode is used.
Previously, when using download feature, opus audio would be listed as
audio/webm. This may be confusing and so exact codec is specified.
@hikkamorii hikkamorii requested a review from a team as a code owner November 23, 2023 17:42
@hikkamorii hikkamorii requested review from syeopite and removed request for a team November 23, 2023 17:42
@unixfox
Copy link
Member

unixfox commented Nov 27, 2023

May conflict with #3620

@hikkamorii
Copy link
Author

Indeed, they conflict.

Copy link

This pull request has been automatically marked as stale and will be closed in 30 days because it has not had recent activity and is much likely abandoned or outdated. If you think this pull request is still relevant and applicable, you just have to post a comment and it will be unmarked.

@github-actions github-actions bot added the stale label Feb 26, 2024
@github-actions github-actions bot closed this Mar 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants