-
Notifications
You must be signed in to change notification settings - Fork 38
Review all existing tests #353
Copy link
Copy link
Open
Labels
duplicateThis issue or pull request already existsThis issue or pull request already existsexamplesExamples showcasing Iris APIs and usageExamples showcasing Iris APIs and usageirisIris project issueIris project issuetestUnit/integration test related issuesUnit/integration test related issues
Metadata
Metadata
Labels
duplicateThis issue or pull request already existsThis issue or pull request already existsexamplesExamples showcasing Iris APIs and usageExamples showcasing Iris APIs and usageirisIris project issueIris project issuetestUnit/integration test related issuesUnit/integration test related issues
Review all existing tests
We like tests but we like tests too much. We have 2,454 tests (see #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:
No code changes. Just analysis and recommendations.
Tools:
.github/workflows/to understand the CI testing structure (60 test matrix jobs across 5 directories, 4 rank configs, 3 install methods)ghCLI to check CI run times and patterns from recent PRs (start with Fix DMA-BUF Export/Import with PyTorch Caching Allocator Offsets #348)tests/Give us specific recommendations with reasoning. No handwaving - we need actionable insights.