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

.Net: Increase auto-invoke and in-flight tool calling hard-coded limits #6272

Merged
merged 7 commits into from
May 16, 2024

Conversation

stephentoub
Copy link
Member

As we discussed offline yesterday, with auto function calling filters, someone can now put their own limits in place, so raising these hard-coded back stops to something much less likely to be hit. We could subsequently get rid of them completely if desired.

@stephentoub stephentoub requested a review from a team as a code owner May 15, 2024 20:09
@markwallace-microsoft markwallace-microsoft added .NET Issue or Pull requests regarding .NET code kernel Issues or pull requests impacting the core kernel labels May 15, 2024
@github-actions github-actions bot changed the title Increase auto-invoke and in-flight tool calling hard-coded limits .Net: Increase auto-invoke and in-flight tool calling hard-coded limits May 15, 2024
As we discussed offline, with auto function calling filters, someone can now put their own limits in place, so raising these to something much less likely to be hit. We could subsequently get rid of them completely if desired, but keeping them now as a backstop.
@SergeyMenshykh SergeyMenshykh added this pull request to the merge queue May 16, 2024
Merged via the queue into microsoft:main with commit 41f072d May 16, 2024
15 checks passed
bochris pushed a commit to bochris/semantic-kernel that referenced this pull request May 16, 2024
…ts (microsoft#6272)

As we discussed offline yesterday, with auto function calling filters,
someone can now put their own limits in place, so raising these
hard-coded back stops to something much less likely to be hit. We could
subsequently get rid of them completely if desired.

---------

Co-authored-by: SergeyMenshykh <68852919+SergeyMenshykh@users.noreply.github.com>
Co-authored-by: SergeyMenshykh <sergemenshikh@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kernel Issues or pull requests impacting the core kernel .NET Issue or Pull requests regarding .NET code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants