-
Notifications
You must be signed in to change notification settings - Fork 5k
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.
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
feat: Add DeepSeek API to modelProviders #1636
Conversation
Added Support for DeepSeek V3 using `deepseek-chat` for the model, set defaults, included support for the DeepSeek API
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi @SYMBaiEX! Welcome to the ai16z community. Thanks for submitting your first pull request; your efforts are helping us accelerate towards AGI. We'll review it shortly. You are now a ai16z contributor!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for doing this - great work! Just one question should the models all be the same and shouldn't there be an embedding / image model default as well
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Please run pnpm i -r -no-frozen-lockfile and resubmit please thank you (unrelated change to fix) |
pnpm i -r -no-frozen-lockfile
Done |
@SYMBaiEX can you resolve the conflicts? |
I'll take a look when I get back today, no problem |
I really need this , please |
Check develop
I think is already there
…On Thu, Jan 9, 2025 at 20:46, mrlonely ***@***.***(mailto:On Thu, Jan 9, 2025 at 20:46, mrlonely <<a href=)> wrote:
I really need this , please
—
Reply to this email directly, [view it on GitHub](#1636 (comment)), or [unsubscribe](https://github.com/notifications/unsubscribe-auth/BE5QO4CNIAPCDPIEFAOJN6L2JZ4U7AVCNFSM6AAAAABUOXONPWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKOBQGIYTMNJSGU).
You are receiving this because you are subscribed to this thread.Message ID: ***@***.***>
|
closing in favor of #2067 |
Relates to:
No Tickets.
Adding DeepSeek as a new ModelProvider for ElizaOS.
Risks
new model added
Background
What’s new in DeepSeek V3?
671B MoE parameters
37B activated parameters
Trained on 14.8T high-quality tokens
What does this PR do?
Adds support for DeepSeek API so ElizaOS can use the DeepSeek V3 model directly.
What kind of change is this?
I personally wanted to try out the DeepSeek V3 model using the DeepSeek API. I see DeepSeek_AI doing a lot of updates and I believe having accessibility long term to their API could be useful.
Documentation changes needed?
Testing
Where should a reviewer start?
Follow the test steps below (should be really easy!)
Detailed testing steps