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

Support opentelemetry-instrumentation-llamaindex #614

Closed
yanqianglu opened this issue Nov 19, 2024 · 3 comments
Closed

Support opentelemetry-instrumentation-llamaindex #614

yanqianglu opened this issue Nov 19, 2024 · 3 comments

Comments

@yanqianglu
Copy link

Description

Hi,

There is a package to auto-instrument llmaindex opentelemetry-instrumentation-llamaindex , I wonder if that can be added to logfire. I tried to use it directly LlamaIndexInstrumentor().instrument(), but got maximum recursion depth exceeded error when adding it.

Best,

@alexmojaki
Copy link
Contributor

This doesn't seem to be a logfire-specific problem, I can't see how to get import opentelemetry.instrumentation.llamaindex to work. It works sometimes with pydantic v2 but I don't think llama-index actually supports that.

@Kludex
Copy link
Member

Kludex commented Dec 31, 2024

@yanqianglu I'll close this issue given Alex comment above.

If you have the availability, please give us an MRE so we can try to play with it. Assuming that it works without Logfire.

Happy new year! :)

@Kludex Kludex closed this as not planned Won't fix, can't repro, duplicate, stale Dec 31, 2024
@alexmojaki
Copy link
Contributor

It looks like I was wrong about llama-index not supporting pydantic v2. So I think that's what you have to use.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants