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

Upgraded to GPT 4 Model #4

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

cameronaaron
Copy link

I upgraded it to GPT 4 Model to increase token limit and general reasoning and logic. God just became smarter and more logical. I will work on cleaning up the rest of the code next.

@YOUSY0US3F
Copy link
Owner

First of all thank you for making a contribution, the way you set up the models here makes the program assume the model has a limit of 4,000 - 16,000 tokens. gpt-4-0125-preview has a token limit of 128,000 and while I do believe it is unwise to allow the program to make use of all of those tokens (very expensive) I think we could raise the tokens to 32,000. I think the least destructive way to add GPT-4 to the code is to make another model object like the ones you've edited and make the program use that model only. Can't wait to see what else you push in

@YOUSY0US3F
Copy link
Owner

First of all thank you for making a contribution, the way you set up the models here makes the program assume the model has a limit of 4,000 - 16,000 tokens. gpt-4-0125-preview has a token limit of 128,000 and while I do believe it is unwise to allow the program to make use of all of those tokens (very expensive) I think we could raise the tokens to 32,000. I think the least destructive way to add GPT-4 to the code is to make another model object like the ones you've edited and make the program use that model only. Can't wait to see what else you push in

Also the function logic will need to be updated to accommodate the new GPT models

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.

2 participants