执行摘要
精度测试始终打印测量值
该 PR 响应 issue 评论,旨在解决分布式模式下多种模型组合运行测试时精度方差高于预期的问题。通过始终打印测量值,可在断言触发前追踪其变化。
值得合入,对调试高方差测试场景有实际帮助,无副作用。
无 reviewer 讨论。
该 PR 响应 issue 评论,旨在解决分布式模式下多种模型组合运行测试时精度方差高于预期的问题。通过始终打印测量值,可在断言触发前追踪其变化。
值得合入,对调试高方差测试场景有实际帮助,无副作用。
无 reviewer 讨论。
在 tests/v1/kv_connector/nixl_integration/test_accuracy.py 的 test_accuracy 函数中,将原先仅在 expected_value is None 分支内打印 measured_value 的 print(f"Measured value: {measured_value}") 移到分支外部,改为无条件执行 print(f"Measured accuracy value: {measured_value}\n"),确保每次运行都记录精度值。
| 文件 | 模块 | 状态 | 重要度 |
|---|---|---|---|
tests/v1/kv_connector/nixl_integration/test_accuracy.py |
测试 | modified | 2.85 |
tests/v1/kv_connector/nixl_integration/test_accuracy.py
test-coverage
唯一变更文件,将测量精度打印移到断言前,确保始终输出。
# tests/v1/kv_connector/nixl_integration/test_accuracy.py
measured_value = results["results"][TASK][FILTER]
expected_value = EXPECTED_VALUES.get(MODEL_NAME)
# 始终打印测量精度值,便于追踪跨运行变化
print(f"Measured accuracy value: {measured_value}\n")
if expected_value is None:
print(
f"Warning: No expected value found for {MODEL_NAME}. "
"Skipping accuracy check."
)
return
assert (
measured_value - RTOL < expected_value
and measured_value + RTOL > expected_value
), f"Expected: {expected_value} | Measured: {measured_value}"
当前评论区没有形成足够清晰的争议点或结论,后续有更多讨论时会体现在这里。
极低风险,仅修改了测试中的日志打印逻辑,不影响测试逻辑或系统行为。
仅影响精度集成测试的输出,使其始终打印测量值,便于开发者在 CI 或本地调试时诊断精度波动。
当前没有检测到明确关联的 Issue 链接,后续同步到相关引用后会出现在这里。
参与讨论