Skip to content

Conversation

@lcsvcn
Copy link

@lcsvcn lcsvcn commented May 22, 2024

No description provided.

@lcsvcn lcsvcn changed the title Feat: add gpt4o feat: add gpt4o model May 22, 2024
@clharman
Copy link

Thanks for creating this PR. Please merge this 🙏 or we are gonna have to use something else :(

@lgrammel
Copy link
Collaborator

@clharman i recommend using the ai sdk instead https://sdk.vercel.ai/docs/introduction which supports gpt4o. have you given it a try?

@clharman
Copy link

clharman commented Jun 1, 2024

@lgrammel I will check it out! Still hoping to get 4o in modelfusion until we make the switch. Is modelfusion planned for deprecation in favor of ai sdk?

@lcsvcn
Copy link
Author

lcsvcn commented Jun 3, 2024

@clharman just a tip if you are so eager to use it before migrating (as we were too), you can run my modelfusion repository by cloning our custom repository.

We are doing that at the moment and we are able to run on production gpt4o successfully, without any issues.

Basically what we did so far is very similar how it is done on the modelfusion examples folder, but you need to do that on your package.json folder and clone the modelfusion repository into your project.

"modelfusion-experimental": "workspace:*",

don't forget the pnpm-workspace file:

packages:
- "docs"
- "examples/*"
- "packages/*"
- "test-environments/*"
- "tools/*"

here is how it looks like on our github repository:
Screenshot 2024-06-03 at 09 38 43

the downside is that you need to clone the project and build before you can run your project, that adds time when we run our CI/CD.

@Hexodus
Copy link

Hexodus commented Jun 19, 2024

@clharman i recommend using the ai sdk instead https://sdk.vercel.ai/docs/introduction which supports gpt4o. have you given it a try?

Is there a migration guide? Would be nice to know what has to be changed. Just the imports or are there more in depth changes needed?

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.

5 participants