xiaoxia/xiaoxia-saas
1
0
Fork 0
Code Issues 10 Pull Requests 2 Actions Packages Projects Releases 7 Wiki Activity

perf(#1341): 封面生成改用 TimeInterval 策略（避免 OOM，提高稳定性） #1351

Merged
auto-approve-bot merged 1 commits from optimize/cover-use-timeinterval into develop 2026-08-13 00:36:00 +08:00
Conversation 5 Commits 1 Files Changed 1
+1 -1
xiaoxia commented 2026-08-13 00:24:16 +08:00
Owner
Copy Link
Copy Source

优化内容

将抽帧策略从 SceneChange 改为 TimeInterval

原因：

  • SceneChange 策略对高分辨率视频容易 OOM（内存溢出）
  • TimeInterval 策略更稳定，处理速度更快
  • 代码注释明确建议："TimeInterval: 按固定时间间隔（推荐，稳定性好）"

配置：

  • 策略：SceneChange → TimeInterval
  • 保持 max_frames=3（不减少抽取帧数）

预期效果：

  1. ✅ 避免 OOM 错误
  2. ✅ 加快处理速度（TimeInterval 无需复杂场景检测算法）
  3. ✅ 提高稳定性
  4. ✅ 保持封面质量（仍然抽取 3 帧）

其他优化方案（如果 TimeInterval 仍不够快）

  1. 降低视频分辨率：抽帧前降低视频分辨率
  2. 缓存机制：相同预览视频缓存已抽取的帧
  3. 预加载优化：预览视频生成完成后提前触发封面生成
  4. 异步处理：前端异步等待，避免超时
## 优化内容 ### 将抽帧策略从 SceneChange 改为 TimeInterval **原因：** - SceneChange 策略对高分辨率视频容易 OOM（内存溢出） - TimeInterval 策略更稳定，处理速度更快 - 代码注释明确建议："TimeInterval: 按固定时间间隔（推荐，稳定性好）" **配置：** - 策略：`SceneChange` → `TimeInterval` - 保持 `max_frames=3`（不减少抽取帧数） **预期效果：** 1. ✅ 避免 OOM 错误 2. ✅ 加快处理速度（TimeInterval 无需复杂场景检测算法） 3. ✅ 提高稳定性 4. ✅ 保持封面质量（仍然抽取 3 帧） ## 其他优化方案（如果 TimeInterval 仍不够快） 1. **降低视频分辨率**：抽帧前降低视频分辨率 2. **缓存机制**：相同预览视频缓存已抽取的帧 3. **预加载优化**：预览视频生成完成后提前触发封面生成 4. **异步处理**：前端异步等待，避免超时
xiaoxia added 1 commit 2026-08-13 00:24:16 +08:00
perf(#1341): 封面生成改用 TimeInterval 策略
CI/CD Pipeline / Build Staging API Image (pull_request) Has been skipped
Details
CI/CD Pipeline / Build Staging Web Image (pull_request) Has been skipped
Details
CI/CD Pipeline / Build Staging Worker Image (pull_request) Has been skipped
Details
CI/CD Pipeline / Check if frontend-only change (pull_request) Successful in 31s
Details
AI Code Review / AI Code Review (pull_request) Successful in 1m6s
Details
CI/CD Pipeline / Deploy Staging (Watchtower auto-deploy) (pull_request) Has been skipped
Details
CI/CD Pipeline / Validate - Migration (alembic) (pull_request) Successful in 1m24s
Details
CI/CD Pipeline / Validate - Type Check (mypy) (pull_request) Successful in 1m33s
Details
CI/CD Pipeline / Frontend Lint (pull_request) Has been skipped
Details
CI/CD Pipeline / Frontend Unit Tests (pull_request) Has been skipped
Details
CI/CD Pipeline / PR Build Web Image (pull_request) Has been skipped
Details
CI/CD Pipeline / Staging E2E Tests (pull_request) Has been skipped
Details
CI/CD Pipeline / Staging API Integration Tests (pull_request) Has been skipped
Details
CI/CD Pipeline / ACR Image Cleanup (pull_request) Has been skipped
Details
CI/CD Pipeline / PR Build Worker Image (pull_request) Successful in 19s
Details
PR Automation / Auto Merge on CI Green + Approved (pull_request) Successful in 1m39s
Details
Preview Deploy / Deploy Preview Environment (pull_request) Successful in 51s
Details
CI/CD Pipeline / PR Build API Image (pull_request) Successful in 1m47s
Details
PR Automation / Auto Approve on CI Green (pull_request) Successful in 3m35s
Details
CI/CD Pipeline / Validate - Code Quality (pull_request) Successful in 4m46s
Details
CI/CD Pipeline / Integration Tests (pull_request) Successful in 2m15s
Details
CI/CD Pipeline / Unit Tests (pull_request) Successful in 7m10s
Details
CI/CD Pipeline / Build Production API Image (pull_request) Has been skipped
Details
CI/CD Pipeline / Build Production Web Image (pull_request) Has been skipped
Details
CI/CD Pipeline / Build Production Worker Image (pull_request) Has been skipped
Details
CI/CD Pipeline / Canary Release to Production (pull_request) Has been skipped
Details
CI/CD Pipeline / Deploy Production (pull_request) Has been skipped
Details
CI/CD Pipeline / Production Browser E2E (pull_request) Has been skipped
Details
CI/CD Pipeline / CI Gate (pull_request) Successful in 6s
Details
Preview Cleanup / Cleanup Preview Environment (pull_request) Successful in 49s
Details
ACR Cleanup / ACR Image Cleanup (pull_request_target) Successful in 58s
Details
b523f8a6bd 
- 从 SceneChange 改为 TimeInterval 策略
- 原因：SceneChange 对高分辨率视频易 OOM，TimeInterval 更稳定
- 保持 max_frames=3，确保封面质量

预期效果：
- 避免 OOM 错误
- 加快处理速度（TimeInterval 无需复杂场景检测）
- 提高稳定性
auto-approve-bot commented 2026-08-13 00:25:44 +08:00
Collaborator
Copy Link
Copy Source

【阻塞级判定】

  • 是否存在阻塞级问题：否
  • 阻塞级问题数量：0 个

📊 审查概览

  • 整体评价：通过
  • 建议级问题数量：0 个

🔴 阻塞级问题（必须修复）

无

💡 改进建议（不阻塞合并）

无

✅ 良好实践

  1. 配置优化与注释：将抽帧策略从 SceneChange 调整为 TimeInterval，并附带清晰的注释说明了变更理由（提高稳定性、避免 OOM）。这有助于后续维护者理解该参数调整的背景。
  2. 性能权衡：将 max_frames 从默认值减少到 3，并在注释中说明了是为了“平衡速度和质量”，符合资源优化的最佳实践。

🤖 由 AI 代码审查机器人自动生成 | 2026-08-12 16:25:44 | 模型:

### 【阻塞级判定】 - 是否存在阻塞级问题：否 - 阻塞级问题数量：0 个 ### 📊 审查概览 - 整体评价：通过 - 建议级问题数量：0 个 ### 🔴 阻塞级问题（必须修复） 无 ### 💡 改进建议（不阻塞合并） 无 ### ✅ 良好实践 1. **配置优化与注释**：将抽帧策略从 `SceneChange` 调整为 `TimeInterval`，并附带清晰的注释说明了变更理由（提高稳定性、避免 OOM）。这有助于后续维护者理解该参数调整的背景。 2. **性能权衡**：将 `max_frames` 从默认值减少到 3，并在注释中说明了是为了“平衡速度和质量”，符合资源优化的最佳实践。 --- <sub>🤖 由 AI 代码审查机器人自动生成 | 2026-08-12 16:25:44 | 模型: </sub> <!-- AI_CODE_REVIEW_AUTO_COMMENT -->
gitea-actions bot commented 2026-08-13 00:26:29 +08:00
Copy Link
Copy Source

🚀 预览环境已部署

项目 详情
PR号 #1351
预览链接 https://pr-1351.preview.xiaoxiajianji.com
API环境 staging

💡 预览环境使用 staging API 数据，请勿在预览环境中操作重要数据。

🔄 每次提交新代码后预览环境会自动更新。

🗑️ PR 关闭或合并后，预览环境会自动清理。

🚀 **预览环境已部署** | 项目 | 详情 | |------|------| | PR号 | #1351 | | 预览链接 | [https://pr-1351.preview.xiaoxiajianji.com](https://pr-1351.preview.xiaoxiajianji.com) | | API环境 | staging | > 💡 预览环境使用 staging API 数据，请勿在预览环境中操作重要数据。 > > 🔄 每次提交新代码后预览环境会自动更新。 > > 🗑️ PR 关闭或合并后，预览环境会自动清理。
auto-approve-bot approved these changes 2026-08-13 00:30:30 +08:00
auto-approve-bot left a comment
Collaborator
Copy Link
Copy Source

CI全绿，自动审批通过。

CI全绿，自动审批通过。
auto-approve-bot approved these changes 2026-08-13 00:30:30 +08:00
auto-approve-bot left a comment
Collaborator
Copy Link
Copy Source

CI全绿，自动审批通过。

CI全绿，自动审批通过。
auto-approve-bot merged commit cd6fde790a into develop 2026-08-13 00:36:00 +08:00
auto-approve-bot deleted branch optimize/cover-use-timeinterval 2026-08-13 00:36:00 +08:00
gitea-actions bot commented 2026-08-13 00:36:48 +08:00
Copy Link
Copy Source

🗑️ 预览环境已清理

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

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

🗑️ **预览环境已清理** PR #1351 已关闭或合并，对应的预览环境已被清理。 > 如有需要，可以重新打开 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
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: xiaoxia/xiaoxia-saas#1351
Delete Branch "optimize/cover-use-timeinterval"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?