Skip to content

Fix: Update general validation benchmarks#41

Merged
mbenavidez-amd merged 1 commit into
ROCm:mainfrom
mbenavidez-amd:fix/update-test-criteria
Jun 3, 2026
Merged

Fix: Update general validation benchmarks#41
mbenavidez-amd merged 1 commit into
ROCm:mainfrom
mbenavidez-amd:fix/update-test-criteria

Conversation

@mbenavidez-amd

@mbenavidez-amd mbenavidez-amd commented Jun 2, 2026

Copy link
Copy Markdown
Collaborator

Motivation

This pull request updates the test duration tables in the docs/index.md documentation to improve clarity, provide more detailed instructions, and add missing tests. The tables are now formatted using reStructuredText list-tables, and several test durations and descriptions have been updated or expanded.

Technical Details

Documentation formatting and clarity:

  • Replaced markdown tables with reStructuredText list-table format for both single-node and multi-node test duration tables, improving readability and consistency in the documentation.

Test coverage and duration updates:

  • Updated estimated durations for several tests (e.g., increased AGFHC all_lvl5 from 2 hours to 6 hours, AGFHC minihpl from 3 to 4 hours, Single Node RCCL to 11 minutes, etc.), and added missing tests such as rocHPL and additional optional workloads.
  • Expanded multi-node test descriptions to include more detailed instructions (e.g., specifying which bandwidth/latency tests to run for OFED, and which performance tests to run for RCCL), and added several new benchmarks (e.g., rocBLAS FP32 GEMM, rocBLAS BF16, rocBLAS INT8, Babelstream) with references to the System Validation documentation.
  • Clarified optional and specialized tests.

Test Plan

Run and confirm a successful local build of the documentation.

Test Result

Local build was successful.

Submission Checklist

@mbenavidez-amd mbenavidez-amd self-assigned this Jun 2, 2026
@mbenavidez-amd mbenavidez-amd added the documentation Improvements or additions to documentation label Jun 2, 2026
@mbenavidez-amd mbenavidez-amd merged commit cbca3df into ROCm:main Jun 3, 2026
3 checks passed

@rzimmer512 rzimmer512 left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

PR reviewed after it was merged. No concerns.

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

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants