Skip to content
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

Python: Add Azure AI Agent AI Search Sample. Update memory sample with README. #10497

Merged
merged 9 commits into from
Feb 12, 2025

Conversation

moonbox3
Copy link
Contributor

@moonbox3 moonbox3 commented Feb 12, 2025

Motivation and Context

To showcase other ways to interact with Azure AI Agents, we're adding a sample that leverages the Azure AI Search tool.

Description

Adds the Azure AI Agent with AI Search sample.

  • Includes README and hopefully enough information to get started. Assumes the user is able to set up the hotels-sample-index with the proper data model fields (outlined in SK).
  • Updates the memory samples with a README to help make the usage more clear
  • Update the default termination strategy max iterations from 1 to 5.

Contribution Checklist

@moonbox3 moonbox3 self-assigned this Feb 12, 2025
@moonbox3 moonbox3 requested a review from a team as a code owner February 12, 2025 02:48
@markwallace-microsoft markwallace-microsoft added python Pull requests for the Python Semantic Kernel documentation labels Feb 12, 2025
@markwallace-microsoft
Copy link
Member

markwallace-microsoft commented Feb 12, 2025

Python Test Coverage

Python Test Coverage Report •
FileStmtsMissCoverMissing
semantic_kernel/agents/azure_ai
   azure_ai_agent.py811088%88, 90, 96, 105–110, 112, 114, 157, 217
semantic_kernel/agents/group_chat
   agent_group_chat.py1141289%73, 163, 213, 217–227
TOTAL18533217088% 

Python Unit Test Overview

Tests Skipped Failures Errors Time
3174 4 💤 0 ❌ 0 🔥 1m 36s ⏱️

@eavanvalkenburg eavanvalkenburg added this pull request to the merge queue Feb 12, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Feb 12, 2025
@eavanvalkenburg eavanvalkenburg added this pull request to the merge queue Feb 12, 2025
Merged via the queue into microsoft:main with commit 56bd394 Feb 12, 2025
27 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation python Pull requests for the Python Semantic Kernel
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

4 participants