Skip to content

Conversation

@elbruno
Copy link
Collaborator

@elbruno elbruno commented Feb 19, 2025

DETAILS
This pull request introduces a new project named ReasoningLabs-01-SK to the solution and includes initial implementation for using OpenAI's chat completion service. The most important changes include adding the new project to the solution, configuring the build settings, and implementing the initial code for the chat completion service.

Addition of new project:

  • Added ReasoningLabs-01-SK project to the solution file 03-CoreGenerativeAITechniques.sln [1] [2] [3].

Build configuration:

  • Created ReasoningLabs-01-SK.csproj with necessary configurations and package references.

Initial implementation:

  • Implemented initial code in Program.cs to use OpenAI's chat completion service with a sample prompt.
  • Updated CoreGenerativeAITechniques.sln to include new projects: "07 Reasoning" and "ReasoningLabs-01-SK".
  • Added ReasoningLabs-01-SK.csproj with .NET SDK and necessary package references.
  • Refactored Program.cs to use OpenAI API directly, commenting out Azure OpenAI model initialization.

Source Semantic Kernel Blog

Using OpenAI’s o3-mini Reasoning Model in Semantic Kernel

DETAILS

- Updated `CoreGenerativeAITechniques.sln` to include new projects: "07 Reasoning" and "ReasoningLabs-01-SK".
- Added `ReasoningLabs-01-SK.csproj` with .NET SDK and necessary package references.
- Refactored `Program.cs` to use OpenAI API directly, commenting out Azure OpenAI model initialization.
@elbruno elbruno requested a review from Copilot February 19, 2025 18:09
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot reviewed 1 out of 3 changed files in this pull request and generated 1 comment.

Files not reviewed (2)
  • 03-CoreGenerativeAITechniques/src/CoreGenerativeAITechniques.sln: Language not supported
  • 03-CoreGenerativeAITechniques/src/ReasoningLabs-01-SK/ReasoningLabs-01-SK.csproj: Language not supported

);

// Create a new chat history and add a user message to prompt the model.
ChatHistory chatHistory = [];
Copy link

Copilot AI Feb 19, 2025

Choose a reason for hiding this comment

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

The ChatHistory should be initialized as a list or a collection, not an empty array. Use new List<ChatMessage>() instead.

Suggested change
ChatHistory chatHistory = [];
ChatHistory chatHistory = new List<ChatMessage>();

Copilot uses AI. Check for mistakes.
@elbruno elbruno merged commit 6434b3f into main Feb 19, 2025
1 check passed
@elbruno elbruno deleted the bruno-reasoningmodel-sksample branch February 19, 2025 19:15
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