-
Notifications
You must be signed in to change notification settings - Fork 545
feat: Inception provider #394
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
base: main
Are you sure you want to change the base?
Conversation
|
It looks like Inception is an OpenAI-compatible provider. Is there a reason you can't reuse the types directly from the OAI module? See Hyperbolic's intergration |
Oh great catch. That would simplify things for sure. |
|
So since the beta only supports text, it seems they don't yet accept structured content data in the // this works
{ ... "messages": [{ "content": "Hello, world!" }] }
// this is what OpenAI expects/OpenAI Message type serializes to. Does not work with Inception API right now
{ ... "messages": [{ "content": { "type": "text", "text": "Hello, world!" }}]}So I was able to simplify it by using the OpenAI provider's streaming utility, but not by using its The resulting runtime error if using the other |
It seems like a lot of "openai compatible" apis are like this which makes me think we might want a custom serializer here.. |
This reverts commit 01c9317.
|
@0xMochan Reverted to the custom message implementation |
Is this bc they support the full content type now? otherwise, we can flag this for review? |
We discussed in discord dms about reverting because they don't. I haven't actually checked to see if the API has changed though now that it's been a few weeks. |
|
@cbrit Don't suppose anything has changed on this? I'm looking into potentially adding more new providers. |
Closes #393