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

TokenCount performance issues #580

Merged
merged 1 commit into from
Nov 7, 2023
Merged

TokenCount performance issues #580

merged 1 commit into from
Nov 7, 2023

Conversation

JohanYman
Copy link
Contributor

Motivation and Context

We encountered some performance issues and implemented the following fix.

Description

We updated the TokenUtils class within the Skills Web API to address these issues. A newly-introduced tokenizer has been implemented for encoding text, which significantly improves the efficiency of the TokenCount method. As a result of this change, the performance of the token counting operations has been enhanced, ensuring more accurate and faster responses for users.

Contribution Checklist

…fix. We updated the TokenUtils class within the Skills Web API to address these issues. A newly-introduced tokenizer has been implemented for encoding text, which significantly improves the efficiency of the TokenCount method. As a result of this change, the performance of the token counting operations has been enhanced, ensuring more accurate and faster responses for users.
@github-actions github-actions bot added the webapi Pull requests that update .net code label Nov 7, 2023
@JohanYman JohanYman changed the title We encountered some performance issues and implemented the following … TokenCount performance issues Nov 7, 2023
@crickman
Copy link
Contributor

crickman commented Nov 7, 2023

Thank you for contributing this impactful update!

@crickman crickman added this pull request to the merge queue Nov 7, 2023
Merged via the queue into microsoft:main with commit 39edfcc Nov 7, 2023
7 checks passed
@JohanYman JohanYman deleted the TokenUtils-performace-fix branch November 8, 2023 14:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
webapi Pull requests that update .net code
Projects
No open projects
Development

Successfully merging this pull request may close these issues.

None yet

2 participants