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.

Sign up for GitHub

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

Jump to bottom

Python: Deprecated retry_mechanism #9965

Merged
merged 4 commits into from
Dec 13, 2024
Merged

Python: Deprecated retry_mechanism #9965

merged 4 commits into from
Dec 13, 2024

Conversation

TaoChenOSU
Copy link
Contributor

@TaoChenOSU TaoChenOSU commented Dec 13, 2024 •
edited
Loading

Motivation and Context

In Python, we have the retry_mechanism property in the kernel but it's never used.

Address: #9015

Description

  1. Deprecate the retry_mechanism field.
  2. Add a sample showing how to perform retries with the kernel using filter.

Contribution Checklist

@TaoChenOSU TaoChenOSU added the python Pull requests for the Python Semantic Kernel label Dec 13, 2024
@TaoChenOSU TaoChenOSU self-assigned this Dec 13, 2024
@TaoChenOSU TaoChenOSU requested a review from a team as a code owner December 13, 2024 01:20
@TaoChenOSU TaoChenOSU linked an issue Dec 13, 2024 that may be closed by this pull request
Python: Bug: retry_machanism is not used #9015
Closed
@github-actions github-actions bot changed the title Remove retry_mechanism Python: Remove retry_mechanism Dec 13, 2024
@TaoChenOSU TaoChenOSU changed the title Python: Remove retry_mechanism Python: Deprecated retry_mechanism Dec 13, 2024
@TaoChenOSU TaoChenOSU added this pull request to the merge queue Dec 13, 2024
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Dec 13, 2024
@TaoChenOSU TaoChenOSU added this pull request to the merge queue Dec 13, 2024
Merged via the queue into microsoft:main with commit 7531abf Dec 13, 2024
25 checks passed
@TaoChenOSU TaoChenOSU deleted the taochen/python-remove-retry-mechanism branch December 13, 2024 22:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@alliscode alliscode alliscode approved these changes

@eavanvalkenburg eavanvalkenburg eavanvalkenburg approved these changes

Assignees

@TaoChenOSU TaoChenOSU

Labels
python Pull requests for the Python Semantic Kernel
Projects
Semantic Kernel
Archived in project
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Python: Bug: retry_machanism is not used
3 participants
@TaoChenOSU @eavanvalkenburg @alliscode