Skip to content

Conversation

@realark
Copy link
Collaborator

@realark realark commented Jan 22, 2026

No description provided.

@realark realark added the enhancement New feature or request label Jan 22, 2026
@realark realark force-pushed the ark/remote-scorers branch 4 times, most recently from 16ec1b5 to a42b87a Compare January 22, 2026 21:34
@Override
public Object invokeFunction(
@Nonnull String functionId, @Nonnull FunctionInvokeRequest request) {
throw new RuntimeException("will not be invoked");
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I'm beginning to switch all braintrust tests away from mocks/double in favor of VCR so these test methods won't be needed

@realark realark force-pushed the ark/remote-scorers branch 3 times, most recently from 07f4a77 to 50bde7e Compare January 22, 2026 22:43

// Set up test harness with config pointing to mock API
// Set up test harness with VCR (records/replays HTTP interactions)
testHarness = TestHarness.setup();
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

starting to convert this test to use pure VCR (work in progress, but better than it was before)

@realark realark marked this pull request as ready for review January 22, 2026 22:57
@realark realark changed the title remote scorers for evals remote scorers for evals and devserver Jan 22, 2026
@realark realark changed the title remote scorers for evals and devserver add support for remote scorers for evals+devserver Jan 22, 2026
@realark realark force-pushed the ark/remote-scorers branch 2 times, most recently from 8c130e1 to b97424e Compare January 23, 2026 05:37
@realark realark force-pushed the ark/remote-scorers branch from b97424e to 0f6b221 Compare January 23, 2026 05:43
@realark realark merged commit 316b7b2 into main Jan 23, 2026
1 check passed
@realark realark deleted the ark/remote-scorers branch January 23, 2026 23:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants