Skip to content

feat: Add DeepSeek sample#861

Draft
erikvilima wants to merge 2 commits into
masterfrom
dev/erli/deepseek-sample
Draft

feat: Add DeepSeek sample#861
erikvilima wants to merge 2 commits into
masterfrom
dev/erli/deepseek-sample

Conversation

@erikvilima

Copy link
Copy Markdown
Contributor

No description provided.

@erikvilima
erikvilima marked this pull request as draft January 29, 2025 19:16
/// <param name="Date">Gets the Date of the Forecast.</param>
/// <param name="TemperatureC">Gets the Forecast Temperature in Celsius.</param>
/// <param name="Summary">Get a description of how the weather will feel.</param>
public record WeatherForecast(DateOnly Date, double TemperatureC, string? Summary)

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Remove weather stuff

);
}

private static ToggleButton Header(MainViewModel vm)

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Remove if streaming's not used

Comment on lines +69 to +71
var section = context.Configuration.GetSection(nameof(AppConfig));
var apiKey = section[nameof(AppConfig.ApiKey)];
var baseUrl = section[nameof(AppConfig.BaseUrl)];

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

@agneszitte agneszitte left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

@eriklimakc make sure to also add proper documentation for the sample please:

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.

2 participants