Skip to content
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

Closed
wants to merge 5 commits into from

Conversation

SYMBaiEX
Copy link

@SYMBaiEX SYMBaiEX commented Jan 1, 2025

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
Copy link
Contributor

@github-actions github-actions bot left a 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!

Copy link
Collaborator

@monilpat monilpat left a 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

monilpat
monilpat previously approved these changes Jan 1, 2025
Copy link
Collaborator

@monilpat monilpat left a comment

Choose a reason for hiding this comment

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

LGTM

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

The base branch was changed.

monilpat
monilpat previously approved these changes Jan 1, 2025
@monilpat
Copy link
Collaborator

monilpat commented Jan 1, 2025

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

pnpm i -r -no-frozen-lockfile
@SYMBaiEX
Copy link
Author

SYMBaiEX commented Jan 1, 2025

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

Done

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

The base branch was changed.

@monilpat monilpat 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
Copy link
Contributor

btspoony commented Jan 5, 2025

@SYMBaiEX can you resolve the conflicts?

@SYMBaiEX
Copy link
Author

SYMBaiEX commented Jan 5, 2025

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

@mameikagou
Copy link
Contributor

I really need this , please

@AIFlowML
Copy link
Collaborator

AIFlowML commented Jan 9, 2025 via email

@wtfsayo
Copy link
Member

wtfsayo commented Jan 11, 2025

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.

None yet

8 participants