Skip to content
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

Add OpenLIT to Haystack Integrations #234

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

patcher9
Copy link

@patcher9 patcher9 commented May 22, 2024

How OpenLIT works with Haystack?

OpenLIT basically wraps around OpenAI, Anthropic etc functions and configures OpenTelemetry tracing and metrics for it. In the Haystack pipelines whenever the user runs a pipeline(Since Haystack also uses the same LLM functions), We are automatically able to wrap the pipeline (without needing to create any additional instrumentation for Haystack).

We did add instrumentation just for Document loaders as the rest of the pipeline can directly be tracked for reasons mentioned above.

@patcher9 patcher9 requested a review from a team as a code owner May 22, 2024 10:23
@patcher9
Copy link
Author

patcher9 commented Jul 9, 2024

@TuanaCelik , Tagging you for sanity as not sure who to tag for review on this :)

@TuanaCelik
Copy link
Member

Thanks for the contribution @patcher9 - Sorry we're slow at getting to PRs on this repository. I'm assigning this to myself now and will test it and make a review 🙏

@TuanaCelik TuanaCelik self-requested a review July 12, 2024 18:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants