Skip to content

changing the command to run TB schmoo to get better numbers#232

Merged
urtiwari merged 2 commits into
mainfrom
urtiwari/TBissue
Jun 19, 2026
Merged

changing the command to run TB schmoo to get better numbers#232
urtiwari merged 2 commits into
mainfrom
urtiwari/TBissue

Conversation

@urtiwari

Copy link
Copy Markdown
Collaborator

changing the command to run TB schmoo to get better numbers, added flag to the existing command

Signed-off-by: Urvashi Tiwari <urtiwari.com>

Copilot AI 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.

Pull request overview

Updates the TransferBench “schmoo” health test to run with additional tuning environment variables and adjusts the result parsing to accommodate a slightly different output/table format.

Changes:

  • Update schmoo bandwidth parsing regex for the 32-CU row.
  • Run TransferBench schmoo with GFX_UNROLL=32 and SWEEP_MIN=32 set in the command line.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment on lines 165 to 168
match = re.search(
'\s+32\s+([0-9\.]+)\s+([0-9\.]+)\s+([0-9\.]+)\s+([0-9\.]+)\s+([0-9\.]+)\s+([0-9\.]+)', out_dict[node]
r'(?:│\s*)?32(?:\s*│)?\s+([0-9\.]+)\s+([0-9\.]+)\s+([0-9\.]+)\s+([0-9\.]+)\s+([0-9\.]+)\s+([0-9\.]+)',
out_dict[node],
)
Signed-off-by: Urvashi Tiwari <urtiwari.com>

@anujmittal-amd anujmittal-amd left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM

@urtiwari urtiwari merged commit 7d29edf into main Jun 19, 2026
2 checks passed
@urtiwari urtiwari deleted the urtiwari/TBissue branch June 19, 2026 18:20
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