Skip to content

Conversation

@lennart02
Copy link

Since this is a pure text application, a vision-capable model is unnecessary.

@github-actions
Copy link

👋 Thanks for contributing @lennart02! We will review the pull request and get back to you soon.

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.

Pull Request Overview

This PR updates the application to use the text-only llama3.2 model rather than the vision-capable version, and adjusts the API endpoint accordingly.

  • Changed the model identifier from "llama3.2-vision" to "llama3.2".
  • Updated the API endpoint URL to include the "/v1" suffix.
Comments suppressed due to low confidence (2)

03-CoreGenerativeAITechniques/src/RAGSimple-01SK/Program.cs:25

  • Ensure that the updated modelId 'llama3.2' aligns with all dependent API calls and internal logic for text-only operations.
var modelId = "llama3.2";

03-CoreGenerativeAITechniques/src/RAGSimple-01SK/Program.cs:31

  • Verify that the updated endpoint URL with the '/v1' suffix is intended and that all API interactions properly accommodate this change.
endpoint: new Uri("http://localhost:11434/v1"),

@elbruno elbruno merged commit bc20a6e into microsoft:main May 6, 2025
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants