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

Adds GPT4-o model to Wasp AI #2081

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

Conversation

infomiho
Copy link
Contributor

@infomiho infomiho commented Jun 5, 2024

Adds GPT4-o to the list of supported list of Wasp AI models.

Based on https://platform.openai.com/docs/models/gpt-4o

Closes #2043

@infomiho infomiho requested a review from Martinsos June 5, 2024 10:10
@Martinsos
Copy link
Member

Ok sweet! We can actually merge this already, since it allows them to pick this model now via CLI via that config object, but I was also hoping to do a bit more with this:

  1. Test how well it works, if it works as good as GPT4, then make GPT4o new default.
  2. In CLI, in wizard where we offer them 3 different options for which model they want to use, again, if GPT4o is a good replacement, replace GPT4 with GPT4o in those.

So that way GPT4o would replace GPT4 in current setup, and GPT4 would become something you can pick only via CLI via config object, so the opposite of what we have now.

If you want to do this in this PR (maybe easiest to do it now!), let's do it, if not, then let's create new issue for it. I would go for it now while it is fresh, especially if you did some testing already.

Copy link
Member

@Martinsos Martinsos left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Commented above :)! We can merge this but suggest going step further and making GPT4o default instead of GPT4, if it works well.

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.

Add gpt4-o to Mage
3 participants