-
Notifications
You must be signed in to change notification settings - Fork 135
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
support openai lib: 4.15.3 #335
Comments
Yep this makes sense. Feel free to open a PR, if not we'll definitely get to this as soon as we can! |
Reopening to fix the streaming bug in particular. |
Thanks @arcticfly and @corbt !
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
the code here only works for
4.8.0
: https://github.com/OpenPipe/OpenPipe/blob/main/client-libs/typescript/src/openai/streaming.ts#L13in the latest version, current(
4.15.3
): it is here: https://github.com/openai/openai-node/blob/e0138e8a5939f6fe53639df78cd541e4e20927ed/src/streaming.ts#L21C24-L21C24related PR: openai/openai-node#370
The problem is that current OpenPipe's OpenAI lib does not work for
streaming
API due to refactoring of the PR above.can we update to support latest code?
@arcticfly @corbt
The text was updated successfully, but these errors were encountered: