-
Notifications
You must be signed in to change notification settings - Fork 128
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
docs: add link to langfuse in LangfuseConnector #981
Conversation
@vblagoje What do you think? I saw a link when checking out the chroma integration api reference and thought it would be useful on the langfuse page as well. |
Sounds good @marcklingen ! A small ask from our side is to add "Usage Notice" from https://docs.haystack.deepset.ai/docs/langfuseconnector somewhere in your https://langfuse.com/docs/integrations/haystack/get-started page as well. We had some users accidentally import one of the Haystack components before setting tracing enabled, leading to improper initialization ultimately resulting in no logs showing up in Langfuse. It's a common mistake our users face, see: |
Thank you! Got this feedback as well and added a note here: https://langfuse.com/docs/integrations/haystack/get-started#set-environment-variables |
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.
Thanks for this update @marcklingen
Co-authored-by: Vladimir Blagojevic <[email protected]>
Related Issues
None
Proposed Changes:
I propose to add a link to Langfuse to help people figure out what Langfuse actually is when browsing the api reference.
How did you test it?
I did not test it
Notes for the reviewer
None
Checklist
fix:
,feat:
,build:
,chore:
,ci:
,docs:
,style:
,refactor:
,perf:
,test:
.