-
Notifications
You must be signed in to change notification settings - Fork 3.6k
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
TokensCounter package added #8985
Conversation
Looks like tests failing not by my fault. |
d9eb4d1
to
5c254a1
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Automated testing result: SUCCESS
Repo link: TokensCounter
Packages added:
- TokensCounter
Processing package "TokensCounter"
- All checks passed
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Took me a while to realize what kind of tokens 😅 Package looks good though. Just one minor remark about your entry here.
Co-authored-by: Koen Lageveen <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Automated testing result: SUCCESS
Repo link: TokensCounter
Packages added:
- TokensCounter
Processing package "TokensCounter"
- All checks passed
@braver could you please check it once again, as I applied fixes that have been requested. |
My package is simply counting selected text as token, which is useful for folks that are working with llm. Is uses phantoms to present the calculations.
There's no package alike that, because there were lack of dependencies to perform this operation. Now they are available in PC.