-
Notifications
You must be signed in to change notification settings - Fork 279
Introduce unified test framework for benchmark and functional tests #2803
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Closed
Closed
Changes from 9 commits
Commits
Show all changes
42 commits
Select commit
Hold shift + click to select a range
2188233
Restructure benchmarks to unified test framework
lajagapp 5adda5a
Add functional test framework with MIOpen driver test
lajagapp 64d9a40
Fix import errors
lajagapp 7db3292
Fix pre-commit errors
lajagapp ee49925
Merge branch 'main' into users/lajagapp/add-miopen-driver-test
lajagapp 8013e71
Update overall README.md
lajagapp fb3521b
Merge main into users/lajagapp/add-miopen-driver-test
lajagapp 0d2b187
Merge branch 'main' into users/lajagapp/add-miopen-driver-test
lajagapp 398c9b7
Fix pre-commit errors
lajagapp 7f6bbd3
Merge branch 'main' into users/lajagapp/add-miopen-driver-test
lajagapp bf9cf14
Merge branch 'main' into users/lajagapp/add-miopen-driver-test
lajagapp a56f9c4
Refactor test framework READMEs to eliminate duplication
lajagapp 48d75c7
Rename test matrix files for clarity
lajagapp 049af2e
Update functional_base.py
lajagapp 1ef8ecb
Merge branch 'main' into users/lajagapp/add-miopen-driver-test
lajagapp a5957e3
Merge branch 'main' into users/lajagapp/add-miopen-driver-test
lajagapp c87ecac
Refactor test_miopendriver_conv.py with JSON result parsing
lajagapp a9d7c26
Simplify exception handling and enhance functional tests
lajagapp 22d338e
Rename test_framework to extended_tests
lajagapp 23d0b0f
Update test framework name
lajagapp c0d0ed0
Merge branch 'main' into users/lajagapp/add-miopen-driver-test
lajagapp 9d5f728
Merge branch 'main' into users/lajagapp/add-miopen-driver-test
lajagapp 0112ef5
Update therock_dir path in benchmark scripts
lajagapp 49097fe
Merge branch 'main' into users/lajagapp/add-miopen-driver-test
lajagapp 1f1451b
Add extended functional tests to nightly CI
lajagapp d752368
Fix pre-commit errors
lajagapp 245c6b0
Restrict extended functional tests to nightly runs only
lajagapp 7181402
Refactor: Improve naming consistency for extended functional tests
lajagapp 82579d5
Docs: Improve extended tests documentation and environment setup guid…
lajagapp c1839fa
Refactor: Improve naming and documentation for extended test framework
lajagapp 6564b8e
Update functional/README.md
lajagapp b725f8d
Merge branch 'main' into users/lajagapp/add-miopen-driver-test
lajagapp 6590d7e
Update README with test_extended_functional_tests.yml
lajagapp f0568fe
Merge branch 'main' into users/lajagapp/add-miopen-driver-test
lajagapp 52cae62
Fix pre-commit errors
lajagapp 7387c37
Merge branch 'main' into users/lajagapp/add-miopen-driver-test
lajagapp fece860
Restore rocm-libraries submodule to match main
lajagapp 39deb97
Merge branch 'main' into users/lajagapp/add-miopen-driver-test
lajagapp e9e6ed1
Merge branch 'main' into users/lajagapp/add-miopen-driver-test
lajagapp 49a64c0
Improve functional test logging and remove redundancy
lajagapp 7329da8
Rename functional test workflow for clarity
lajagapp f58d124
Fix pre-commit errors
lajagapp File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file was deleted.
Oops, something went wrong.
This file was deleted.
Oops, something went wrong.
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
Oops, something went wrong.
Oops, something went wrong.
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
could we perhaps be more specific than
test_framework? maybeextendedor a better wordtest_framework is broad
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the feedback. Renamed top-level folder from
test_frameworktoextended_testsfor better clarity and updated all references accordingly.