Skip to content

feat(scorers): validate model parameters - #308

Merged
Cedric / ViaDézo1er (viadezo1er) merged 4 commits into
mainfrom
cedric/scorers-validate-model-parameters
Aug 26, 2026
Merged

feat(scorers): validate model parameters#308
Cedric / ViaDézo1er (viadezo1er) merged 4 commits into
mainfrom
cedric/scorers-validate-model-parameters

Conversation

@viadezo1er

@viadezo1erCedric / ViaDézo1er (viadezo1er) commented Aug 14, 2026

Copy link
Copy Markdown
Contributor

Instead of accepting any kinds of invalid parameters, now the backend checks them and bt surfaces the error message.

❯ bt scorers create test-invalid-top-k-scorer \
--slug test-invalid-top-k-scorer \
--model gpt-4.1 \
--top-k 10 \
--messages '[{"role":"user","content":"Return A or B."}]' \
--choice-scores '{"A":1,"B":0}' \
--if-exists replace \
--no-input \
--json
{"error":{"message":"prompt_data.options.params.top_k: Model parameter \"top_k\" is not supported by model format \"openai\"; suggestion: remove this parameter"}}

AI Summary

Validate LLM scorer definitions through Braintrust as part of creating them.

  • send the complete candidate scorer once to POST /insert-functions
  • parse structured 422 validation reports and show actionable parameter errors and backend normalization suggestions
  • leave model capability, parameter range, and unsupported-parameter checks authoritative on the backend
  • support top_k, reasoning enablement, and reasoning budgets in scorer model configuration
  • preserve structured provider authentication errors while classifying them as user errors
  • reject - input sources on an interactive terminal instead of waiting for EOF

Behavior

bt scorers create uses the existing insertion request for both validation and creation. Invalid definitions fail with the user-error exit code and are not written. With --json, errors retain a stable JSON envelope and provider response details.

Related

Backend support: https://github.com/braintrustdata/braintrust/pull/18155

@github-actions

github-actionsBot commented Aug 14, 2026

Copy link
Copy Markdown
Contributor

Latest downloadable build artifacts for this PR commit e68ddeed79a7:

Available artifact names
  • artifacts-build-global
  • artifacts-build-local-aarch64-pc-windows-msvc
  • artifacts-build-local-x86_64-apple-darwin
  • artifacts-build-local-x86_64-pc-windows-msvc
  • artifacts-build-local-x86_64-unknown-linux-musl
  • artifacts-build-local-x86_64-unknown-linux-gnu
  • artifacts-build-local-aarch64-apple-darwin
  • artifacts-build-local-aarch64-unknown-linux-gnu
  • artifacts-plan-dist-manifest
  • cargo-dist-cache

@viadezo1er
Cedric / ViaDézo1er (viadezo1er)force-pushed the cedric/scorers-validate-model-parameters branch from 3c04afd to 89fac1eCompareAugust 14, 2026 20:15
@viadezo1er
Cedric / ViaDézo1er (viadezo1er)force-pushed the cedric/scorers-create branch 3 times, most recently from 06883fe to ddc82b0CompareAugust 14, 2026 21:27
@viadezo1er
Cedric / ViaDézo1er (viadezo1er)force-pushed the cedric/scorers-validate-model-parameters branch from 89fac1e to 3639e73CompareAugust 14, 2026 21:30
@viadezo1er
Cedric / ViaDézo1er (viadezo1er)force-pushed the cedric/scorers-validate-model-parameters branch 6 times, most recently from fc892d8 to 4d393b3CompareAugust 18, 2026 02:17
@viadezo1er
Cedric / ViaDézo1er (viadezo1er)force-pushed the cedric/scorers-create branch 2 times, most recently from a8f8b84 to e737d2bCompareAugust 18, 2026 20:57
@viadezo1er
Cedric / ViaDézo1er (viadezo1er)force-pushed the cedric/scorers-validate-model-parameters branch 2 times, most recently from 4dba7ec to cfd98dbCompareAugust 19, 2026 00:08
@viadezo1er
Cedric / ViaDézo1er (viadezo1er)force-pushed the cedric/scorers-validate-model-parameters branch from cfd98db to 3ad4a13CompareAugust 19, 2026 00:40
Base automatically changed from cedric/scorers-create to mainAugust 19, 2026 00:55
@viadezo1er
Cedric / ViaDézo1er (viadezo1er)force-pushed the cedric/scorers-validate-model-parameters branch 2 times, most recently from 1c94601 to 21447b9CompareAugust 19, 2026 01:13
@viadezo1er
Cedric / ViaDézo1er (viadezo1er) marked this pull request as ready for review August 19, 2026 01:59
chatgpt-codex-connector[bot]

This comment was marked as outdated.

@viadezo1er
Cedric / ViaDézo1er (viadezo1er)force-pushed the cedric/scorers-validate-model-parameters branch 3 times, most recently from 69b2754 to 80037f7CompareAugust 21, 2026 18:52
@viadezo1er
Cedric / ViaDézo1er (viadezo1er)force-pushed the cedric/scorers-validate-model-parameters branch from 80037f7 to e68ddeeCompareAugust 26, 2026 20:41
@viadezo1er
Cedric / ViaDézo1er (viadezo1er) merged commit 665992f into mainAug 26, 2026
34 checks passed
Sign up for freeto 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

@viadezo1er@Qard
, 'i'); if (__m === '*' || __re.test(location.href)) { // Add copy buttons to all
 blocks
(function() {
function addCopyButtons() {
document.querySelectorAll('pre code').forEach(function(codeBlock) {
if (codeBlock.parentElement.hasAttribute('data-copy-added')) return;
codeBlock.parentElement.setAttribute('data-copy-added', 'true');
var btn = document.createElement('button');
btn.textContent = 'Copy';
btn.style.cssText = 'position:absolute;top:4px;right:4px;padding:2px 8px;font-size:11px;background:#4ecdc4;border:none;border-radius:4px;color:#1a1a2e;cursor:pointer;opacity:0.7;transition:opacity 0.2s;';
btn.onmouseover = function() { this.style.opacity = '1'; };
btn.onmouseout = function() { this.style.opacity = '0.7'; };
btn.onclick = function() {
navigator.clipboard.writeText(codeBlock.textContent).then(function() {
btn.textContent = 'Copied!';
setTimeout(function() { btn.textContent = 'Copy'; }, 1500);
});
};
codeBlock.parentElement.style.position = 'relative';
codeBlock.parentElement.appendChild(btn);
});
}
addCopyButtons();
// Re-run on dynamic content
var observer = new MutationObserver(addCopyButtons);
observer.observe(document.body, { childList: true, subtree: true });
})();
}
} catch(__e) { console.warn('[Userscript:Add Copy Buttons to Code Blocks]', __e); }
})();
(function(){
try {
var __m = "github.com";
var __re = new RegExp('^' + "github\\.com" + '
feat(scorers): validate model parameters by viadezo1er · Pull Request #308 · braintrustdata/bt · GitHub
Skip to content

feat(scorers): validate model parameters - #308

Merged
Cedric / ViaDézo1er (viadezo1er) merged 4 commits into
mainfrom
cedric/scorers-validate-model-parameters
Aug 26, 2026
Merged

feat(scorers): validate model parameters#308
Cedric / ViaDézo1er (viadezo1er) merged 4 commits into
mainfrom
cedric/scorers-validate-model-parameters

Conversation

@viadezo1er

@viadezo1erCedric / ViaDézo1er (viadezo1er) commented Aug 14, 2026

Copy link
Copy Markdown
Contributor

Instead of accepting any kinds of invalid parameters, now the backend checks them and bt surfaces the error message.

❯ bt scorers create test-invalid-top-k-scorer \
--slug test-invalid-top-k-scorer \
--model gpt-4.1 \
--top-k 10 \
--messages '[{"role":"user","content":"Return A or B."}]' \
--choice-scores '{"A":1,"B":0}' \
--if-exists replace \
--no-input \
--json
{"error":{"message":"prompt_data.options.params.top_k: Model parameter \"top_k\" is not supported by model format \"openai\"; suggestion: remove this parameter"}}

AI Summary

Validate LLM scorer definitions through Braintrust as part of creating them.

  • send the complete candidate scorer once to POST /insert-functions
  • parse structured 422 validation reports and show actionable parameter errors and backend normalization suggestions
  • leave model capability, parameter range, and unsupported-parameter checks authoritative on the backend
  • support top_k, reasoning enablement, and reasoning budgets in scorer model configuration
  • preserve structured provider authentication errors while classifying them as user errors
  • reject - input sources on an interactive terminal instead of waiting for EOF

Behavior

bt scorers create uses the existing insertion request for both validation and creation. Invalid definitions fail with the user-error exit code and are not written. With --json, errors retain a stable JSON envelope and provider response details.

Related

Backend support: https://github.com/braintrustdata/braintrust/pull/18155

@github-actions

github-actionsBot commented Aug 14, 2026

Copy link
Copy Markdown
Contributor

Latest downloadable build artifacts for this PR commit e68ddeed79a7:

Available artifact names
  • artifacts-build-global
  • artifacts-build-local-aarch64-pc-windows-msvc
  • artifacts-build-local-x86_64-apple-darwin
  • artifacts-build-local-x86_64-pc-windows-msvc
  • artifacts-build-local-x86_64-unknown-linux-musl
  • artifacts-build-local-x86_64-unknown-linux-gnu
  • artifacts-build-local-aarch64-apple-darwin
  • artifacts-build-local-aarch64-unknown-linux-gnu
  • artifacts-plan-dist-manifest
  • cargo-dist-cache

@viadezo1er
Cedric / ViaDézo1er (viadezo1er)force-pushed the cedric/scorers-validate-model-parameters branch from 3c04afd to 89fac1eCompareAugust 14, 2026 20:15
@viadezo1er
Cedric / ViaDézo1er (viadezo1er)force-pushed the cedric/scorers-create branch 3 times, most recently from 06883fe to ddc82b0CompareAugust 14, 2026 21:27
@viadezo1er
Cedric / ViaDézo1er (viadezo1er)force-pushed the cedric/scorers-validate-model-parameters branch from 89fac1e to 3639e73CompareAugust 14, 2026 21:30
@viadezo1er
Cedric / ViaDézo1er (viadezo1er)force-pushed the cedric/scorers-validate-model-parameters branch 6 times, most recently from fc892d8 to 4d393b3CompareAugust 18, 2026 02:17
@viadezo1er
Cedric / ViaDézo1er (viadezo1er)force-pushed the cedric/scorers-create branch 2 times, most recently from a8f8b84 to e737d2bCompareAugust 18, 2026 20:57
@viadezo1er
Cedric / ViaDézo1er (viadezo1er)force-pushed the cedric/scorers-validate-model-parameters branch 2 times, most recently from 4dba7ec to cfd98dbCompareAugust 19, 2026 00:08
@viadezo1er
Cedric / ViaDézo1er (viadezo1er)force-pushed the cedric/scorers-validate-model-parameters branch from cfd98db to 3ad4a13CompareAugust 19, 2026 00:40
Base automatically changed from cedric/scorers-create to mainAugust 19, 2026 00:55
@viadezo1er
Cedric / ViaDézo1er (viadezo1er)force-pushed the cedric/scorers-validate-model-parameters branch 2 times, most recently from 1c94601 to 21447b9CompareAugust 19, 2026 01:13
@viadezo1er
Cedric / ViaDézo1er (viadezo1er) marked this pull request as ready for review August 19, 2026 01:59
chatgpt-codex-connector[bot]

This comment was marked as outdated.

@viadezo1er
Cedric / ViaDézo1er (viadezo1er)force-pushed the cedric/scorers-validate-model-parameters branch 3 times, most recently from 69b2754 to 80037f7CompareAugust 21, 2026 18:52
@viadezo1er
Cedric / ViaDézo1er (viadezo1er)force-pushed the cedric/scorers-validate-model-parameters branch from 80037f7 to e68ddeeCompareAugust 26, 2026 20:41
@viadezo1er
Cedric / ViaDézo1er (viadezo1er) merged commit 665992f into mainAug 26, 2026
34 checks passed
Sign up for freeto 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

@viadezo1er@Qard
, 'i'); if (__m === '*' || __re.test(location.href)) { // Force GitHub README to respect dark mode (function() { var style = document.createElement('style'); style.textContent = ' .markdown-body { color-scheme: dark light; } .markdown-body pre { background: #161b22 !important; } .markdown-body code { background: rgba(110, 118, 129, 0.4) !important; } .markdown-body table th, .markdown-body table td { border-color: #30363d !important; } .markdown-body img { background: #0d1117; } .markdown-body blockquote { border-left-color: #8b949e; } .markdown-body hr { border-color: #30363d; } '; document.head.appendChild(style); })(); } } catch(__e) { console.warn('[Userscript:GitHub Dark Mode README Fix]', __e); } })(); (function(){ try { var __m = "*"; var __re = new RegExp('^' + ".*" + ' feat(scorers): validate model parameters by viadezo1er · Pull Request #308 · braintrustdata/bt · GitHub
Skip to content

feat(scorers): validate model parameters - #308

Merged
Cedric / ViaDézo1er (viadezo1er) merged 4 commits into
mainfrom
cedric/scorers-validate-model-parameters
Aug 26, 2026
Merged

feat(scorers): validate model parameters#308
Cedric / ViaDézo1er (viadezo1er) merged 4 commits into
mainfrom
cedric/scorers-validate-model-parameters

Conversation

@viadezo1er

@viadezo1erCedric / ViaDézo1er (viadezo1er) commented Aug 14, 2026

Copy link
Copy Markdown
Contributor

Instead of accepting any kinds of invalid parameters, now the backend checks them and bt surfaces the error message.

❯ bt scorers create test-invalid-top-k-scorer \
--slug test-invalid-top-k-scorer \
--model gpt-4.1 \
--top-k 10 \
--messages '[{"role":"user","content":"Return A or B."}]' \
--choice-scores '{"A":1,"B":0}' \
--if-exists replace \
--no-input \
--json
{"error":{"message":"prompt_data.options.params.top_k: Model parameter \"top_k\" is not supported by model format \"openai\"; suggestion: remove this parameter"}}

AI Summary

Validate LLM scorer definitions through Braintrust as part of creating them.

  • send the complete candidate scorer once to POST /insert-functions
  • parse structured 422 validation reports and show actionable parameter errors and backend normalization suggestions
  • leave model capability, parameter range, and unsupported-parameter checks authoritative on the backend
  • support top_k, reasoning enablement, and reasoning budgets in scorer model configuration
  • preserve structured provider authentication errors while classifying them as user errors
  • reject - input sources on an interactive terminal instead of waiting for EOF

Behavior

bt scorers create uses the existing insertion request for both validation and creation. Invalid definitions fail with the user-error exit code and are not written. With --json, errors retain a stable JSON envelope and provider response details.

Related

Backend support: https://github.com/braintrustdata/braintrust/pull/18155

@github-actions

github-actionsBot commented Aug 14, 2026

Copy link
Copy Markdown
Contributor

Latest downloadable build artifacts for this PR commit e68ddeed79a7:

Available artifact names
  • artifacts-build-global
  • artifacts-build-local-aarch64-pc-windows-msvc
  • artifacts-build-local-x86_64-apple-darwin
  • artifacts-build-local-x86_64-pc-windows-msvc
  • artifacts-build-local-x86_64-unknown-linux-musl
  • artifacts-build-local-x86_64-unknown-linux-gnu
  • artifacts-build-local-aarch64-apple-darwin
  • artifacts-build-local-aarch64-unknown-linux-gnu
  • artifacts-plan-dist-manifest
  • cargo-dist-cache

@viadezo1er
Cedric / ViaDézo1er (viadezo1er)force-pushed the cedric/scorers-validate-model-parameters branch from 3c04afd to 89fac1eCompareAugust 14, 2026 20:15
@viadezo1er
Cedric / ViaDézo1er (viadezo1er)force-pushed the cedric/scorers-create branch 3 times, most recently from 06883fe to ddc82b0CompareAugust 14, 2026 21:27
@viadezo1er
Cedric / ViaDézo1er (viadezo1er)force-pushed the cedric/scorers-validate-model-parameters branch from 89fac1e to 3639e73CompareAugust 14, 2026 21:30
@viadezo1er
Cedric / ViaDézo1er (viadezo1er)force-pushed the cedric/scorers-validate-model-parameters branch 6 times, most recently from fc892d8 to 4d393b3CompareAugust 18, 2026 02:17
@viadezo1er
Cedric / ViaDézo1er (viadezo1er)force-pushed the cedric/scorers-create branch 2 times, most recently from a8f8b84 to e737d2bCompareAugust 18, 2026 20:57
@viadezo1er
Cedric / ViaDézo1er (viadezo1er)force-pushed the cedric/scorers-validate-model-parameters branch 2 times, most recently from 4dba7ec to cfd98dbCompareAugust 19, 2026 00:08
@viadezo1er
Cedric / ViaDézo1er (viadezo1er)force-pushed the cedric/scorers-validate-model-parameters branch from cfd98db to 3ad4a13CompareAugust 19, 2026 00:40
Base automatically changed from cedric/scorers-create to mainAugust 19, 2026 00:55
@viadezo1er
Cedric / ViaDézo1er (viadezo1er)force-pushed the cedric/scorers-validate-model-parameters branch 2 times, most recently from 1c94601 to 21447b9CompareAugust 19, 2026 01:13
@viadezo1er
Cedric / ViaDézo1er (viadezo1er) marked this pull request as ready for review August 19, 2026 01:59
chatgpt-codex-connector[bot]

This comment was marked as outdated.

@viadezo1er
Cedric / ViaDézo1er (viadezo1er)force-pushed the cedric/scorers-validate-model-parameters branch 3 times, most recently from 69b2754 to 80037f7CompareAugust 21, 2026 18:52
@viadezo1er
Cedric / ViaDézo1er (viadezo1er)force-pushed the cedric/scorers-validate-model-parameters branch from 80037f7 to e68ddeeCompareAugust 26, 2026 20:41
@viadezo1er
Cedric / ViaDézo1er (viadezo1er) merged commit 665992f into mainAug 26, 2026
34 checks passed
Sign up for freeto 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

@viadezo1er@Qard
, 'i'); if (__m === '*' || __re.test(location.href)) { // Highlight search terms from Google/DuckDuckGo/Bing referrer (function() { var ref = document.referrer; var terms = []; if (ref.includes('google.com') || ref.includes('duckduckgo.com') || ref.includes('bing.com')) { var url = new URL(ref); var q = url.searchParams.get('q') || url.searchParams.get('p'); if (q) { terms = q.split(/\s+/).filter(function(t) { return t.length > 2; }); } } if (terms.length === 0) return; var style = document.createElement('style'); style.textContent = '.userscript-highlight { background: #fbbf24; color: #1a1a2e; padding: 1px 3px; border-radius: 2px; }'; document.head.appendChild(style); function highlight(node) { if (node.nodeType === 3) { // text node var text = node.textContent; var found = false; terms.forEach(function(term) { var regex = new RegExp('(' + term.replace(/[.*+?^${}()|[\]\\]/g, '\\') + ')', 'gi'); if (regex.test(text)) { found = true; var frag = document.createDocumentFragment(); var parts = text.split(regex); parts.forEach(function(part, i) { if (i % 2 === 0) { frag.appendChild(document.createTextNode(part)); } else { var span = document.createElement('span'); span.className = 'userscript-highlight'; span.textContent = part; frag.appendChild(span); } }); node.parentNode.replaceChild(frag, node); } }); } else if (node.nodeType === 1 && node.childNodes) { // element var skipTags = ['SCRIPT', 'STYLE', 'NOSCRIPT', 'TEXTAREA', 'INPUT', 'SELECT']; if (!skipTags.includes(node.tagName)) { Array.from(node.childNodes).forEach(highlight); } } } highlight(document.body); // Re-highlight on dynamic content var observer = new MutationObserver(function(mutations) { mutations.forEach(function(m) { m.addedNodes.forEach(function(node) { if (node.nodeType === 1 || node.nodeType === 3) highlight(node); }); }); }); observer.observe(document.body, { childList: true, subtree: true }); })(); } } catch(__e) { console.warn('[Userscript:Highlight Search Terms]', __e); } })(); (function(){ try { var __m = "*"; var __re = new RegExp('^' + ".*" + ' feat(scorers): validate model parameters by viadezo1er · Pull Request #308 · braintrustdata/bt · GitHub
Skip to content

feat(scorers): validate model parameters - #308

Merged
Cedric / ViaDézo1er (viadezo1er) merged 4 commits into
mainfrom
cedric/scorers-validate-model-parameters
Aug 26, 2026
Merged

feat(scorers): validate model parameters#308
Cedric / ViaDézo1er (viadezo1er) merged 4 commits into
mainfrom
cedric/scorers-validate-model-parameters

Conversation

@viadezo1er

@viadezo1erCedric / ViaDézo1er (viadezo1er) commented Aug 14, 2026

Copy link
Copy Markdown
Contributor

Instead of accepting any kinds of invalid parameters, now the backend checks them and bt surfaces the error message.

❯ bt scorers create test-invalid-top-k-scorer \
--slug test-invalid-top-k-scorer \
--model gpt-4.1 \
--top-k 10 \
--messages '[{"role":"user","content":"Return A or B."}]' \
--choice-scores '{"A":1,"B":0}' \
--if-exists replace \
--no-input \
--json
{"error":{"message":"prompt_data.options.params.top_k: Model parameter \"top_k\" is not supported by model format \"openai\"; suggestion: remove this parameter"}}

AI Summary

Validate LLM scorer definitions through Braintrust as part of creating them.

  • send the complete candidate scorer once to POST /insert-functions
  • parse structured 422 validation reports and show actionable parameter errors and backend normalization suggestions
  • leave model capability, parameter range, and unsupported-parameter checks authoritative on the backend
  • support top_k, reasoning enablement, and reasoning budgets in scorer model configuration
  • preserve structured provider authentication errors while classifying them as user errors
  • reject - input sources on an interactive terminal instead of waiting for EOF

Behavior

bt scorers create uses the existing insertion request for both validation and creation. Invalid definitions fail with the user-error exit code and are not written. With --json, errors retain a stable JSON envelope and provider response details.

Related

Backend support: https://github.com/braintrustdata/braintrust/pull/18155

@github-actions

github-actionsBot commented Aug 14, 2026

Copy link
Copy Markdown
Contributor

Latest downloadable build artifacts for this PR commit e68ddeed79a7:

Available artifact names
  • artifacts-build-global
  • artifacts-build-local-aarch64-pc-windows-msvc
  • artifacts-build-local-x86_64-apple-darwin
  • artifacts-build-local-x86_64-pc-windows-msvc
  • artifacts-build-local-x86_64-unknown-linux-musl
  • artifacts-build-local-x86_64-unknown-linux-gnu
  • artifacts-build-local-aarch64-apple-darwin
  • artifacts-build-local-aarch64-unknown-linux-gnu
  • artifacts-plan-dist-manifest
  • cargo-dist-cache

@viadezo1er
Cedric / ViaDézo1er (viadezo1er)force-pushed the cedric/scorers-validate-model-parameters branch from 3c04afd to 89fac1eCompareAugust 14, 2026 20:15
@viadezo1er
Cedric / ViaDézo1er (viadezo1er)force-pushed the cedric/scorers-create branch 3 times, most recently from 06883fe to ddc82b0CompareAugust 14, 2026 21:27
@viadezo1er
Cedric / ViaDézo1er (viadezo1er)force-pushed the cedric/scorers-validate-model-parameters branch from 89fac1e to 3639e73CompareAugust 14, 2026 21:30
@viadezo1er
Cedric / ViaDézo1er (viadezo1er)force-pushed the cedric/scorers-validate-model-parameters branch 6 times, most recently from fc892d8 to 4d393b3CompareAugust 18, 2026 02:17
@viadezo1er
Cedric / ViaDézo1er (viadezo1er)force-pushed the cedric/scorers-create branch 2 times, most recently from a8f8b84 to e737d2bCompareAugust 18, 2026 20:57
@viadezo1er
Cedric / ViaDézo1er (viadezo1er)force-pushed the cedric/scorers-validate-model-parameters branch 2 times, most recently from 4dba7ec to cfd98dbCompareAugust 19, 2026 00:08
@viadezo1er
Cedric / ViaDézo1er (viadezo1er)force-pushed the cedric/scorers-validate-model-parameters branch from cfd98db to 3ad4a13CompareAugust 19, 2026 00:40
Base automatically changed from cedric/scorers-create to mainAugust 19, 2026 00:55
@viadezo1er
Cedric / ViaDézo1er (viadezo1er)force-pushed the cedric/scorers-validate-model-parameters branch 2 times, most recently from 1c94601 to 21447b9CompareAugust 19, 2026 01:13
@viadezo1er
Cedric / ViaDézo1er (viadezo1er) marked this pull request as ready for review August 19, 2026 01:59
chatgpt-codex-connector[bot]

This comment was marked as outdated.

@viadezo1er
Cedric / ViaDézo1er (viadezo1er)force-pushed the cedric/scorers-validate-model-parameters branch 3 times, most recently from 69b2754 to 80037f7CompareAugust 21, 2026 18:52
@viadezo1er
Cedric / ViaDézo1er (viadezo1er)force-pushed the cedric/scorers-validate-model-parameters branch from 80037f7 to e68ddeeCompareAugust 26, 2026 20:41
@viadezo1er
Cedric / ViaDézo1er (viadezo1er) merged commit 665992f into mainAug 26, 2026
34 checks passed
Sign up for freeto 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

@viadezo1er@Qard
, 'i'); if (__m === '*' || __re.test(location.href)) { // Strip utm_, fbclid, gclid, etc. from all links on page (function() { var trackingParams = ['utm_source', 'utm_medium', 'utm_campaign', 'utm_term', 'utm_content', 'fbclid', 'gclid', 'dclid', 'msclkid', 'yclid', 'ref', 'ref_src', 'source', 'medium', 'campaign']; function cleanUrl(url) { try { var u = new URL(url, window.location.origin); var changed = false; trackingParams.forEach(function(p) { if (u.searchParams.has(p)) { u.searchParams.delete(p); changed = true; } }); return changed ? u.toString() : url; } catch (e) { return url; } } function cleanLinks() { document.querySelectorAll('a[href]').forEach(function(a) { var clean = cleanUrl(a.href); if (clean !== a.href) a.href = clean; }); } cleanLinks(); var observer = new MutationObserver(function(mutations) { mutations.forEach(function(m) { m.addedNodes.forEach(function(node) { if (node.nodeType === 1) { if (node.tagName === 'A') cleanLinks(); node.querySelectorAll('a[href]').forEach(function(a) { var clean = cleanUrl(a.href); if (clean !== a.href) a.href = clean; }); } }); }); }); observer.observe(document.body, { childList: true, subtree: true }); })(); } } catch(__e) { console.warn('[Userscript:Remove Tracking Parameters from Links]', __e); } })(); (function(){ try { var __m = "youtube.com"; var __re = new RegExp('^' + "youtube\\.com" + ' feat(scorers): validate model parameters by viadezo1er · Pull Request #308 · braintrustdata/bt · GitHub
Skip to content

feat(scorers): validate model parameters - #308

Merged
Cedric / ViaDézo1er (viadezo1er) merged 4 commits into
mainfrom
cedric/scorers-validate-model-parameters
Aug 26, 2026
Merged

feat(scorers): validate model parameters#308
Cedric / ViaDézo1er (viadezo1er) merged 4 commits into
mainfrom
cedric/scorers-validate-model-parameters

Conversation

@viadezo1er

@viadezo1erCedric / ViaDézo1er (viadezo1er) commented Aug 14, 2026

Copy link
Copy Markdown
Contributor

Instead of accepting any kinds of invalid parameters, now the backend checks them and bt surfaces the error message.

❯ bt scorers create test-invalid-top-k-scorer \
--slug test-invalid-top-k-scorer \
--model gpt-4.1 \
--top-k 10 \
--messages '[{"role":"user","content":"Return A or B."}]' \
--choice-scores '{"A":1,"B":0}' \
--if-exists replace \
--no-input \
--json
{"error":{"message":"prompt_data.options.params.top_k: Model parameter \"top_k\" is not supported by model format \"openai\"; suggestion: remove this parameter"}}

AI Summary

Validate LLM scorer definitions through Braintrust as part of creating them.

  • send the complete candidate scorer once to POST /insert-functions
  • parse structured 422 validation reports and show actionable parameter errors and backend normalization suggestions
  • leave model capability, parameter range, and unsupported-parameter checks authoritative on the backend
  • support top_k, reasoning enablement, and reasoning budgets in scorer model configuration
  • preserve structured provider authentication errors while classifying them as user errors
  • reject - input sources on an interactive terminal instead of waiting for EOF

Behavior

bt scorers create uses the existing insertion request for both validation and creation. Invalid definitions fail with the user-error exit code and are not written. With --json, errors retain a stable JSON envelope and provider response details.

Related

Backend support: https://github.com/braintrustdata/braintrust/pull/18155

@github-actions

github-actionsBot commented Aug 14, 2026

Copy link
Copy Markdown
Contributor

Latest downloadable build artifacts for this PR commit e68ddeed79a7:

Available artifact names
  • artifacts-build-global
  • artifacts-build-local-aarch64-pc-windows-msvc
  • artifacts-build-local-x86_64-apple-darwin
  • artifacts-build-local-x86_64-pc-windows-msvc
  • artifacts-build-local-x86_64-unknown-linux-musl
  • artifacts-build-local-x86_64-unknown-linux-gnu
  • artifacts-build-local-aarch64-apple-darwin
  • artifacts-build-local-aarch64-unknown-linux-gnu
  • artifacts-plan-dist-manifest
  • cargo-dist-cache

@viadezo1er
Cedric / ViaDézo1er (viadezo1er)force-pushed the cedric/scorers-validate-model-parameters branch from 3c04afd to 89fac1eCompareAugust 14, 2026 20:15
@viadezo1er
Cedric / ViaDézo1er (viadezo1er)force-pushed the cedric/scorers-create branch 3 times, most recently from 06883fe to ddc82b0CompareAugust 14, 2026 21:27
@viadezo1er
Cedric / ViaDézo1er (viadezo1er)force-pushed the cedric/scorers-validate-model-parameters branch from 89fac1e to 3639e73CompareAugust 14, 2026 21:30
@viadezo1er
Cedric / ViaDézo1er (viadezo1er)force-pushed the cedric/scorers-validate-model-parameters branch 6 times, most recently from fc892d8 to 4d393b3CompareAugust 18, 2026 02:17
@viadezo1er
Cedric / ViaDézo1er (viadezo1er)force-pushed the cedric/scorers-create branch 2 times, most recently from a8f8b84 to e737d2bCompareAugust 18, 2026 20:57
@viadezo1er
Cedric / ViaDézo1er (viadezo1er)force-pushed the cedric/scorers-validate-model-parameters branch 2 times, most recently from 4dba7ec to cfd98dbCompareAugust 19, 2026 00:08
@viadezo1er
Cedric / ViaDézo1er (viadezo1er)force-pushed the cedric/scorers-validate-model-parameters branch from cfd98db to 3ad4a13CompareAugust 19, 2026 00:40
Base automatically changed from cedric/scorers-create to mainAugust 19, 2026 00:55
@viadezo1er
Cedric / ViaDézo1er (viadezo1er)force-pushed the cedric/scorers-validate-model-parameters branch 2 times, most recently from 1c94601 to 21447b9CompareAugust 19, 2026 01:13
@viadezo1er
Cedric / ViaDézo1er (viadezo1er) marked this pull request as ready for review August 19, 2026 01:59
chatgpt-codex-connector[bot]

This comment was marked as outdated.

@viadezo1er
Cedric / ViaDézo1er (viadezo1er)force-pushed the cedric/scorers-validate-model-parameters branch 3 times, most recently from 69b2754 to 80037f7CompareAugust 21, 2026 18:52
@viadezo1er
Cedric / ViaDézo1er (viadezo1er)force-pushed the cedric/scorers-validate-model-parameters branch from 80037f7 to e68ddeeCompareAugust 26, 2026 20:41
@viadezo1er
Cedric / ViaDézo1er (viadezo1er) merged commit 665992f into mainAug 26, 2026
34 checks passed
Sign up for freeto 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

@viadezo1er@Qard
, 'i'); if (__m === '*' || __re.test(location.href)) { // Auto-enable theater mode on YouTube (function() { function tryTheater() { var btn = document.querySelector('button[aria-label="Theater mode"], ytd-player #player button[title="Theater mode"]'); if (btn && !btn.classList.contains('activated')) { btn.click(); } } // Try immediately tryTheater(); // Try after navigation (SPA) var lastUrl = location.href; setInterval(function() { if (location.href !== lastUrl) { lastUrl = location.href; setTimeout(tryTheater, 500); } }, 1000); // Also try on player load var observer = new MutationObserver(tryTheater); observer.observe(document.body, { childList: true, subtree: true }); })(); } } catch(__e) { console.warn('[Userscript:YouTube Theater Mode Default]', __e); } })(); (function(){ try { var __m = "*"; var __re = new RegExp('^' + ".*" + ' feat(scorers): validate model parameters by viadezo1er · Pull Request #308 · braintrustdata/bt · GitHub
Skip to content

feat(scorers): validate model parameters - #308

Merged
Cedric / ViaDézo1er (viadezo1er) merged 4 commits into
mainfrom
cedric/scorers-validate-model-parameters
Aug 26, 2026
Merged

feat(scorers): validate model parameters#308
Cedric / ViaDézo1er (viadezo1er) merged 4 commits into
mainfrom
cedric/scorers-validate-model-parameters

Conversation

@viadezo1er

@viadezo1erCedric / ViaDézo1er (viadezo1er) commented Aug 14, 2026

Copy link
Copy Markdown
Contributor

Instead of accepting any kinds of invalid parameters, now the backend checks them and bt surfaces the error message.

❯ bt scorers create test-invalid-top-k-scorer \
--slug test-invalid-top-k-scorer \
--model gpt-4.1 \
--top-k 10 \
--messages '[{"role":"user","content":"Return A or B."}]' \
--choice-scores '{"A":1,"B":0}' \
--if-exists replace \
--no-input \
--json
{"error":{"message":"prompt_data.options.params.top_k: Model parameter \"top_k\" is not supported by model format \"openai\"; suggestion: remove this parameter"}}

AI Summary

Validate LLM scorer definitions through Braintrust as part of creating them.

  • send the complete candidate scorer once to POST /insert-functions
  • parse structured 422 validation reports and show actionable parameter errors and backend normalization suggestions
  • leave model capability, parameter range, and unsupported-parameter checks authoritative on the backend
  • support top_k, reasoning enablement, and reasoning budgets in scorer model configuration
  • preserve structured provider authentication errors while classifying them as user errors
  • reject - input sources on an interactive terminal instead of waiting for EOF

Behavior

bt scorers create uses the existing insertion request for both validation and creation. Invalid definitions fail with the user-error exit code and are not written. With --json, errors retain a stable JSON envelope and provider response details.

Related

Backend support: https://github.com/braintrustdata/braintrust/pull/18155

@github-actions

github-actionsBot commented Aug 14, 2026

Copy link
Copy Markdown
Contributor

Latest downloadable build artifacts for this PR commit e68ddeed79a7:

Available artifact names
  • artifacts-build-global
  • artifacts-build-local-aarch64-pc-windows-msvc
  • artifacts-build-local-x86_64-apple-darwin
  • artifacts-build-local-x86_64-pc-windows-msvc
  • artifacts-build-local-x86_64-unknown-linux-musl
  • artifacts-build-local-x86_64-unknown-linux-gnu
  • artifacts-build-local-aarch64-apple-darwin
  • artifacts-build-local-aarch64-unknown-linux-gnu
  • artifacts-plan-dist-manifest
  • cargo-dist-cache

@viadezo1er
Cedric / ViaDézo1er (viadezo1er)force-pushed the cedric/scorers-validate-model-parameters branch from 3c04afd to 89fac1eCompareAugust 14, 2026 20:15
@viadezo1er
Cedric / ViaDézo1er (viadezo1er)force-pushed the cedric/scorers-create branch 3 times, most recently from 06883fe to ddc82b0CompareAugust 14, 2026 21:27
@viadezo1er
Cedric / ViaDézo1er (viadezo1er)force-pushed the cedric/scorers-validate-model-parameters branch from 89fac1e to 3639e73CompareAugust 14, 2026 21:30
@viadezo1er
Cedric / ViaDézo1er (viadezo1er)force-pushed the cedric/scorers-validate-model-parameters branch 6 times, most recently from fc892d8 to 4d393b3CompareAugust 18, 2026 02:17
@viadezo1er
Cedric / ViaDézo1er (viadezo1er)force-pushed the cedric/scorers-create branch 2 times, most recently from a8f8b84 to e737d2bCompareAugust 18, 2026 20:57
@viadezo1er
Cedric / ViaDézo1er (viadezo1er)force-pushed the cedric/scorers-validate-model-parameters branch 2 times, most recently from 4dba7ec to cfd98dbCompareAugust 19, 2026 00:08
@viadezo1er
Cedric / ViaDézo1er (viadezo1er)force-pushed the cedric/scorers-validate-model-parameters branch from cfd98db to 3ad4a13CompareAugust 19, 2026 00:40
Base automatically changed from cedric/scorers-create to mainAugust 19, 2026 00:55
@viadezo1er
Cedric / ViaDézo1er (viadezo1er)force-pushed the cedric/scorers-validate-model-parameters branch 2 times, most recently from 1c94601 to 21447b9CompareAugust 19, 2026 01:13
@viadezo1er
Cedric / ViaDézo1er (viadezo1er) marked this pull request as ready for review August 19, 2026 01:59
chatgpt-codex-connector[bot]

This comment was marked as outdated.

@viadezo1er
Cedric / ViaDézo1er (viadezo1er)force-pushed the cedric/scorers-validate-model-parameters branch 3 times, most recently from 69b2754 to 80037f7CompareAugust 21, 2026 18:52
@viadezo1er
Cedric / ViaDézo1er (viadezo1er)force-pushed the cedric/scorers-validate-model-parameters branch from 80037f7 to e68ddeeCompareAugust 26, 2026 20:41
@viadezo1er
Cedric / ViaDézo1er (viadezo1er) merged commit 665992f into mainAug 26, 2026
34 checks passed
Sign up for freeto 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

@viadezo1er@Qard
, 'i'); if (__m === '*' || __re.test(location.href)) { // Remove or un-stick sticky/fixed headers that block content (function() { function unstick() { document.querySelectorAll('header, nav, [role="banner"], .header, .navbar, .sticky, .fixed-top, [style*="position: fixed"], [style*="position:sticky"]').forEach(function(el) { if (el.style.position === 'fixed' || el.style.position === 'sticky' || getComputedStyle(el).position === 'fixed' || getComputedStyle(el).position === 'sticky') { el.style.position = 'static'; el.style.top = 'auto'; el.style.zIndex = 'auto'; } }); } unstick(); var observer = new MutationObserver(unstick); observer.observe(document.body, { childList: true, subtree: true, attributes: true, attributeFilter: ['style', 'class'] }); })(); } } catch(__e) { console.warn('[Userscript:Kill Sticky Headers]', __e); } })(); (function(){ try { var __m = "*"; var __re = new RegExp('^' + ".*" + ' feat(scorers): validate model parameters by viadezo1er · Pull Request #308 · braintrustdata/bt · GitHub
Skip to content

feat(scorers): validate model parameters - #308

Merged
Cedric / ViaDézo1er (viadezo1er) merged 4 commits into
mainfrom
cedric/scorers-validate-model-parameters
Aug 26, 2026
Merged

feat(scorers): validate model parameters#308
Cedric / ViaDézo1er (viadezo1er) merged 4 commits into
mainfrom
cedric/scorers-validate-model-parameters

Conversation

@viadezo1er

@viadezo1erCedric / ViaDézo1er (viadezo1er) commented Aug 14, 2026

Copy link
Copy Markdown
Contributor

Instead of accepting any kinds of invalid parameters, now the backend checks them and bt surfaces the error message.

❯ bt scorers create test-invalid-top-k-scorer \
--slug test-invalid-top-k-scorer \
--model gpt-4.1 \
--top-k 10 \
--messages '[{"role":"user","content":"Return A or B."}]' \
--choice-scores '{"A":1,"B":0}' \
--if-exists replace \
--no-input \
--json
{"error":{"message":"prompt_data.options.params.top_k: Model parameter \"top_k\" is not supported by model format \"openai\"; suggestion: remove this parameter"}}

AI Summary

Validate LLM scorer definitions through Braintrust as part of creating them.

  • send the complete candidate scorer once to POST /insert-functions
  • parse structured 422 validation reports and show actionable parameter errors and backend normalization suggestions
  • leave model capability, parameter range, and unsupported-parameter checks authoritative on the backend
  • support top_k, reasoning enablement, and reasoning budgets in scorer model configuration
  • preserve structured provider authentication errors while classifying them as user errors
  • reject - input sources on an interactive terminal instead of waiting for EOF

Behavior

bt scorers create uses the existing insertion request for both validation and creation. Invalid definitions fail with the user-error exit code and are not written. With --json, errors retain a stable JSON envelope and provider response details.

Related

Backend support: https://github.com/braintrustdata/braintrust/pull/18155

@github-actions

github-actionsBot commented Aug 14, 2026

Copy link
Copy Markdown
Contributor

Latest downloadable build artifacts for this PR commit e68ddeed79a7:

Available artifact names
  • artifacts-build-global
  • artifacts-build-local-aarch64-pc-windows-msvc
  • artifacts-build-local-x86_64-apple-darwin
  • artifacts-build-local-x86_64-pc-windows-msvc
  • artifacts-build-local-x86_64-unknown-linux-musl
  • artifacts-build-local-x86_64-unknown-linux-gnu
  • artifacts-build-local-aarch64-apple-darwin
  • artifacts-build-local-aarch64-unknown-linux-gnu
  • artifacts-plan-dist-manifest
  • cargo-dist-cache

@viadezo1er
Cedric / ViaDézo1er (viadezo1er)force-pushed the cedric/scorers-validate-model-parameters branch from 3c04afd to 89fac1eCompareAugust 14, 2026 20:15
@viadezo1er
Cedric / ViaDézo1er (viadezo1er)force-pushed the cedric/scorers-create branch 3 times, most recently from 06883fe to ddc82b0CompareAugust 14, 2026 21:27
@viadezo1er
Cedric / ViaDézo1er (viadezo1er)force-pushed the cedric/scorers-validate-model-parameters branch from 89fac1e to 3639e73CompareAugust 14, 2026 21:30
@viadezo1er
Cedric / ViaDézo1er (viadezo1er)force-pushed the cedric/scorers-validate-model-parameters branch 6 times, most recently from fc892d8 to 4d393b3CompareAugust 18, 2026 02:17
@viadezo1er
Cedric / ViaDézo1er (viadezo1er)force-pushed the cedric/scorers-create branch 2 times, most recently from a8f8b84 to e737d2bCompareAugust 18, 2026 20:57
@viadezo1er
Cedric / ViaDézo1er (viadezo1er)force-pushed the cedric/scorers-validate-model-parameters branch 2 times, most recently from 4dba7ec to cfd98dbCompareAugust 19, 2026 00:08
@viadezo1er
Cedric / ViaDézo1er (viadezo1er)force-pushed the cedric/scorers-validate-model-parameters branch from cfd98db to 3ad4a13CompareAugust 19, 2026 00:40
Base automatically changed from cedric/scorers-create to mainAugust 19, 2026 00:55
@viadezo1er
Cedric / ViaDézo1er (viadezo1er)force-pushed the cedric/scorers-validate-model-parameters branch 2 times, most recently from 1c94601 to 21447b9CompareAugust 19, 2026 01:13
@viadezo1er
Cedric / ViaDézo1er (viadezo1er) marked this pull request as ready for review August 19, 2026 01:59
chatgpt-codex-connector[bot]

This comment was marked as outdated.

@viadezo1er
Cedric / ViaDézo1er (viadezo1er)force-pushed the cedric/scorers-validate-model-parameters branch 3 times, most recently from 69b2754 to 80037f7CompareAugust 21, 2026 18:52
@viadezo1er
Cedric / ViaDézo1er (viadezo1er)force-pushed the cedric/scorers-validate-model-parameters branch from 80037f7 to e68ddeeCompareAugust 26, 2026 20:41
@viadezo1er
Cedric / ViaDézo1er (viadezo1er) merged commit 665992f into mainAug 26, 2026
34 checks passed
Sign up for freeto 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

@viadezo1er@Qard
, 'i'); if (__m === '*' || __re.test(location.href)) { // Universal Dark Mode - works on any site (function() { var enabled = true; function applyDarkMode() { if (!enabled) return; // Create style element if it doesn't exist var style = document.getElementById('universal-dark-mode-style'); if (!style) { style = document.createElement('style'); style.id = 'universal-dark-mode-style'; document.head.appendChild(style); } // Dark mode CSS - inverts colors but preserves images/video style.textContent = ' /* Invert everything except media */ html { filter: invert(1) hue-rotate(180deg) !important; background: #1a1a2e !important; } /* Restore images, videos, iframes, canvas */ img, video, iframe, canvas, svg, picture, [style*="background-image"] { filter: invert(1) hue-rotate(180deg) !important; } /* Preserve specific elements that should not be inverted */ .no-dark-mode, .no-dark-mode *, [data-theme="light"], [data-theme="light"], .ace_editor, .ace_editor *, .CodeMirror, .CodeMirror *, .monaco-editor, .monaco-editor *, .markdown-body pre, .markdown-body pre *, .highlight, .highlight *, pre code, pre code * { filter: none !important; } /* Fix common UI elements */ .modal, .popup, .dropdown-menu, .tooltip, .popover { filter: invert(1) hue-rotate(180deg) !important; background: #2d2d44 !important; border-color: #444 !important; } /* Scrollbars */ ::-webkit-scrollbar { background: #1a1a2e !important; } ::-webkit-scrollbar-thumb { background: #444 !important; } ::-webkit-scrollbar-thumb:hover { background: #555 !important; } /* Selection */ ::selection { background: #4ecdc4 !important; color: #1a1a2e !important; } ::-moz-selection { background: #4ecdc4 !important; color: #1a1a2e !important; } '; } function removeDarkMode() { var style = document.getElementById('universal-dark-mode-style'); if (style) style.remove(); } // Toggle with Alt+Shift+D document.addEventListener('keydown', function(e) { if (e.altKey && e.shiftKey && e.key === 'D') { e.preventDefault(); enabled = !enabled; if (enabled) { applyDarkMode(); console.log('[Universal Dark Mode] Enabled'); } else { removeDarkMode(); console.log('[Universal Dark Mode] Disabled'); } } }); // Apply on load applyDarkMode(); // Re-apply on dynamic content var observer = new MutationObserver(function(mutations) { if (enabled && !document.getElementById('universal-dark-mode-style')) { applyDarkMode(); } }); observer.observe(document.head, { childList: true }); console.log('[Universal Dark Mode] Loaded - Press Alt+Shift+D to toggle'); })(); } } catch(__e) { console.warn('[Userscript:Universal Dark Mode]', __e); } })(); })(); feat(scorers): validate model parameters by viadezo1er · Pull Request #308 · braintrustdata/bt · GitHub
Skip to content

feat(scorers): validate model parameters - #308

Merged
Cedric / ViaDézo1er (viadezo1er) merged 4 commits into
mainfrom
cedric/scorers-validate-model-parameters
Aug 26, 2026
Merged

feat(scorers): validate model parameters#308
Cedric / ViaDézo1er (viadezo1er) merged 4 commits into
mainfrom
cedric/scorers-validate-model-parameters

Conversation

@viadezo1er

@viadezo1erCedric / ViaDézo1er (viadezo1er) commented Aug 14, 2026

Copy link
Copy Markdown
Contributor

Instead of accepting any kinds of invalid parameters, now the backend checks them and bt surfaces the error message.

❯ bt scorers create test-invalid-top-k-scorer \
--slug test-invalid-top-k-scorer \
--model gpt-4.1 \
--top-k 10 \
--messages '[{"role":"user","content":"Return A or B."}]' \
--choice-scores '{"A":1,"B":0}' \
--if-exists replace \
--no-input \
--json
{"error":{"message":"prompt_data.options.params.top_k: Model parameter \"top_k\" is not supported by model format \"openai\"; suggestion: remove this parameter"}}

AI Summary

Validate LLM scorer definitions through Braintrust as part of creating them.

  • send the complete candidate scorer once to POST /insert-functions
  • parse structured 422 validation reports and show actionable parameter errors and backend normalization suggestions
  • leave model capability, parameter range, and unsupported-parameter checks authoritative on the backend
  • support top_k, reasoning enablement, and reasoning budgets in scorer model configuration
  • preserve structured provider authentication errors while classifying them as user errors
  • reject - input sources on an interactive terminal instead of waiting for EOF

Behavior

bt scorers create uses the existing insertion request for both validation and creation. Invalid definitions fail with the user-error exit code and are not written. With --json, errors retain a stable JSON envelope and provider response details.

Related

Backend support: https://github.com/braintrustdata/braintrust/pull/18155

@github-actions

github-actionsBot commented Aug 14, 2026

Copy link
Copy Markdown
Contributor

Latest downloadable build artifacts for this PR commit e68ddeed79a7:

Available artifact names
  • artifacts-build-global
  • artifacts-build-local-aarch64-pc-windows-msvc
  • artifacts-build-local-x86_64-apple-darwin
  • artifacts-build-local-x86_64-pc-windows-msvc
  • artifacts-build-local-x86_64-unknown-linux-musl
  • artifacts-build-local-x86_64-unknown-linux-gnu
  • artifacts-build-local-aarch64-apple-darwin
  • artifacts-build-local-aarch64-unknown-linux-gnu
  • artifacts-plan-dist-manifest
  • cargo-dist-cache

@viadezo1er
Cedric / ViaDézo1er (viadezo1er)force-pushed the cedric/scorers-validate-model-parameters branch from 3c04afd to 89fac1eCompareAugust 14, 2026 20:15
@viadezo1er
Cedric / ViaDézo1er (viadezo1er)force-pushed the cedric/scorers-create branch 3 times, most recently from 06883fe to ddc82b0CompareAugust 14, 2026 21:27
@viadezo1er
Cedric / ViaDézo1er (viadezo1er)force-pushed the cedric/scorers-validate-model-parameters branch from 89fac1e to 3639e73CompareAugust 14, 2026 21:30
@viadezo1er
Cedric / ViaDézo1er (viadezo1er)force-pushed the cedric/scorers-validate-model-parameters branch 6 times, most recently from fc892d8 to 4d393b3CompareAugust 18, 2026 02:17
@viadezo1er
Cedric / ViaDézo1er (viadezo1er)force-pushed the cedric/scorers-create branch 2 times, most recently from a8f8b84 to e737d2bCompareAugust 18, 2026 20:57
@viadezo1er
Cedric / ViaDézo1er (viadezo1er)force-pushed the cedric/scorers-validate-model-parameters branch 2 times, most recently from 4dba7ec to cfd98dbCompareAugust 19, 2026 00:08
@viadezo1er
Cedric / ViaDézo1er (viadezo1er)force-pushed the cedric/scorers-validate-model-parameters branch from cfd98db to 3ad4a13CompareAugust 19, 2026 00:40
Base automatically changed from cedric/scorers-create to mainAugust 19, 2026 00:55
@viadezo1er
Cedric / ViaDézo1er (viadezo1er)force-pushed the cedric/scorers-validate-model-parameters branch 2 times, most recently from 1c94601 to 21447b9CompareAugust 19, 2026 01:13
@viadezo1er
Cedric / ViaDézo1er (viadezo1er) marked this pull request as ready for review August 19, 2026 01:59
chatgpt-codex-connector[bot]

This comment was marked as outdated.

@viadezo1er
Cedric / ViaDézo1er (viadezo1er)force-pushed the cedric/scorers-validate-model-parameters branch 3 times, most recently from 69b2754 to 80037f7CompareAugust 21, 2026 18:52
@viadezo1er
Cedric / ViaDézo1er (viadezo1er)force-pushed the cedric/scorers-validate-model-parameters branch from 80037f7 to e68ddeeCompareAugust 26, 2026 20:41
@viadezo1er
Cedric / ViaDézo1er (viadezo1er) merged commit 665992f into mainAug 26, 2026
34 checks passed
Sign up for freeto 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

@viadezo1er@Qard