Skip to content

[TRTLLM-13694][infra] Add benchmark refresh mappings - #17317

Open
Mgluhovskoi wants to merge 3 commits into
NVIDIA:mainfrom
Mgluhovskoi:agent/benchmark-recipe-scope
Open

[TRTLLM-13694][infra] Add benchmark refresh mappings#17317
Mgluhovskoi wants to merge 3 commits into
NVIDIA:mainfrom
Mgluhovskoi:agent/benchmark-recipe-scope

Conversation

@Mgluhovskoi

@Mgluhovskoi Mgluhovskoi commented Aug 5, 2026

Copy link
Copy Markdown
Collaborator

Description

Add a repository-owned mapping file that defines which model, GPU, and precision groups may be refreshed from the latest validated benchmark data.

The file groups repeated accelerator translations and target path rules into shared defaults, so adding a model normally requires only its source model and precision, public target model and architecture, and GPU list. It also documents how generated recipes are stored, how to add or remove models and GPUs, and how to override the defaults when necessary.

Artifact logs are discovered automatically from each record's runtime dimensions. An optional artifact_log.match_tokens override is available only when multiple complete logs remain ambiguous.

Test Coverage

  • Parsed and expanded three model definitions into five source-to-target mappings with the automation schema loader.
  • Verified all five mapped model/GPU targets exist in examples/configs/database/lookup.yaml.
  • Ran the companion automation unit tests: 22 passed.

PR Checklist

  • Please check this after reviewing the above items as appropriate for this PR.

Dev Engineer Review

  • Added version 2 benchmark refresh mappings for DeepSeek-R1 and GPT-OSS-120B.
  • Added B200 and H200 accelerator translations.
  • Added default recipe paths and filenames.
  • Documented model and GPU registration, default overrides, artifact-log matching, and removal semantics.
  • Validation expanded the configuration into five mappings and confirmed target entries in lookup.yaml.
  • No public API or exported entity changes.
  • 22 automation unit tests passed.

QA Engineer Review

No test changes.

Signed-off-by: Maxim Gluhovskoi <mgluhovskoi@nvidia.com>
Signed-off-by: Maxim Gluhovskoi <mgluhovskoi@nvidia.com>
Signed-off-by: Maxim Gluhovskoi <mgluhovskoi@nvidia.com>
@Mgluhovskoi
Mgluhovskoi marked this pull request as ready for review August 7, 2026 21:30
@Mgluhovskoi
Mgluhovskoi requested a review from a team as a code owner August 7, 2026 21:30
@Mgluhovskoi

Copy link
Copy Markdown
Collaborator Author

/bot run

@coderabbitai

coderabbitai Bot commented Aug 7, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Enterprise

Run ID: 8f334e3d-f3b3-4654-b432-2e8bea0d59dd

📥 Commits

Reviewing files that changed from the base of the PR and between 1dfb7b1 and d61f6a6.

📒 Files selected for processing (1)
  • examples/configs/benchmark_refresh_mappings.yaml

Walkthrough

Adds a version 2 benchmark refresh mapping configuration. The configuration defines output defaults, B200/H200 mappings, and refresh targets for DeepSeek-R1 FP4/FP8 and GPT-OSS-120B FP4.

Changes

Benchmark refresh configuration

Layer / File(s) Summary
Refresh mapping schema and targets
examples/configs/benchmark_refresh_mappings.yaml
Defines default recipe output paths and filenames, accelerator mappings, model metadata, GPU assignments, artifact-log disambiguation, and removal semantics.

Estimated code review effort: 2 (Simple) | ~10 minutes

Suggested reviewers: bowenfu

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title follows the required ticket/type format and clearly identifies the addition of benchmark refresh mappings.
Description check ✅ Passed The description explains the change, documents test coverage, and includes the required checklist section.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Comment @coderabbitai help to get the list of available commands.

@tensorrt-cicd

Copy link
Copy Markdown
Collaborator

PR_Github #64709 [ run ] triggered by Bot. Commit: d61f6a6 Link to invocation

@tensorrt-cicd

Copy link
Copy Markdown
Collaborator

PR_Github #64709 [ run ] completed with state SUCCESS. Commit: d61f6a6
/LLM/main/L0_MergeRequest_PR pipeline #52564 completed with status: 'FAILURE'

CI Report

⚠️ Action Required:

  • Please check the failed tests and fix your PR
  • If you cannot view the failures, ask the CI triggerer to share details
  • Once fixed, request an NVIDIA team member to trigger CI again

CI Agent Failure Analysis

Link to invocation

@Mgluhovskoi

Copy link
Copy Markdown
Collaborator Author

/bot run

@tensorrt-cicd

Copy link
Copy Markdown
Collaborator

PR_Github #64726 [ run ] triggered by Bot. Commit: d61f6a6 Link to invocation

@tensorrt-cicd

Copy link
Copy Markdown
Collaborator

PR_Github #64726 [ run ] completed with state FAILURE. Commit: d61f6a6
/LLM/main/L0_MergeRequest_PR pipeline #52578 completed with status: 'FAILURE'

CI Report

⚠️ Action Required:

  • Please check the failed tests and fix your PR
  • If you cannot view the failures, ask the CI triggerer to share details
  • Once fixed, request an NVIDIA team member to trigger CI again

CI Agent Failure Analysis

Link to invocation

@Mgluhovskoi

Copy link
Copy Markdown
Collaborator Author

/bot run

@tensorrt-cicd

Copy link
Copy Markdown
Collaborator

PR_Github #65113 [ run ] triggered by Bot. Commit: d61f6a6 Link to invocation

@tensorrt-cicd

Copy link
Copy Markdown
Collaborator

PR_Github #65113 [ run ] completed with state SUCCESS. Commit: d61f6a6
/LLM/main/L0_MergeRequest_PR pipeline #52911 completed with status: 'FAILURE'

CI Report

⚠️ Action Required:

  • Please check the failed tests and fix your PR
  • If you cannot view the failures, ask the CI triggerer to share details
  • Once fixed, request an NVIDIA team member to trigger CI again

CI Agent Failure Analysis

Link to invocation

@Mgluhovskoi

Copy link
Copy Markdown
Collaborator Author

/bot run

@tensorrt-cicd

Copy link
Copy Markdown
Collaborator

PR_Github #65155 [ run ] triggered by Bot. Commit: d61f6a6 Link to invocation

@tensorrt-cicd

Copy link
Copy Markdown
Collaborator

PR_Github #65155 [ run ] completed with state FAILURE. Commit: d61f6a6
/LLM/main/L0_MergeRequest_PR pipeline #52948 completed with status: 'FAILURE'

CI Report

⚠️ Action Required:

  • Please check the failed tests and fix your PR
  • If you cannot view the failures, ask the CI triggerer to share details
  • Once fixed, request an NVIDIA team member to trigger CI again

CI Agent Failure Analysis

Link to invocation

@Mgluhovskoi

Copy link
Copy Markdown
Collaborator Author

/bot run

1 similar comment
@Mgluhovskoi

Copy link
Copy Markdown
Collaborator Author

/bot run

@tensorrt-cicd

Copy link
Copy Markdown
Collaborator

PR_Github #65264 [ run ] triggered by Bot. Commit: d61f6a6 Link to invocation

@tensorrt-cicd

Copy link
Copy Markdown
Collaborator

PR_Github #65266 [ run ] triggered by Bot. Commit: d61f6a6 Link to invocation

@tensorrt-cicd

Copy link
Copy Markdown
Collaborator

PR_Github #65264 [ run ] completed with state ABORTED. Commit: d61f6a6

Link to invocation

@tensorrt-cicd

Copy link
Copy Markdown
Collaborator

PR_Github #65266 [ run ] completed with state FAILURE. Commit: d61f6a6
/LLM/main/L0_MergeRequest_PR pipeline #53043 completed with status: 'FAILURE'

CI Report

⚠️ Action Required:

  • Please check the failed tests and fix your PR
  • If you cannot view the failures, ask the CI triggerer to share details
  • Once fixed, request an NVIDIA team member to trigger CI again

CI Agent Failure Analysis

Link to invocation

@Mgluhovskoi

Copy link
Copy Markdown
Collaborator Author

/bot run

@tensorrt-cicd

Copy link
Copy Markdown
Collaborator

PR_Github #65376 [ run ] triggered by Bot. Commit: d61f6a6 Link to invocation

Copy link
Copy Markdown
Collaborator Author

/bot run

@tensorrt-cicd

Copy link
Copy Markdown
Collaborator

PR_Github #65540 [ run ] triggered by Bot. Commit: d61f6a6 Link to invocation

@tensorrt-cicd

Copy link
Copy Markdown
Collaborator

PR_Github #65540 [ run ] completed with state FAILURE. Commit: d61f6a6
/LLM/main/L0_MergeRequest_PR pipeline #53278 completed with status: 'FAILURE'

CI Report

⚠️ Action Required:

  • Please check the failed tests and fix your PR
  • If you cannot view the failures, ask the CI triggerer to share details
  • Once fixed, request an NVIDIA team member to trigger CI again

CI Agent Failure Analysis

Link to invocation

Copy link
Copy Markdown
Collaborator Author

/bot run

@tensorrt-cicd

Copy link
Copy Markdown
Collaborator

PR_Github #65558 [ run ] triggered by Bot. Commit: d61f6a6 Link to invocation

@tensorrt-cicd

Copy link
Copy Markdown
Collaborator

PR_Github #65558 [ run ] completed with state FAILURE. Commit: d61f6a6
/LLM/main/L0_MergeRequest_PR pipeline #53294 completed with status: 'FAILURE'

CI Report

⚠️ Action Required:

  • Please check the failed tests and fix your PR
  • If you cannot view the failures, ask the CI triggerer to share details
  • Once fixed, request an NVIDIA team member to trigger CI again

CI Agent Failure Analysis

Link to invocation

Copy link
Copy Markdown
Collaborator Author

/bot run

@tensorrt-cicd

Copy link
Copy Markdown
Collaborator

PR_Github #65568 [ run ] triggered by Bot. Commit: d61f6a6 Link to invocation

@tensorrt-cicd

Copy link
Copy Markdown
Collaborator

PR_Github #65568 [ run ] completed with state ABORTED. Commit: d61f6a6

Link to invocation

@Mgluhovskoi

Copy link
Copy Markdown
Collaborator Author

/bot run

@tensorrt-cicd

Copy link
Copy Markdown
Collaborator

PR_Github #65718 [ run ] triggered by Bot. Commit: d61f6a6 Link to invocation

@tensorrt-cicd

Copy link
Copy Markdown
Collaborator

PR_Github #65718 [ run ] completed with state FAILURE. Commit: d61f6a6
/LLM/main/L0_MergeRequest_PR pipeline #53434 completed with status: 'FAILURE'

CI Report

⚠️ Action Required:

  • Please check the failed tests and fix your PR
  • If you cannot view the failures, ask the CI triggerer to share details
  • Once fixed, request an NVIDIA team member to trigger CI again

CI Agent Failure Analysis

Link to invocation

@Mgluhovskoi

Copy link
Copy Markdown
Collaborator Author

/bot run

@tensorrt-cicd

Copy link
Copy Markdown
Collaborator

PR_Github #65729 [ run ] triggered by Bot. Commit: d61f6a6 Link to invocation

@tensorrt-cicd

Copy link
Copy Markdown
Collaborator

PR_Github #65729 [ run ] completed with state FAILURE. Commit: d61f6a6
/LLM/main/L0_MergeRequest_PR pipeline #53445 completed with status: 'FAILURE'

CI Report

⚠️ Action Required:

  • Please check the failed tests and fix your PR
  • If you cannot view the failures, ask the CI triggerer to share details
  • Once fixed, request an NVIDIA team member to trigger CI again

CI Agent Failure Analysis

Link to invocation

@Mgluhovskoi

Copy link
Copy Markdown
Collaborator Author

/bot run

@tensorrt-cicd

Copy link
Copy Markdown
Collaborator

PR_Github #65739 [ run ] triggered by Bot. Commit: d61f6a6 Link to invocation

@tensorrt-cicd

Copy link
Copy Markdown
Collaborator

PR_Github #65739 [ run ] completed with state FAILURE. Commit: d61f6a6
/LLM/main/L0_MergeRequest_PR pipeline #53455 completed with status: 'FAILURE'

CI Report

⚠️ Action Required:

  • Please check the failed tests and fix your PR
  • If you cannot view the failures, ask the CI triggerer to share details
  • Once fixed, request an NVIDIA team member to trigger CI again

CI Agent Failure Analysis

Link to invocation

@Mgluhovskoi

Copy link
Copy Markdown
Collaborator Author

/bot run

@tensorrt-cicd

Copy link
Copy Markdown
Collaborator

PR_Github #65874 [ run ] triggered by Bot. Commit: d61f6a6 Link to invocation

@tensorrt-cicd

Copy link
Copy Markdown
Collaborator

PR_Github #65874 [ run ] completed with state FAILURE. Commit: d61f6a6
/LLM/main/L0_MergeRequest_PR pipeline #53568 completed with status: 'FAILURE'

CI Report

⚠️ Action Required:

  • Please check the failed tests and fix your PR
  • If you cannot view the failures, ask the CI triggerer to share details
  • Once fixed, request an NVIDIA team member to trigger CI again

CI Agent Failure Analysis

Link to invocation

@Mgluhovskoi

Copy link
Copy Markdown
Collaborator Author

/bot run

@tensorrt-cicd

Copy link
Copy Markdown
Collaborator

PR_Github #65913 [ run ] triggered by Bot. Commit: d61f6a6 Link to invocation

@tensorrt-cicd

Copy link
Copy Markdown
Collaborator

PR_Github #65913 [ run ] completed with state SUCCESS. Commit: d61f6a6
/LLM/main/L0_MergeRequest_PR pipeline #53604 completed with status: 'FAILURE'

CI Report

⚠️ Action Required:

  • Please check the failed tests and fix your PR
  • If you cannot view the failures, ask the CI triggerer to share details
  • Once fixed, request an NVIDIA team member to trigger CI again

CI Agent Failure Analysis

Link to invocation

@Mgluhovskoi

Copy link
Copy Markdown
Collaborator Author

/bot run

@tensorrt-cicd

Copy link
Copy Markdown
Collaborator

PR_Github #66050 [ run ] triggered by Bot. Commit: d61f6a6 Link to invocation

@tensorrt-cicd

Copy link
Copy Markdown
Collaborator

PR_Github #66050 [ run ] completed with state SUCCESS. Commit: d61f6a6
/LLM/main/L0_MergeRequest_PR pipeline #53733 completed with status: 'FAILURE'

CI Report

⚠️ Action Required:

  • Please check the failed tests and fix your PR
  • If you cannot view the failures, ask the CI triggerer to share details
  • Once fixed, request an NVIDIA team member to trigger CI again

CI Agent Failure Analysis

Link to invocation

@Mgluhovskoi

Copy link
Copy Markdown
Collaborator Author

/bot run

@tensorrt-cicd

Copy link
Copy Markdown
Collaborator

PR_Github #66061 [ run ] triggered by Bot. Commit: d61f6a6 Link to invocation

@tensorrt-cicd

Copy link
Copy Markdown
Collaborator

PR_Github #66061 [ run ] completed with state SUCCESS. Commit: d61f6a6
/LLM/main/L0_MergeRequest_PR pipeline #53744 completed with status: 'FAILURE'

CI Report

⚠️ Action Required:

  • Please check the failed tests and fix your PR
  • If you cannot view the failures, ask the CI triggerer to share details
  • Once fixed, request an NVIDIA team member to trigger CI again

CI Agent Failure Analysis

Link to invocation

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants