Skip to content

Update default handler in Langfuse Tracer to serialize ToolCallResult properly #1664

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.

Sign up for GitHub

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

Jump to bottom
Closed
sjrl opened this issue Apr 22, 2025 · 1 comment
Closed

Update default handler in Langfuse Tracer to serialize ToolCallResult properly #1664

sjrl opened this issue Apr 22, 2025 · 1 comment
Labels
P1

Comments

@sjrl
Copy link
Contributor

sjrl commented Apr 22, 2025 •
edited
Loading

In the example shown below we can see that tool_messages isn't a JSON dict but a string due to it not being JSON serializable.

This is most likely due to State not having a to_dict method. A to_dict method should be added to State so the tool_messages can be represented as a dict in Langfuse.

Image
@sjrl sjrl mentioned this issue Apr 22, 2025
Open
@julian-risch
Copy link
Member

Closing because we track the issue now in deepset-ai/haystack#9286

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
P1
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@julian-risch @sjrl