Skip to content

Commit 8b65fa2

Browse files
fix(opencode): remove Qwen sampling defaults (anomalyco#43310)
Co-authored-by: rekram1-node <63023139+rekram1-node@users.noreply.github.com>
1 parent 0033bb3 commit 8b65fa2

2 files changed

Lines changed: 13 additions & 2 deletions

File tree

packages/opencode/src/provider/transform.ts

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -528,7 +528,6 @@ const GEMINI_MODELS_WITH_SAMPLING_DEFAULTS = [
528528
export function temperature(model: Provider.Model) {
529529
const id = model.api.id.toLowerCase()
530530
if (id.includes("north-mini-code")) return 1.0
531-
if (id.includes("qwen")) return 0.55
532531
if (id.includes("claude")) return undefined
533532
if (id.includes("gemini"))
534533
return GEMINI_MODELS_WITH_SAMPLING_DEFAULTS.some((model) => model.test(id)) ? 1.0 : undefined
@@ -547,7 +546,6 @@ export function temperature(model: Provider.Model) {
547546

548547
export function topP(model: Provider.Model) {
549548
const id = model.api.id.toLowerCase()
550-
if (id.includes("qwen")) return 1
551549
if (id.includes("gemini"))
552550
return GEMINI_MODELS_WITH_SAMPLING_DEFAULTS.some((model) => model.test(id)) ? 0.95 : undefined
553551
if (["minimax-m2", "kimi-k2.5", "kimi-k2p5", "kimi-k2-5"].some((s) => id.includes(s))) {

packages/opencode/test/provider/transform.test.ts

Lines changed: 13 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -3215,6 +3215,19 @@ describe("ProviderTransform.temperature - Cohere North", () => {
32153215
})
32163216
})
32173217

3218+
describe("ProviderTransform sampling defaults - Qwen", () => {
3219+
test.each(["Qwen3.8-27B", "qwen3-coder-30b-a3b-instruct"])('leaves sampling unset for "%s"', (id) => {
3220+
const model = {
3221+
id: `custom/${id}`,
3222+
api: { id },
3223+
} as any
3224+
3225+
expect(ProviderTransform.temperature(model)).toBeUndefined()
3226+
expect(ProviderTransform.topP(model)).toBeUndefined()
3227+
expect(ProviderTransform.topK(model)).toBeUndefined()
3228+
})
3229+
})
3230+
32183231
describe("ProviderTransform sampling defaults - Gemini", () => {
32193232
const model = (id: string) =>
32203233
({

0 commit comments

Comments
 (0)