Skip to content

feat: Add DeepSeek API to modelProviders#1636

Closed
SYMBaiEX wants to merge 5 commits into
elizaOS:developfrom
SYMBaiEX:main
Closed

feat: Add DeepSeek API to modelProviders#1636
SYMBaiEX wants to merge 5 commits into
elizaOS:developfrom
SYMBaiEX:main

Conversation

@SYMBaiEX

@SYMBaiEX SYMBaiEX commented Jan 1, 2025

Copy link
Copy Markdown
Member

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

image

Where should a reviewer start?

Follow the test steps below (should be really easy!)

Detailed testing steps

Added Support for DeepSeek V3 using `deepseek-chat` for the model, set defaults, included support for the DeepSeek API

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown
Contributor

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!

@mcp97 mcp97 left a comment

Copy link
Copy Markdown
Contributor

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

Comment thread packages/core/src/models.ts
mcp97
mcp97 previously approved these changes Jan 1, 2025

@mcp97 mcp97 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@mcp97 mcp97 changed the base branch from main to develop January 1, 2025 17:07
@mcp97 mcp97 dismissed their stale review January 1, 2025 17:07

The base branch was changed.

mcp97
mcp97 previously approved these changes Jan 1, 2025
@mcp97

mcp97 commented Jan 1, 2025

Copy link
Copy Markdown
Contributor

Please run pnpm i -r -no-frozen-lockfile and resubmit please thank you (unrelated change to fix)

pnpm i -r -no-frozen-lockfile
@SYMBaiEX

SYMBaiEX commented Jan 1, 2025

Copy link
Copy Markdown
Member Author

Please run pnpm i -r -no-frozen-lockfile and resubmit please thank you (unrelated change to fix)

Done

mcp97
mcp97 previously approved these changes Jan 1, 2025
@mcp97 mcp97 changed the base branch from develop to main January 1, 2025 18:10
@mcp97 mcp97 dismissed their stale review January 1, 2025 18:10

The base branch was changed.

@mcp97 mcp97 changed the base branch from main to develop January 1, 2025 18:19
@odilitime odilitime added the ModelProvider_new New model provider plugin PR label Jan 1, 2025
@btspoony

btspoony commented Jan 5, 2025

Copy link
Copy Markdown
Contributor

@SYMBaiEX can you resolve the conflicts?

@SYMBaiEX

SYMBaiEX commented Jan 5, 2025

Copy link
Copy Markdown
Member Author

I'll take a look when I get back today, no problem

@mameikagou

Copy link
Copy Markdown
Contributor

I really need this , please

@AIFlowML

AIFlowML commented Jan 9, 2025 via email

Copy link
Copy Markdown
Contributor

@wtfsayo

wtfsayo commented Jan 11, 2025

Copy link
Copy Markdown
Contributor

closing in favor of #2067

@wtfsayo wtfsayo closed this Jan 11, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ModelProvider_new New model provider plugin PR needs_documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants