chore(worker): BGM preset diagnostics + upload helper #2094

Merged

xiaoxia merged 4 commits from chore/preset-bgm-audio-upload-helper into develop 2026-09-29 16:03:30 +08:00

Conversation 2 Commits 4 Files Changed 6

+1220 -42

xiaoxia commented 2026-09-29 15:29:10 +08:00

Owner

Copy Link

Copy Source

背景

预设BGM 的 audio_url 全部为空，渲染时日志只能看到「所有来源都无法获取BGM，跳过」，无法快速定位是哪种来源（audio_url/asset_id/preset_id）出问题。同时缺少一个方便运维把音频文件上传到 OSS 并回填 URL 的工具。

改动

1. apps/worker/video_processing/render_adapter.py _prepare_bgm 优化日志：
   • preset 分支区分「preset_id 不存在」/「preset 存在但 audio_url 未部署」/「下载失败」三种情况，未部署时提示运维上传
   • 最终 warning 输出 enabled/audio_url/asset_id/preset_id 四个字段的状态，一眼定位
2. scripts/upload_preset_bgm.py（新增）：一键上传预设 BGM 到 OSS + 回填 preset_bgm.py
   • 读取 bgm_assets/{preset_id}.mp3，上传到 OSS preset/bgm/，设置 public-read ACL
   • 自动改写 packages/domain/preset_bgm.py 中对应 PresetBGM 的 audio_url 字段为公网 URL
   • 支持 --dry-run、--bgm-dir、--oss-prefix 参数
3. bgm_assets/README.md（新增）：10 首预设 BGM 的清单表 + 命名规范 + 操作步骤

不在本 PR

音频文件本身由运维/设计同学准备，本 PR 不包含任何 mp3 文件。上传操作在有 OSS 凭证的环境执行。

## 背景 预设BGM 的 audio_url 全部为空，渲染时日志只能看到「所有来源都无法获取BGM，跳过」，无法快速定位是哪种来源（audio_url/asset_id/preset_id）出问题。同时缺少一个方便运维把音频文件上传到 OSS 并回填 URL 的工具。 ## 改动 1. **apps/worker/video_processing/render_adapter.py `_prepare_bgm` 优化日志**： - preset 分支区分「preset_id 不存在」/「preset 存在但 audio_url 未部署」/「下载失败」三种情况，未部署时提示运维上传 - 最终 warning 输出 enabled/audio_url/asset_id/preset_id 四个字段的状态，一眼定位 2. **scripts/upload_preset_bgm.py（新增）**：一键上传预设 BGM 到 OSS + 回填 preset_bgm.py - 读取 bgm_assets/{preset_id}.mp3，上传到 OSS `preset/bgm/`，设置 public-read ACL - 自动改写 packages/domain/preset_bgm.py 中对应 PresetBGM 的 audio_url 字段为公网 URL - 支持 --dry-run、--bgm-dir、--oss-prefix 参数 3. **bgm_assets/README.md（新增）**：10 首预设 BGM 的清单表 + 命名规范 + 操作步骤 ## 不在本 PR 音频文件本身由运维/设计同学准备，本 PR 不包含任何 mp3 文件。上传操作在有 OSS 凭证的环境执行。

xiaoxia added 2 commits 2026-09-29 15:29:11 +08:00

fix(gpu-direct): passthrough template title/subtitle/BGM config ... 11c554e43a

Problem: GPU direct render MVP hardcoded title/subtitle/BGM styles and
ignored template config fields:
- P0 static subtitle text (subtitle.text) was never rendered
- P0 title styles (font/size/color/position/stroke/shadow/bold) all hardcoded
- P1 BGM volume hardcoded at 0.35, no fade/offset/adjust-db
- P1 ASR subtitle styles (color/size/position/font) ignored
- P2 extra_audio_tracks (TTS/voiceover) volume hardcoded at 1.0

Changes:
1. build_direct_render() now accepts title_config/subtitle_config/
   bgm_config dicts + static_subtitle_text.
2. Added helpers _hex_to_drawtext_color (#RGB/#RRGGBB/#RRGGBBAA/named),
   _position_to_drawtext_xy (top/center/bottom), _build_drawtext_filters
   (shadow layer + main layer, stroke via borderw/bordercolor, bold
   simulated via same-color borderw).
3. Title parses font/size/color/position/stroke/shadow/bold with safe
   defaults; legacy title_text still used as fallback.
4. Static subtitle_text renders as full-span (0→total_duration) segment;
   when ASR segments are present they take priority.
5. BGM reads volume (default 0.3), volume_adjust_db (linear gain),
   fade_in/fade_out (afade), audio_offset (adelay ms|ms), and honors
   enabled=false even if path supplied. Volume clamped to [0,1.5].
6. _try_gpu_direct() now passes cfg['title']/['subtitle']/['bgm'] whole
   dicts through, computes static_subtitle_text for non-auto subs,
   and sources extra_audio_tracks volume from audio_tracks_config
   (voiceover tracks) instead of hardcoding 1.0.
7. DirectRenderPlan exposes filter_complex for testing.
8. Added 24 unit tests covering: default backward-compat, color hex/BGR
   conversion (short/long/alpha/invalid), position (top/center/bottom),
   stroke, shadow (two drawtext layers), font override, enabled flag,
   static subtitle full-span, ASR style passthrough, BGM
   volume/fade/offset/adjust-db/enabled=false, extra audio volume,
   bold simulation, ASR vs static priority, volume clamping.

Fixes template-style rendering not taking effect in GPU pipeline.

chore(worker): improve BGM preset download diagnostics + add upload helper script ... a04e363d1b

- _prepare_bgm preset branch now differentiates: preset not found /
  preset exists but audio_url empty (未部署) / download failure, with
  a clear hint to ops to upload audio and fill preset_bgm.py
- final warning now surfaces enabled/audio_url/asset_id/preset_id state
  so triage is immediate
- add scripts/upload_preset_bgm.py: one-shot uploader that uploads
  mp3s from ./bgm_assets/{preset_id}.mp3 to OSS preset/bgm/, sets
  public-read ACL, and rewrites packages/domain/preset_bgm.py with
  the public URL. Dry-run supported.
- add bgm_assets/README.md with the 10-preset file naming table

gitea-actions bot commented 2026-09-29 15:30:33 +08:00

Copy Link

Copy Source

🚀 预览环境已部署

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

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

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

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

🚀 **预览环境已部署** | 项目 | 详情 | |------|------| | PR号 | #2094 | | 预览链接 | [https://pr-2094.preview.xiaoxiajianji.com](https://pr-2094.preview.xiaoxiajianji.com) | | API环境 | staging | > 💡 预览环境使用 staging API 数据，请勿在预览环境中操作重要数据。 > > 🔄 每次提交新代码后预览环境会自动更新。 > > 🗑️ PR 关闭或合并后，预览环境会自动清理。

auto-approve-bot added 1 commit 2026-09-29 15:33:24 +08:00

style: auto-format with black + isort + ruff + prettier [skip ci-format-check] d449496f90

xiaoxia added 1 commit 2026-09-29 15:35:39 +08:00

chore(worker): move bgm upload readme under docs/ to avoid ruff parsing md ca6803e1a5

xiaoxia merged commit 02199d80ee into develop 2026-09-29 16:03:30 +08:00

gitea-actions bot commented 2026-09-29 16:04:28 +08:00

Copy Link

Copy Source

🗑️ 预览环境已清理

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

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

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

Sign in to join this conversation.

Reviewers

No Reviewers

Labels

Clear labels

architecture

架构设计

backend

后端相关

blocked

阻塞中

bug

Bug修复

ci-bug

CI本身的bug

ci-optimization

CI优化类任务

ci-pipeline

CI流水线/workflow相关

cleanup

代码清理

devops

运维/部署/CI

documentation

文档相关

feature

新功能

frontend

前端相关

P0

阻塞性，必须立即处理

P1

高优先级

P2

中优先级

P3

低优先级/后考虑

question

需要讨论确认

ready

PR CI通过，可合并

refactor

代码重构

No Label

Milestone

No items

No Milestone

Projects

Clear projects

No project

Assignees

Clear assignees

auto-approve-bot frontend-dev xiaoxia

No Assignees

3 Participants

Notifications

Subscribe

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: xiaoxia/xiaoxia-saas#2094