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

Added support for deepseek LLMs #175

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

Riddhimaan-Senapati
Copy link

@Riddhimaan-Senapati Riddhimaan-Senapati commented Jan 8, 2025

I have added code to support the deepseek suite of LLMs along with testing code. I used the current implementations of the openAI providers and Groq providers as inspiration.

This partially solves issue #140

@Riddhimaan-Senapati
Copy link
Author

@ksolo @foxty @jeffxtang @joaomdmoura Can anyone of you review my PR please?

@ksolo ksolo self-assigned this Jan 9, 2025
@ksolo
Copy link
Collaborator

ksolo commented Jan 9, 2025

@Riddhimaan-Senapati I will take a look at this later today.

@ksolo
Copy link
Collaborator

ksolo commented Jan 10, 2025

@Riddhimaan-Senapati can you add a guide similar to the other providers in the guides directory - https://github.com/andrewyng/aisuite/tree/main/guides

@Riddhimaan-Senapati
Copy link
Author

@Riddhimaan-Senapati can you add a guide similar to the other providers in the guides directory - https://github.com/andrewyng/aisuite/tree/main/guides

Done. I have added a DeepSeek guide in the guides directory.

@ksolo
Copy link
Collaborator

ksolo commented Jan 10, 2025

I'll do a manual test of this tomorrow and provided that goes well, we'll get this merged. Thank you!

@JiaLiangC
Copy link

DeepSeek v3 is really both affordable and user-friendly.

@Riddhimaan-Senapati
Copy link
Author

@ksolo Were you able to test it?

@ksolo
Copy link
Collaborator

ksolo commented Jan 17, 2025

@Riddhimaan-Senapati Sorry I've been pulled into a few things lately that had to take priority. I can get to this over the weekend.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants