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: Improve agent getting started samples #10667

Merged

Conversation

TaoChenOSU
Copy link
Contributor

@TaoChenOSU TaoChenOSU commented Feb 25, 2025

Motivation and Context

We are improving developer experience by updating the agent getting started samples.

Description

Simplify the agent getting started samples by removing unnecessary code and add excessive comments to make sure we are only doing what we need to get started.

Contribution Checklist

@TaoChenOSU TaoChenOSU added python Pull requests for the Python Semantic Kernel agents labels Feb 25, 2025
@TaoChenOSU TaoChenOSU self-assigned this Feb 25, 2025
@TaoChenOSU TaoChenOSU requested a review from a team as a code owner February 25, 2025 05:21
@github-actions github-actions bot changed the title Improve agent getting started samples Python: Improve agent getting started samples Feb 25, 2025
@markwallace-microsoft
Copy link
Member

markwallace-microsoft commented Feb 26, 2025

Python Test Coverage

Python Test Coverage Report •
FileStmtsMissCoverMissing
semantic_kernel/agents/azure_ai
   agent_thread_actions.py34112962%140, 142, 177–180, 365, 367, 406–407, 433, 436–440, 443–455, 458–489, 495–502, 505–518, 534–565, 609, 612, 641–675, 738, 743–748, 768–775, 806–815, 858–868
   azure_ai_agent.py931485%95, 97, 103, 112–117, 119, 121, 142–145, 193, 253
semantic_kernel/agents/chat_completion
   chat_completion_agent.py110794%82, 85, 94–99, 124, 192
TOTAL18839236687% 

Python Unit Test Overview

Tests Skipped Failures Errors Time
3164 5 💤 0 ❌ 0 🔥 1m 24s ⏱️

@TaoChenOSU TaoChenOSU enabled auto-merge February 26, 2025 18:16
@TaoChenOSU TaoChenOSU added this pull request to the merge queue Feb 26, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Feb 26, 2025
@TaoChenOSU TaoChenOSU added this pull request to the merge queue Feb 26, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Feb 26, 2025
@eavanvalkenburg eavanvalkenburg added this pull request to the merge queue Feb 26, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Feb 26, 2025
@TaoChenOSU TaoChenOSU added this pull request to the merge queue Feb 26, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Feb 26, 2025
@eavanvalkenburg eavanvalkenburg added this pull request to the merge queue Feb 26, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Feb 26, 2025
@TaoChenOSU TaoChenOSU added this pull request to the merge queue Feb 26, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Feb 26, 2025
@moonbox3 moonbox3 added this pull request to the merge queue Feb 26, 2025
Merged via the queue into microsoft:main with commit 60df17d Feb 26, 2025
28 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
agents documentation python Pull requests for the Python Semantic Kernel
Projects
Status: Sprint: Done
Development

Successfully merging this pull request may close these issues.

5 participants