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

fix: npm package @dqbd/tiktoken #48 #49

Open

sts07142 wants to merge 3 commits into context-labs:master from sts07142:fix-npm-package-#48

Contributor

sts07142 commented Aug 10, 2024

fix: #48

@dqbd/tiktoken 1.0.2 -> 1.0.15

sts07142 added 2 commits

August 10, 2024 11:23


          tiktoken version up

62dd78a

@dqbd/tiktoken 1.0.2 -> 1.0.15


          tiktoken version up

d62ede1

@dqbd/tiktoken 1.0.2 -> 1.0.15

sts07142 mentioned this pull request

Recent commit breaks the build #48

Open


          fix: convert model name when using @dqbd/tiktoken

e7a6237

Contributor Author

sts07142 commented Aug 12, 2024

fix: #48
Fix errors : If @dqbd/tiktoken does not have the same model name, convert it to the name of a compatible model

exist : gpt-4o
non-exist : gpt-4o-mini
gpt-4o-mini -> gpt-4o when convert model to encoding (o200k_base)

Errors Files
1 src/cli/commands/index/processRepository.ts:124
1 src/cli/commands/index/selectModel.ts:78

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet