feat(agentx): fail incomplete profiling metric coverage / 检测 AgentX 分析指标覆盖不完整 - #35
Merged
Merged
Conversation
Require TTFT and inter-token-latency observations through 98% of valid AgentX profiling phases. Preserve diagnostic artifacts, mark invalid submissions, and exit non-zero with an actionable server error.\n\n中文:要求有效 AgentX 分析阶段的 TTFT 和 token 间延迟观测覆盖至少 98% 的时长。保留诊断产物,将提交标记为无效,并通过明确的服务端错误以非零状态退出。 Signed-off-by: Cam Quilici <cjquilici@gmail.com>
Try out this PRQuick install: pip install --upgrade --force-reinstall git+https://github.com/ai-dynamo/aiperf.git@f77c31a715850ce8f0eb9dd37ddc90ce0bffcdbeRecommended with virtual environment (using uv): uv venv --python 3.12 && source .venv/bin/activate
uv pip install --upgrade --force-reinstall git+https://github.com/ai-dynamo/aiperf.git@f77c31a715850ce8f0eb9dd37ddc90ce0bffcdbeLast updated for commit: |
|
Fern Docs Preview: generation failed — see the Actions log for details. This does not block merge; ask a maintainer to retry if needed. |
Export usable partial profiling results before reporting fatal post-run validation errors and exiting non-zero.\n\n中文:在报告致命的运行后验证错误并以非零状态退出前,先导出可用的部分分析结果。 Signed-off-by: Cam Quilici <cjquilici@gmail.com>
cquil11
marked this pull request as ready for review
August 3, 2026 21:10
cquil11
added a commit
that referenced
this pull request
Aug 6, 2026
Require TTFT and ITL observations through at least 98% of valid AgentX profiling phases. Retain diagnostic artifacts and mark incomplete submissions invalid.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
insufficient_profile_metric_coverage--unsafe-overridesmoke runsValidation
f77c31a715850ce8f0eb9dd37ddc90ce0bffcdbe: 4 PASS, 0 FAILProfileMetricCoverageError, no hang, retained aggregate artifacts,submission_valid: false, andinsufficient_profile_metric_coverage中文说明
insufficient_profile_metric_coverage将场景结果标记为无效--unsafe-override运行的短时冒烟测试不计入该检查验证
f77c31a715850ce8f0eb9dd37ddc90ce0bffcdbe上完成 mock server 审计:4 个通过,0 个失败ProfileMetricCoverageError,保留聚合产物,同时写入submission_valid: false和insufficient_profile_metric_coverage