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

Update and add gpt4turbo #193

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

maakuth
Copy link

@maakuth maakuth commented Nov 11, 2023

No description provided.

@maakuth
Copy link
Author

maakuth commented Nov 11, 2023

docker build results in error without the package.json updates.

@mstoyanovv
Copy link

@maakuth could we merge this :)

@maakuth
Copy link
Author

maakuth commented Nov 24, 2023

Only somebody with write permission to the original repo can merge it.

@Vuthric
Copy link

Vuthric commented Dec 6, 2023

Hi @maakuth, really appreciate the fix! The dependencies update didn't work for me though. Did anything change since 8eaf628 ?

@Vuthric
Copy link

Vuthric commented Dec 6, 2023

Ahh it was my bad. I wasn't at head of the master branch, duh. Sorry for the @ !

@Vuthric
Copy link

Vuthric commented Dec 6, 2023

fyi we also need to add gpt-4-1106-preview to maxTokensByModel in openai.ts

export const maxTokensByModel = {
"gpt-4-1106-preview": 8192,
"gpt-3.5-turbo": 4096,
"gpt-4": 8192,
[...]

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants