执行摘要
删除 tests/models/multimodal/processing/test_audioflamingo3.py 中的 test_audio_feature_pipeline_matches_hf_small_config 函数,修复因缺少 vLLM 配置导致 CI 失败的问题。该测试已被 generation 测试覆盖,删除避免了重复维护。
功能与动机
关联 Issue #40113 报告 CI 测试 models/multimodal/processing/test_audioflamingo3.py::test_audio_feature_pipeline_matches_hf_small_config 因 AssertionError: Current vLLM config is not set. 失败。PR author ywang96 指出该测试的功能已被 generation 测试覆盖,不应保留在 processor 测试中。
实现拆解
- 删除无效测试:从
tests/models/multimodal/processing/test_audioflamingo3.py 中移除 test_audio_feature_pipeline_matches_hf_small_config 函数(81 行),该函数通过实例化 HuggingFace 模型与 vLLM 组件的输出对比来验证音频特征管线。
- 保留其余测试:文件中的
test_audio_dummy_data 和 test_audio_token_count_matches_hf_processor_math 被保留,它们专注于 processing 逻辑而非模型执行。
无。变更仅为删除已有代码。
评论区精华
ywang96: 为什么这个测试被包含在 processor 测试中?
DarkLight1337: 这些测试是关于生成 dummy data 和配置加载,属于 processing 范畴,而不是模型执行。
虽然测试最初被认为合理,但作者最终决定删除以避免 CI 失败和重复维护。
风险与影响
- 风险:极低,仅删除一个测试函数,不修改任何生产代码。需确认 generation 测试覆盖了相同场景。
- 影响:直接修复 CI 失败,减少维护负担。对用户无影响。
关联脉络
无直接关联的其他 PR。本 PR 是针对 Issue #40113 的 CI 失败修复,属于单一目标清理。
参与讨论