-
Notifications
You must be signed in to change notification settings - Fork 14k
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
core[patch]: set schema format for AsyncRootListenersTracer #23214
Conversation
mackong
commented
Jun 20, 2024
- Description: AsyncRootListenersTracer support on_chat_model_start, it's schema_format should be "original+chat".
- Issue: N/A
- Dependencies:
The latest updates on your projects. Learn more about Vercel for Git ↗︎ 1 Ignored Deployment
|
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.
@nicolasnk any concerns with this change? I think we want to merge this in, but wanted to check in with you first.
@mackong thanks, going to give @nicolasnk a chance to opine prior to merging in |
Hey @eyurtsev. Thanks for letting me know. I see that the same code for RootListener. Does it make sense to do it in the parent class _TracerCore ? |
@nicolasnk Yes, but we'll do it as a part of a separate PR. We need to spec out if anything might break for users that are sub-classing from BaseTracer right now |
…n-ai#23214) - **Description:** AsyncRootListenersTracer support on_chat_model_start, it's schema_format should be "original+chat". - **Issue:** N/A - **Dependencies:**