Skip to content

Update RAG tutorial to match actual repository code samples #143

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

Closed
wants to merge 2 commits into from

Conversation

Copilot
Copy link
Contributor

@Copilot Copilot AI commented May 24, 2025

This PR addresses the mismatch between the RAG tutorial documentation and the actual code samples in the repository.

Changes Made

  • Added a clear note at the beginning of the RAG tutorial explaining that while the tutorial demonstrates concepts using GitHub Models (with the https://models.inference.ai.azure.com endpoint), all the actual code samples in the repository use Ollama with local models (connecting to http://localhost:11434)
  • Updated the code examples to show both approaches - the GitHub Models approach (as was already in the tutorial) and the Ollama approach (as is in the actual code samples)
  • Added notes about how to run the code samples, including the requirement to have Ollama set up and the correct models pulled
  • Fixed a typo in the tutorial: "instatiated" -> "instantiated"
  • Added detailed notes explaining that the actual code samples use different approaches than the simplified examples in the tutorial

These changes help users understand the differences between the tutorial examples and the actual code samples, and provide clear guidance on how to run the samples correctly.

Fixes #122.


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Co-authored-by: elbruno <3533489+elbruno@users.noreply.github.com>
@Copilot Copilot AI changed the title [WIP] Code in the Core Techniques RAG tutorial does not match repo contents Update RAG tutorial to match actual repository code samples May 24, 2025
@Copilot Copilot AI requested a review from elbruno May 24, 2025 20:34
Copilot finished work on behalf of elbruno May 24, 2025 20:34
@elbruno elbruno closed this May 30, 2025
@elbruno elbruno deleted the copilot/fix-122 branch May 30, 2025 17:26
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.

Code in the Core Techniques RAG tutorial does not match repo contents
2 participants