Skip to content

feat: Add DeepSeek sample#861

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

feat: Add DeepSeek sample#861
erikvilima wants to merge 2 commits intomasterfrom
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.

Copy link
Copy Markdown
Contributor

@agneszitte agneszitte left a comment

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