# PR #7254 完整报告

- 仓库：`PaddlePaddle/FastDeploy`
- 标题：Revert "[Other] support video_fps args for video bench (#7077)"
- 合并时间：2026-04-08 20:13
- 原文链接：http://prhub.com.cn/PaddlePaddle/FastDeploy/pull/7254

---

# 执行摘要

- 一句话：回滚 OpenAI 协议中为视频基准测试临时添加的 video_fps 参数。
- 推荐动作：该 PR 值得快速浏览以了解 API 清理决策，但技术内容简单。关注点：1) 回滚决策反映临时参数应避免加入核心协议；2) 视频处理参数传递的最佳实践（通过 mm_processor_kwargs）。无需深入代码分析。

# 功能与动机

根据 review 评论，回滚原因未在 PR body 中明确说明，但推测是 video_fps 参数设计为临时性（原 commit 注释标注为 "temporary parameter"），且视频处理应通过 mm_processor_kwargs 传递。fastdeploy-bot 的评论建议补充回滚原因，例如 " 该参数设计不合理、与现有机制重复、或后续将通过 mm_processor_kwargs 传递等 "。

# 实现拆解

该 PR 仅修改一个文件：fastdeploy/entrypoints/openai/protocol.py。具体改动是从 CompletionRequest 和 ChatCompletionRequest 两个 Pydantic 模型类中删除 video_fps 字段（Optional[float] 类型）及其相关注释。删除操作通过 revert commit 实现，移除了 6 行代码。

关键文件：
- `fastdeploy/entrypoints/openai/protocol.py`（模块 APIServer）: 唯一修改文件，包含 OpenAI 协议请求类定义，删除 video_fps 参数直接影响 API 接口。

关键符号：CompletionRequest.to_dict_for_infer, ChatCompletionRequest.to_dict_for_infer


# 评论区精华

review 讨论主要集中在 PR 规范而非技术内容：1) Copilot 指出 PR 标题缺少标签，建议使用 [Others] 或 [APIServer]；2) fastdeploy-bot 详细列出规范问题：标题标签应为 [APIServer] 而非 [Other]，描述缺少回滚原因和文档更新说明。讨论结论是回滚代码正确，但需完善标题和描述以符合规范。未解决的技术疑虑：回滚后视频基准测试如何传递 fps 参数未明确说明。

- PR 标题标签规范 (style): 应调整标题以符合规范，但 PR 已合并未修改。
- PR 描述完整性 (documentation): 描述未补充，但回滚操作被接受。

# 风险与影响

- 风险：技术风险较低：1) 回归风险：如果视频基准测试仍依赖 video_fps 参数，可能导致功能失效或错误，但原参数标记为临时性，且 fastdeploy-bot 评论指出视频处理可通过 mm_processor_kwargs 传递，因此影响可能有限；2) 兼容性风险：删除 API 字段可能影响客户端调用，但该参数为可选且临时添加，风险可控；3) 安全风险：无。
- 影响：影响范围：1) 用户影响：使用 OpenAI 协议且依赖 video_fps 参数的视频基准测试用户需调整参数传递方式，可能通过 mm_processor_kwargs；2) 系统影响：简化协议定义，移除临时字段，提升 API 一致性；3) 团队影响：需更新相关文档或测试用例以反映变更。影响程度：小范围，仅涉及特定基准测试场景。
- 风险标记：API 变更 , 临时参数移除

# 关联脉络

- PR #7077 [Other] support video_fps args for video bench: 该 PR 回滚的正是 #7077 引入的 video_fps 参数，两者直接关联，反映了参数从添加到移除的完整生命周期。
- PR #7183 [Optimization] Enable text-only deployment for multimodal models: 涉及多模态模型处理，可能关联视频参数传递机制（如 mm_processor_kwargs），提供上下文。
- PR #7109 [DataProcessor] Move image_processor to unified directory and add MultiModalProcessor: 涉及多模态处理器统一，可能与视频参数处理方式相关。