feat(dubbing): 按角色批量分配音色并支持单句覆盖 - #423
Merged
buxuku merged 2 commits intoAug 6, 2026
Merged
Conversation
buxuku
requested changes
Aug 5, 2026
buxuku
left a comment
Owner
There was a problem hiding this comment.
感谢PR, 有两个 P1 希望修复一下
[P1] 允许 headless 配音回退到全局音色 — main/helpers/dubbing/dubbingProcessor.ts:823-829
流水线在创建/恢复带 file.proofreadDataFile 的会话时会加载多角色元数据,但在 runDubStage 调用 runDubbingBatch 之前,没有 UI 步骤去填充 session.speakerVoiceMap。因此,对任意多角色 sidecar,该门禁会直接中止自动配音并报「还有 N 个角色未选择音色」,而不是使用任务里已配置的全局音色。建议在 headless/流水线场景跳过该要求,或在开始前将未映射角色初始化为全局哨兵值。
[P1] 在流水线重建路径上拦截 stale 角色音频 — main/helpers/dubbing/dubbingProcessor.ts:1251-1254
当前 stale 音频检查只覆盖手动的 exportDubbing IPC 路径;检查员放行 / 流水线续跑则通过 rebuildDubTrackForFile → buildDubTrack 直接产出成片。若审查者在配音工作台修改了角色音色但未重生成,标记为 needsUpdate 的 cue 仍会静默进入流水线输出。需要在共享的 track 构建/重建路径上执行同样的检查。
Contributor
Author
|
已在追加 commit
验证:
|
buxuku
approved these changes
Aug 6, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
变更内容
安全与兼容说明
验证
npm run test:dubbing:172/172npm run test:proofread-speakers:31/31npm run test:speaker-diarization:配置 10/10,对齐 30/30npm run check:i18nnpm run build:Renderer + Main production build 通过git diff --check尚未覆盖
npm run test:pipeline在 Windows 上仍有 4 个 POSIX 路径分隔符断言失败;已在未修改的upstream/main复现,非本 PR 引入Closes #421