Skip to content

feat(opencode): Add Venice support in temperature, topP, topK and smallOption #13553

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
rekram1-node merged 3 commits into from
Feb 16, 2026
Merged

feat(opencode): Add Venice support in temperature, topP, topK and smallOption #13553

Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
2b02e2b
[provider] add Venice support in temperature, topP, topK and smallOpt…
dpuyosa Feb 14, 2026
62a0c18
Run prettier & fix inconsistency with model name
dpuyosa Feb 14, 2026
62c8917
Resolve minor review suggestions
dpuyosa Feb 14, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
13 changes: 9 additions & 4 deletions packages/opencode/src/provider/transform.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -297,8 +297,8 @@ export namespace ProviderTransform {
if (id.includes("glm-4.7")) return 1.0
if (id.includes("minimax-m2")) return 1.0
if (id.includes("kimi-k2")) {
// kimi-k2-thinking & kimi-k2.5 && kimi-k2p5
if (id.includes("thinking") || id.includes("k2.") || id.includes("k2p")) {
// kimi-k2-thinking & kimi-k2.5 && kimi-k2p5 && kimi-k2-5
if (["thinking", "k2.", "k2p", "k2-5"].some((s) => id.includes(s))) {
return 1.0
}
return 0.6
Expand All @@ -309,7 +309,7 @@ export namespace ProviderTransform {
export function topP(model: Provider.Model) {
const id = model.id.toLowerCase()
if (id.includes("qwen")) return 1
if (id.includes("minimax-m2") || id.includes("kimi-k2.5") || id.includes("kimi-k2p5") || id.includes("gemini")) {
if (["minimax-m2", "gemini", "kimi-k2.5", "kimi-k2p5", "kimi-k2-5"].some((s) => id.includes(s))) {
return 0.95
}
return undefined
Expand All @@ -318,7 +318,7 @@ export namespace ProviderTransform {
export function topK(model: Provider.Model) {
const id = model.id.toLowerCase()
if (id.includes("minimax-m2")) {
if (id.includes("m2.1")) return 40
if (["m2.", "m25", "m21"].some((s) => id.includes(s))) return 40
Copy link
Collaborator

@rekram1-node rekram1-node Feb 14, 2026

Choose a reason for hiding this comment

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

Isn't this a behavioral change for minimax m2.5?

Copy link
Contributor Author

@dpuyosa dpuyosa Feb 14, 2026

Choose a reason for hiding this comment

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

Unsloth recommends 40 as topK, same as m2.1
https://unsloth.ai/docs/models/minimax-2.5

Copy link
Collaborator

@rekram1-node rekram1-node Feb 16, 2026

Choose a reason for hiding this comment

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

Hm the minimax team themselves told me to do this:

FYI - recommended set up:
temperature 1
top_p: 0.95

Copy link
Collaborator

@rekram1-node rekram1-node Feb 16, 2026

Choose a reason for hiding this comment

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

Ig reading their docs they must've forgotten to tell me the top_k too lol, it's funny cause i literally followed what they told me at the time.

return 20
}
if (id.includes("gemini")) return 64
Expand Down Expand Up @@ -750,6 +750,11 @@ export namespace ProviderTransform {
}
return { reasoningEffort: "minimal" }
}

if (model.providerID === "venice") {
return { veniceParameters: { disableThinking: true } }
}

return {}
}

Expand Down
Loading