-
Notifications
You must be signed in to change notification settings - Fork 126
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
Langfuse: add invocation_context to identify traces #1089
Conversation
vblagoje
commented
Sep 17, 2024
- WIP, used to demonstrate a possible solution to Feature request for the possibility of adding user_id to the trace while using Haystack<>Langfuse connector #916
Any estimate on when this could be merged? |
Do you find this PR useful @ggdupont ? Can you tell us a bit about your use case please? |
I want (at least) to track session id accross calls to the llm just like the original issue. Later I also plan to add conversation metrics for performance monitoring. All should be ligh weigths and simple type metadata so the "invocation_context" seems quite perfect for this. |
0b9d070
to
cc16608
Compare
Ok thank you @ggdupont I agree and it seems like a low impact high return PR. Let's see what the peer review says |
@silvanocerza please have a look as well and luk your thoughts. |
ping @silvanocerza - have a look now 🙏 |
Feature included in https://pypi.org/project/langfuse-haystack/0.5.0/ release onward - thanks @silvanocerza @ggdupont @kluhan |
FYI already integrated and happy of the result. |