-
Notifications
You must be signed in to change notification settings - Fork 5
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
Feat: Add GPT-4o model (with caveats) #6
Conversation
whys it closed? looks good enough to me |
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.
i will merge it manually when its ok
@pokegamer5547 it has something to do with their API. if you have been abusing it then you are hitting the rate limit, but if you havent then it needs to be looked into you should like open an issue for it |
It's first time i used this and even on first try i had this issue. And i'm able to access on browser just time. And yes you're right, i'll open new issue for it. Even now after few hours and from other phone too on different network i'm having same issue. |
This PR adds the
gpt-4o-mini
model, which can be set with the stringGPT4OMini
in the config.I also made a change to disable the vqd caching and retrieval functionality. It seems DuckDuckGo have changed something on their backend, which causes errors with the newer model. This change doesn't seem to effect any of the model outputs, I may just be misunderstanding something so please do take a look.