-
Notifications
You must be signed in to change notification settings - Fork 197
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
Some Updates RFC #20
Comments
Check out the pull here if interested in a diff |
Hey man. I'm just a nobbie but I'm trying to learn with your project. |
I may be wrong but you will need I believe this is linked to the Let me know if that works. otherwise will have to do more research. so ...
|
Hey @syonfox, I've also spent some time fixing up this package in #38 (published as gpt-tokenizer on npm). I don't have the tokenStats implemented in my fork, but I've added some other optimizations and features. See |
Hi i have spent a little bit of time adding to this library in my fork. we simply need to change the URLs back and pull to update these repos.
If anyone is looking for some newer stuff take to speak and feel free to open an issue. I think ill basically leave it here the goal is to make it a 1.4.2 release and have that be final as far as what is needed fro this component.
The major things added are:
countTokens (a faster way if you don't care about the contents)
tokenStats : some interesting insights into encoded strings such as frequency and position maps.
jsdocs: for implementation interface docs
browserify: for browser support
The major things to test are: if we can get a good working version of the original python version, I added it to the npm package in case it just works but it would be good to have the package support Node, Browser and Python
Also check out the browser demo browser demo
Compatible with Node >= 12
overall just lmk what you think.
The text was updated successfully, but these errors were encountered: