-
Notifications
You must be signed in to change notification settings - Fork 271
refactor: Agent framework v2 changes. #794
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
Closed
Changes from 57 commits
Commits
Show all changes
58 commits
Select commit
Hold shift + click to select a range
c687780
fix: update ODBC Driver references from version 17 to 18 across setup…
Pavan-Microsoft cf4219a
Agent framework v2 changes - Refactor agent factories and services: r…
Pavan-Microsoft bce02f4
fix: ensure valid chunk text before yielding response in ChatService
Pavan-Microsoft 18d24d0
Fix testcase - agentframework v2 - Refactor tests for ChatWithDataPlu…
Pavan-Microsoft 8b428fc
fix: comment out citation collection logic in ChatService stream proc…
Pavan-Microsoft 572f684
fix: refactor thread cache management in ChatService for improved iso…
Pavan-Microsoft 7a5223c
fix: update Azure credential retrieval to use client ID for improved …
Pavan-Microsoft c1649d1
add agent creation python script and update deployment documentation …
Ragini-Microsoft 94cf6e3
generated main.json
Ragini-Microsoft d908a93
Merge branch 'km-agentframework-v2' of https://github.com/microsoft/C…
Pavan-Microsoft 4868365
Merge branch 'dev' of https://github.com/microsoft/Conversation-Knowl…
Pavan-Microsoft 9363ddc
Merge branch 'dev' of https://github.com/microsoft/Conversation-Knowl…
Pavan-Microsoft f5e6fcf
fix: Update Azure CLI login method to use device code for local authe…
Pavan-Microsoft 0f29f2f
fix: Update VS Code Web instructions and add dependency installation …
Pavan-Microsoft 8ff3601
fix: Clarify deployment time estimate in Deployment Guide
Pavan-Microsoft 89eea3d
Merge branch 'dev' of https://github.com/microsoft/Conversation-Knowl…
Pavan-Microsoft 57ee604
Merge branch 'dev' of https://github.com/microsoft/Conversation-Knowl…
Pavan-Microsoft 81c068b
Merge branch 'dev' of https://github.com/microsoft/Conversation-Knowl…
Ragini-Microsoft cdcd727
fix to manage AI Foundry public network access during agent creation
Ragini-Microsoft 57cf67d
fix: update agent framework package names in requirements.txt
Pavan-Microsoft a550a0c
Merge branch 'dev' of https://github.com/microsoft/Conversation-Knowl…
Pavan-Microsoft 167e1a7
Merge branch 'dev' of https://github.com/microsoft/Conversation-Knowl…
Pavan-Microsoft 673ff45
Refactor data processing scripts and enhance agent integration
Pavan-Microsoft 1eec912
fix: Update script variables and enhance processing with agent framework
Pavan-Microsoft ca03047
fix: Correct agent deletion by using agent names instead of IDs
Pavan-Microsoft 5d244e2
fix: Refine topic mapping instructions and improve agent deletion by …
Pavan-Microsoft 78e5e5e
fix: Update topic guidelines to include additional common topics for …
Pavan-Microsoft 0a802ea
fix: Enhance topic mining by extracting common topics from processed …
Pavan-Microsoft 08efc7e
fix: Improve exception handling and clean up code formatting in data …
Pavan-Microsoft c8af804
fix: Correct parameter order in configuration examples for clarity
Pavan-Microsoft fa5ea9b
fix: Simplify log message for custom data processing script
Pavan-Microsoft cb74296
fix: Implement batch insert for processed records in data processing …
Pavan-Microsoft b59f6ea
fix: Update agent deletion to use delete_version method for consistency
Pavan-Microsoft 97d39ad
fix: Remove unnecessary blank lines in data processing scripts for im…
Pavan-Microsoft c75ac0e
fix: Simplify query formatting in data processing scripts for clarity
Pavan-Microsoft b050197
refactor: Conversation Processing with Agent-Based Topic Mining
Avijit-Microsoft fb28930
Merge branch 'dev' of https://github.com/microsoft/Conversation-Knowl…
Pavan-Microsoft 102bc7a
feat: Enhance agent creation script with improved value retrieval and…
Pavan-Microsoft 6e34676
fix: Update Azure AI Search tool initialization to use resource class…
Pavan-Microsoft d51feda
Merge branch 'dev' of https://github.com/microsoft/Conversation-Knowl…
Pavan-Microsoft 1ca2d22
fix: Remove unnecessary strict parameter from Azure AI Search tool de…
Pavan-Microsoft 36be340
fix: Update AI Search connection name variable and remove redundant p…
Pavan-Microsoft 9b82bfd
Merge branch 'dev' of https://github.com/microsoft/Conversation-Knowl…
Pavan-Microsoft 4fa2c5d
fix: streamline parameter descriptions in Deployment Guide for clarity
Pavan-Microsoft bb29a5c
Merge branch 'dev' of https://github.com/microsoft/Conversation-Knowl…
Pavan-Microsoft 7921c39
Refactor HistoryService and ChatService to use AzureAIProjectAgentPro…
Pavan-Microsoft 6d62b11
Update postprovision hooks in azure.yaml for clarity and command order
Pavan-Microsoft 5bf2982
Add logging configuration to suppress warnings from agent_framework i…
Pavan-Microsoft d92bb43
Update Bicep version and template hashes in main.json
Pavan-Microsoft d01dc97
Merge remote-tracking branch 'origin/dev' into km-agentframework-v2
Pavan-Microsoft 36d2107
Refactor database connection handling and improve error logging in da…
Pavan-Microsoft b788051
Refactor citation handling in ChatService to simplify unique citation…
Pavan-Microsoft c236f7a
Remove unnecessary blank line in citation formatting within ChatService
Pavan-Microsoft 608ed16
Enhance citation handling in ChatService to prevent duplicates and im…
Pavan-Microsoft 4c3ad82
Refactor citation handling in ChatService to improve duplicate tracki…
Pavan-Microsoft b702d91
Add Agent Framework v2 configuration variables to Local Development S…
Pavan-Microsoft 4073e01
Enhance agent creation script to safely parse output and update envir…
Pavan-Microsoft c74cd9f
Update deployment guides and scripts to enhance agent creation proces…
Pavan-Microsoft File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The new env var name here (
AI_FOUNDRY_RESOURCE_ID) doesn’t match the infrastructure outputs / sample env (AZURE_AI_FOUNDRY_RESOURCE_ID). This inconsistency will cause misconfiguration for local dev and contradicts the note below. Align the docs to the actual variable name used by the deployment outputs and.env.sample.