fix(#463): 直通模式调速失效 - final_duration未考虑调速导致减速场景被截断 #625

Merged

auto-approve-bot merged 2 commits from fix/463-pass-through-speed into develop 2026-07-20 11:41:09 +08:00

Conversation 4 Commits 2 Files Changed 2

+89 -2

xiaoxia commented 2026-07-20 10:04:19 +08:00

Owner

Copy Link

Copy Source

问题

直通模式（pass_through）下调速不生效，减速场景视频被截断。

根因

_render_pass_through 中 final_duration 用原始 effective_duration 计算，没有考虑调速（setpts/atempo）导致的实际输出时长变化：

• 减速场景（speed<1）：调速后视频变长，但 -t 仍用原始时长截断 → 后半段丢失
• 加速场景（speed>1）：调速后视频变短，-t 值大于实际时长 → 看起来正常但逻辑错误

修复

改用 _clip_adjusted_duration(clip) 计算调速后的实际时长作为 final_duration 基准，与 video_duration 取较小值。

测试

新增3个单元测试：

• test_pass_through_speed_up_correct_duration - 加速场景时长正确（5s）
• test_pass_through_slow_down_correct_duration - 减速场景时长正确（20s，不被截断）
• test_pass_through_speed_with_video_duration_cap - video_duration截断逻辑正确

108个测试全绿。

## 问题 直通模式（pass_through）下调速不生效，减速场景视频被截断。 ## 根因 `_render_pass_through` 中 `final_duration` 用原始 `effective_duration` 计算，没有考虑调速（setpts/atempo）导致的实际输出时长变化： - 减速场景（speed<1）：调速后视频变长，但 `-t` 仍用原始时长截断 → 后半段丢失 - 加速场景（speed>1）：调速后视频变短，`-t` 值大于实际时长 → 看起来正常但逻辑错误 ## 修复 改用 `_clip_adjusted_duration(clip)` 计算调速后的实际时长作为 `final_duration` 基准，与 `video_duration` 取较小值。 ## 测试 新增3个单元测试： - `test_pass_through_speed_up_correct_duration` - 加速场景时长正确（5s） - `test_pass_through_slow_down_correct_duration` - 减速场景时长正确（20s，不被截断） - `test_pass_through_speed_with_video_duration_cap` - video_duration截断逻辑正确 108个测试全绿。

xiaoxia added 1 commit 2026-07-20 10:04:19 +08:00

fix(#463): 直通模式调速失效 - final_duration未考虑调速导致减速场景被截断 ... 2680df478d

根因：_render_pass_through 中 final_duration 用原始 effective_duration 计算，
      没有考虑调速（setpts/atempo）导致的实际输出时长变化。
      减速场景（speed<1）下视频被 -t 错误截断，加速场景因截断值大于实际
      时长而表现正常但逻辑错误。

修复：改用 _clip_adjusted_duration(clip) 计算调速后的实际时长作为
      final_duration 的基准，与 video_duration 取较小值。

新增3个单元测试覆盖：加速时长、减速时长、video_duration截断。

xiaoxia added 1 commit 2026-07-20 11:31:05 +08:00

style: black format test_unified_render_service.py b45c24cfa8

auto-approve-bot commented 2026-07-20 11:33:45 +08:00

Collaborator

Copy Link

Copy Source

代码审查结果 - PR #625

⚠️ 问题（0个需要修改）

无

💡 建议（0个可选）

无

---

✅ 格式检查通过 | ✅ 逻辑审查需修改 | ✅ 性能

---

_{🤖 由 AI 代码审查机器人自动生成 | 2026-07-20 11:33:45 | 模型:}

## 代码审查结果 - PR #625 ### ⚠️ 问题（0个需要修改） 无 ### 💡 建议（0个可选） 无 --- ✅ 格式检查通过 | ✅ 逻辑审查需修改 | ✅ 性能 --- _{🤖 由 AI 代码审查机器人自动生成 | 2026-07-20 11:33:45 | 模型:} <!-- AI_CODE_REVIEW_AUTO_COMMENT -->

auto-approve-bot approved these changes 2026-07-20 11:37:33 +08:00

auto-approve-bot left a comment

Collaborator

Copy Link

Copy Source

CI全绿，自动审批通过。

CI全绿，自动审批通过。

auto-approve-bot approved these changes 2026-07-20 11:37:34 +08:00

auto-approve-bot left a comment

Collaborator

Copy Link

Copy Source

CI全绿，自动审批通过。

CI全绿，自动审批通过。

auto-approve-bot merged commit 82929fcd10 into develop 2026-07-20 11:41:09 +08:00

auto-approve-bot deleted branch fix/463-pass-through-speed 2026-07-20 11:41:09 +08:00

gitea-actions bot commented 2026-07-20 11:41:37 +08:00

Copy Link

Copy Source

🗑️ 预览环境已清理

PR #625 已关闭或合并，对应的预览环境已被清理。

如有需要，可以重新打开 PR 来重新生成预览环境。

🗑️ **预览环境已清理** PR #625 已关闭或合并，对应的预览环境已被清理。 > 如有需要，可以重新打开 PR 来重新生成预览环境。

Sign in to join this conversation.

Reviewers

No Reviewers

auto-approve-bot

Labels

Clear labels

P0

阻塞性，必须立即处理

P1

高优先级

P2

中优先级

P3

低优先级/后考虑

architecture

架构设计

backend

后端相关

blocked

阻塞中

bug

Bug修复

ci-bug

CI本身的bug

ci-optimization

CI优化类任务

ci-pipeline

CI流水线/workflow相关

cleanup

代码清理

devops

运维/部署/CI

documentation

文档相关

feature

新功能

frontend

前端相关

question

需要讨论确认

ready

PR CI通过，可合并

refactor

代码重构

No Label

Milestone

No items

No Milestone

Projects

Clear projects

No project

Assignees

Clear assignees

auto-approve-bot xiaoxia

No Assignees

3 Participants

Notifications

Subscribe

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: xiaoxia/xiaoxia-saas#625