Skip to content

Python: Add check for not approved and approved for Agent Sample #12316

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

nmoeller
Copy link
Contributor

@nmoeller nmoeller commented May 30, 2025

Motivation and Context

I was having a Demo with the Agents, and found that step06_chat_completion_agent_group_chat.py sample always terminated after one iteration.

The Reviewer Agent answered "not approved", which also triggered the Termination Strategy, due to the fact that the word approved is also in not approved.
To only trigger when there is a real approval I added a check if the answer is approved and the answer also don't include not approve

Contribution Checklist

@nmoeller nmoeller requested a review from a team as a code owner May 30, 2025 06:53
@markwallace-microsoft markwallace-microsoft added the python Pull requests for the Python Semantic Kernel label May 30, 2025
@github-actions github-actions bot changed the title Add check for not approved and approved Python: Add check for not approved and approved May 30, 2025
@nmoeller nmoeller changed the title Python: Add check for not approved and approved Python: Add check for not approved and approved for Agent Sample May 30, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
python Pull requests for the Python Semantic Kernel
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants