Prhub

#43583 [Misc] Print accuracy value for PD tests even on success

原始 PR 作者 NickLucche 合并时间 2026-05-25 17:10 文件变更 1 提交数 1 评论 0 代码增减 +1 / -1

执行摘要

精度测试始终打印测量值

该 PR 响应 issue 评论,旨在解决分布式模式下多种模型组合运行测试时精度方差高于预期的问题。通过始终打印测量值,可在断言触发前追踪其变化。

值得合入,对调试高方差测试场景有实际帮助,无副作用。

讨论亮点

无 reviewer 讨论。

实现拆解

tests/v1/kv_connector/nixl_integration/test_accuracy.pytest_accuracy 函数中,将原先仅在 expected_value is None 分支内打印 measured_valueprint(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

未识别关联 Issue

当前没有检测到明确关联的 Issue 链接,后续同步到相关引用后会出现在这里。

完整报告

参与讨论