Skip to content

Commit 9769445

Browse files
simbasimba
authored andcommitted
fix: use max_tokens instead of max_completion_tokens in benchmark llmCall
1 parent 20cea81 commit 9769445

1 file changed

Lines changed: 1 addition & 2 deletions

File tree

skills/analysis/home-security-benchmark/scripts/run-benchmark.cjs

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -187,7 +187,7 @@ async function llmCall(messages, opts = {}) {
187187
stream: true,
188188
...(model && { model }),
189189
...(opts.temperature !== undefined && { temperature: opts.temperature }),
190-
...(opts.maxTokens && { max_completion_tokens: opts.maxTokens }),
190+
...(opts.maxTokens && { max_tokens: opts.maxTokens }),
191191
// Qwen3.5 non-thinking mode recommended params
192192
...(opts.expectJSON && opts.temperature === undefined && { temperature: 0.7 }),
193193
...(opts.expectJSON && { top_p: 0.8, presence_penalty: 1.5 }),
@@ -1942,7 +1942,6 @@ async function main() {
19421942
const ping = await llmClient.chat.completions.create({
19431943
...(LLM_MODEL && { model: LLM_MODEL }),
19441944
messages: [{ role: 'user', content: 'ping' }],
1945-
max_completion_tokens: 5,
19461945
});
19471946
results.model.name = ping.model || 'unknown';
19481947
log(` Model: ${results.model.name}`);

0 commit comments

Comments
 (0)