Skip to content

chore: Grove support for accuracy tests#1205

Merged
manupedrozo merged 1 commit into
mainfrom
accuracy-tests-grove-support
May 26, 2026
Merged

chore: Grove support for accuracy tests#1205
manupedrozo merged 1 commit into
mainfrom
accuracy-tests-grove-support

Conversation

@manupedrozo

Copy link
Copy Markdown
Collaborator

Summary

Adds Grove support for accuracy tests, which facilitates running accuracy tests locally.

Similar approach as #1140 but limited to adding Grove support only. CI is unmodified, we can update gh secrets as needed if we choose to move to Grove.

  • Included the MDB_ACCURACY_MAX_WORKERS idea from the linked PR as well, useful when running against multiple models.

Checklist

@manupedrozo

Copy link
Copy Markdown
Collaborator Author

Integ test failure is unrelated to the PR changes.

@manupedrozo manupedrozo marked this pull request as ready for review May 25, 2026 17:01
@manupedrozo manupedrozo requested a review from a team as a code owner May 25, 2026 17:01
@manupedrozo manupedrozo requested review from blva and removed request for a team May 25, 2026 17:01
@manupedrozo manupedrozo added the no-title-validation Add this label to disable the title check for this PR. label May 25, 2026
@manupedrozo manupedrozo enabled auto-merge (squash) May 26, 2026 14:47
@manupedrozo manupedrozo merged commit 948c5cb into main May 26, 2026
44 of 47 checks passed
@manupedrozo manupedrozo deleted the accuracy-tests-grove-support branch May 26, 2026 14:54
@coveralls

Copy link
Copy Markdown
Collaborator

Coverage Report for CI Build 26410885242

Warning

Build has drifted: This PR's base is out of sync with its target branch, so coverage data may include unrelated changes.
Quick fix: rebase this PR. Learn more →

Warning

No base build found for commit 425e7c2 on main.
Coverage changes can't be calculated without a base build.
If a base build is processing, this comment will update automatically when it completes.

Coverage: 79.737%

Details

  • Patch coverage: No coverable lines changed in this PR.

Uncovered Changes

No uncovered changes found.

Coverage Regressions

Requires a base build to compare against. How to fix this →


Coverage Stats

Coverage Status
Relevant Lines: 4002
Covered Lines: 3393
Line Coverage: 84.78%
Relevant Branches: 2527
Covered Branches: 1813
Branch Coverage: 71.75%
Branches in Coverage %: Yes
Coverage Strength: 161.28 hits per line

💛 - Coveralls

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

no-title-validation Add this label to disable the title check for this PR. type: chore

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants