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

Python: Add Deepseek service to concept samples #10306

Conversation

TaoChenOSU
Copy link
Contributor

@TaoChenOSU TaoChenOSU commented Jan 27, 2025

Motivation and Context

Models from DeepSeek are rising fast as one of the most capable and cost-effective open-source models. The community will be eager to test these models out.

Description

It's not hard to use these new models as they are compatible with the OpenAI API. This PR simply adds an option where people can test out the DeepSeek models in the chat concept samples.

Contribution Checklist

@TaoChenOSU TaoChenOSU added the python Pull requests for the Python Semantic Kernel label Jan 27, 2025
@TaoChenOSU TaoChenOSU self-assigned this Jan 27, 2025
@TaoChenOSU TaoChenOSU requested a review from a team as a code owner January 27, 2025 18:20
@github-actions github-actions bot changed the title Add Deepseek service to concept samples Python: Add Deepseek service to concept samples Jan 27, 2025
@TaoChenOSU TaoChenOSU enabled auto-merge January 27, 2025 19:00
Copy link
Contributor

@moonbox3 moonbox3 left a comment

Choose a reason for hiding this comment

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

I imagine devs would want to use their reasoning model, as well? This model produces reasoning_content that we don't support:

reasoning_content = response.choices[0].message.reasoning_content

https://api-docs.deepseek.com/guides/reasoning_model

@TaoChenOSU
Copy link
Contributor Author

I imagine devs would want to use their reasoning model, as well? This model produces reasoning_content that we don't support:

reasoning_content = response.choices[0].message.reasoning_content

https://api-docs.deepseek.com/guides/reasoning_model

We still expose it in the inner content of the chat message.

@TaoChenOSU TaoChenOSU added this pull request to the merge queue Jan 27, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Jan 27, 2025
@eavanvalkenburg eavanvalkenburg added this pull request to the merge queue Jan 28, 2025
Merged via the queue into microsoft:main with commit f005058 Jan 28, 2025
24 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
python Pull requests for the Python Semantic Kernel
Projects
Status: Sprint: Done
Development

Successfully merging this pull request may close these issues.

4 participants