Skip to content

Comments

chore(benchmarks): removed scorer#740

Merged
james-rl merged 1 commit intomainfrom
james/rm-validate-scorer
Feb 13, 2026
Merged

chore(benchmarks): removed scorer#740
james-rl merged 1 commit intomainfrom
james/rm-validate-scorer

Conversation

@james-rl
Copy link
Contributor

Removed unused validate scorer references in docs

@james-rl james-rl requested review from dines-rl and sid-rl February 13, 2026 01:33
@sid-rl
Copy link
Contributor

sid-rl commented Feb 13, 2026

what is the intent of this pr? do we want to just remove scorer.validate() from docstrings or remove the method itself? do we want to keep the method but hide it from documentation entirely? and what about for the non-ooscenarios.validate()?

@james-rl
Copy link
Contributor Author

what is the intent of this pr? do we want to just remove scorer.validate() from docstrings or remove the method itself? do we want to keep the method but hide it from documentation entirely? and what about for the non-ooscenarios.validate()?

The API endpoint has been hidden for a week and nobody has been using it. This is the second step of getting rid of it

@james-rl james-rl merged commit 1033264 into main Feb 13, 2026
7 checks passed
@james-rl james-rl deleted the james/rm-validate-scorer branch February 13, 2026 18:50
@stainless-app stainless-app bot mentioned this pull request Feb 13, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants