ROCm / ROCm/iris

Review all existing tests

Open
#353 0 comments 1 reaction 2 assignees View on GitHub

@mawad-amd is already working on this.

Since Feb 4, 2026.

duplicate examples iris test
Dominant language
Python
Stars
202
Forks
47
Avg merge
6d 11h
Merged PRs (30d)
4

Description

Review all existing tests

We like tests but we like tests too much. We have 2,454 tests (see https://github.com/ROCm/iris/pull/348 for reference).

Your job: Review the entire test suite and tell us what's redundant, duplicate, unnecessarily time-consuming, or just plain wasteful.

What we want:

A response with:

  • Summary of findings (what's bloated, what's redundant, what's slow)
  • Specific examples of problematic tests with reasoning
  • A plan for reducing test count while maintaining coverage
  • Estimated time/cost savings

No code changes. Just analysis and recommendations.

Tools:

  • Check .github/workflows/ to understand the CI testing structure (60 test matrix jobs across 5 directories, 4 rank configs, 3 install methods)
  • Use gh CLI to check CI run times and patterns from recent PRs (start with #348)
  • Look at test execution logs for timing data
  • Dig through the test code in tests/

Give us specific recommendations with reasoning. No handwaving - we need actionable insights.

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.