feat: add DeepSeek v3.2 model support#1564
Conversation
Closes OpenHands#1298 - Added VERIFIED_DEEPSEEK_MODELS list with deepseek-chat, deepseek-chat-v3.2, and deepseek-reasoner - Added deepseek-chat-v3.2 to VERIFIED_OPENHANDS_MODELS - Fixed typo: deekseek-chat -> deepseek-chat - Added deepseek to VERIFIED_MODELS dict
There was a problem hiding this comment.
Thank you, we do want to support 3.2, but I thought we already do TBH. I mean, I'm not sure we need additions since it's served via .../deepseek-chat and we already support that?
Edited to add: could you please tell, what happens if you try to use it?
I don't have a DeepSeek API key to verify end-to-end, but the typo fix (deekseek-chat → deepseek-chat) is objectively correct regardless of runtime behavior. Happy to slim down the PR to just that fix if you think the structural additions aren't needed. Or close it entirely if you prefer - your call! |
|
Just to note, you don't need to worry about merging main, we don't require updated PRs to merge. It's fine, as long as it doesn't have conflicts |
Simplified PR per enyst's feedback - only fixing the typo
|
@enyst Done! Simplified the PR to only fix the typo as you suggested. |
| @@ -1,5 +1,6 @@ | |||
| VERIFIED_OPENAI_MODELS = [ | |||
| "gpt-5.2", | |||
| "gpt-5.2-codex", | |||
There was a problem hiding this comment.
| "gpt-5.2-codex", |
I think this came back from some conflict with a slightly older branch, because we just removed this these days
There was a problem hiding this comment.
@enyst Applied the suggestion - removed the gpt-5.2-codex line.
Thanks!
Summary
Adds support for DeepSeek v3.2 model as requested in #1298.
Changes
VERIFIED_DEEPSEEK_MODELSlist with:deepseek-chatdeepseek-chat-v3.2deepseek-reasonerdeepseek-chat-v3.2toVERIFIED_OPENHANDS_MODELSdeekseek-chat→deepseek-chatdeepseektoVERIFIED_MODELSdictReferences
Closes #1298