Skip to content
This repository has been archived by the owner on Oct 26, 2024. It is now read-only.

✨ feat: add adjust audio quality support #35

Merged
merged 3 commits into from
Oct 1, 2020
Merged

Conversation

SigureMo
Copy link
Member

@SigureMo SigureMo commented Sep 26, 2020

related with #34

概述

本 PR 的类型(至少选择一个)

  • ✨ feat: 添加新功能
  • 🐛 fix: 修复 bug
  • 📝 docs: 仅对文档进行修改
  • 🎨 style: 对代码语义无影响的格式修改(如去除无用空格、格式化等等修改)
  • ♻️ refactor: 代码重构(即不是新增功能,也不是修改 bug 的代码变动)
  • ⚡ perf: 提高性能的代码修改
  • ✅ test: 测试用例添加及修改
  • 🔨 build: 影响构建系统或外部依赖关系的更改
  • 👷 ci: 更改 CI 配置文件和脚本
  • ❓ chore: 其它不涉及源码以及测试的修改

@vercel
Copy link

vercel bot commented Sep 26, 2020

This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployments, click below or on the icon next to each commit.

bilili – ./docs

🔍 Inspect: https://vercel.com/siguremo/bilili/nzj9w7qkq
✅ Preview: https://bilili-git-single-stream.siguremo.vercel.app

bilipi – ./

🔍 Inspect: https://vercel.com/siguremo/bilipi/jay5ctd6o
✅ Preview: https://bilipi-git-single-stream.siguremo.vercel.app

@SigureMo
Copy link
Member Author

SigureMo commented Sep 26, 2020

WIP

进度

  • 基本音频质量调节

已知问题

  • 理论上,如果某个视频只存在音频则会因为错误的 quality 参数而崩溃,但这种应该不会有吧……有的话直接放音乐了啊,就暂不考虑了

@SigureMo
Copy link
Member Author

关于音频的质量等级,我目前使用的是 B 站使用的 302803023230216 代号(暂时只发现这三个),分别代表的码率为 320kbps128kbps64kbps,这些代号的具体意义我不太懂,就和视频的 120116 等等那样

https://github.com/SigureMo/bilili/blob/698bed74833ad719d4dea7a511e1546b3557b934/bilili/quality.py#L71-L85

现在的使用方法就是

bilili <url> --quality 116 --audio-quality 30232

音频应该很少调节吧,没加短名称

@antatura 你看这样的音频码率调节是否可以呀,有什么问题尽管提出呀,单一媒体流的还没做,之后我再做

@SigureMo SigureMo changed the title ✨ feat: 添加单一媒体流支持以及音频码率调节机制 ✨ feat: 添加音频码率调节机制 Sep 29, 2020
@SigureMo
Copy link
Member Author

这个 PR 就只做音频调节了,单一媒体流以后用别的 PR 好了

@SigureMo SigureMo changed the title ✨ feat: 添加音频码率调节机制 ✨ feat: add adjust audio quality support Oct 1, 2020
@SigureMo SigureMo merged commit 19248e4 into master Oct 1, 2020
@SigureMo SigureMo deleted the single-stream branch October 1, 2020 08:16
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant