Skip to content

[hipDNN] Include hipdnn test executables that end with singular 'test'#3616

Merged
CMiservaAMD merged 4 commits intomainfrom
users/cmiserva/fix-hipdnn-test-filename-glob
Mar 2, 2026
Merged

[hipDNN] Include hipdnn test executables that end with singular 'test'#3616
CMiservaAMD merged 4 commits intomainfrom
users/cmiserva/fix-hipdnn-test-filename-glob

Conversation

@CMiservaAMD
Copy link
Copy Markdown
Contributor

Motivation

A test executable was added to exercise a specific scenario. The filename ended in "_test" instead of the typical "_tests" and was therefore not being included in the test archive.

Technical Details

Changed test glob patter to end with "_test*" instead of the original "_tests*".

Test Plan

CI tests pass.

Test Result

Submission Checklist

@CMiservaAMD CMiservaAMD changed the title [hipDNN] Include hipdnn test executables that end singular 'test' [hipDNN] Include hipdnn test executables that end with singular 'test' Feb 25, 2026
Copy link
Copy Markdown
Contributor

@BrianHarrisonAMD BrianHarrisonAMD left a comment

Choose a reason for hiding this comment

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

Heh silly.
I think this can go in.

@CMiservaAMD CMiservaAMD merged commit 8d1e6be into main Mar 2, 2026
93 of 95 checks passed
@CMiservaAMD CMiservaAMD deleted the users/cmiserva/fix-hipdnn-test-filename-glob branch March 2, 2026 01:29
@github-project-automation github-project-automation Bot moved this from TODO to Done in TheRock Triage Mar 2, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants