docs: AI project, with framework #1505
Conversation
There was a problem hiding this comment.
Pull Request Overview
This PR introduces an AI framework documentation project that demonstrates how to use LangChain with GitHub Models. The purpose is to provide educational content and code examples for building AI applications using frameworks rather than direct API calls.
Key changes include:
- Complete documentation explaining AI framework benefits and usage patterns
- Python code examples demonstrating basic prompting, chat conversations, and tool calling
- Practical implementations showing integration with external APIs
Reviewed Changes
Copilot reviewed 4 out of 5 changed files in this pull request and generated 10 comments.
| File | Description |
|---|---|
| 10-ai-framework-project/README.md | Comprehensive documentation covering AI frameworks, LangChain usage, and detailed code examples |
| 10-ai-framework-project/code/python/app.py | Basic LangChain implementation for simple prompting |
| 10-ai-framework-project/code/python/app-chat.py | Chat conversation example with message history |
| 10-ai-framework-project/code/python/app-tools.py | Tool calling implementation with external API integration |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
10-ai-framework-project/README.md
Outdated
| ) | ||
|
|
||
| messages = [ | ||
| SystemMessage(content="You are Captain Picard of the Startship Enterprise"), |
There was a problem hiding this comment.
Corrected spelling of 'Startship' to 'Starship'.
Copilot uses AI. Check for mistakes.
Co-authored-by: Copilot <[email protected]>
Co-authored-by: Copilot <[email protected]>
Co-authored-by: Copilot <[email protected]>
Co-authored-by: Copilot <[email protected]>
Co-authored-by: Copilot <[email protected]>
Co-authored-by: Copilot <[email protected]>
Co-authored-by: Copilot <[email protected]>
Co-authored-by: Copilot <[email protected]>
Fix spelling Co-authored-by: Copilot <[email protected]>
There was a problem hiding this comment.
Pull Request Overview
Copilot reviewed 4 out of 5 changed files in this pull request and generated 5 comments.
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
|
|
||
|
|
||
| # works | ||
| response = llm.invoke(messages) |
There was a problem hiding this comment.
Extra space in variable assignment - should be single space between 'response' and '='.
Copilot uses AI. Check for mistakes.
| messages.append(response) | ||
| messages.append(HumanMessage(content="Now that I know about you, I'm Chris, can I be in your crew?")) | ||
|
|
||
| response = llm.invoke(messages) |
There was a problem hiding this comment.
Extra space in variable assignment - should be single space between 'response' and '='.
Copilot uses AI. Check for mistakes.
| ) | ||
|
|
||
| messages = [ | ||
| SystemMessage(content="You are Captain Picard of the Starship Enterprise"), |
There was a problem hiding this comment.
Inconsistent spelling - this instance correctly uses 'Starship' while line 89 uses 'Startship'.
Copilot uses AI. Check for mistakes.
Co-authored-by: Copilot <[email protected]>
Fix spelling Co-authored-by: Copilot <[email protected]>
2 participants
Description
Please include a summary of the changes and the related issue. Please also include relevant motivation and context.
Fixes # (issue)
Type of change