-
Notifications
You must be signed in to change notification settings - Fork 110
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鈥檒l occasionally send you account related emails.
Already on GitHub? Sign in to your account
馃悰 Bug Report: Requirements for Langchain example #1096
Comments
@nirga let me know if you need anything else? Also should I see the fastapi endpoint /invoke in the trace? Thanks, |
Thanks @damianoneill! Will try to reproduce this. For the fastapi - you'll need to add the FastAPI instrumentation. Are you using our SDK? |
Morning @nirga I'm not sure what you mean about the SDK, is there something other than below that I should be doing?
|
No, I meant that we don't instrument FastAPI currently, so you should do it yourself after initializing Traceloop. This is really easy:
|
Hey @nirga , I started a local docker instance of I init Traceloop as you mentioned above:
I also tried _api_endpoint="http://localhost:4318/v1/traces", same result. I wonder if there's a way to debug Traceloop as im not sure it's sending the correct http call. Best, |
Thanks @nirga since this is not a feature / bug rather more of a question i'll convert this into a discussion - thanks. |
Which component is this bug for?
Langchain Instrumentation
馃摐 Description
When using the default pyproject.toml generated by
Langchain instrumentation does not occur.
馃憻 Reproduction steps
'''
langchain app new chat
'''
Results in the following pyproject.toml
馃憤 Expected behavior
The docs should describe what else needs to be added to show full instrumentation for a langchain example.
馃憥 Actual Behavior with Screenshots
This results in only the LLM being traced.
馃 Python Version
3.11
馃搩 Provide any additional context for the Bug.
It looks like the following modules need to be added to get the following trace.
In the langchain example described here - #1043 langchain is explicitly added, and opentelemetry-instrumentation-fastapi is pulled in by chromadb.
馃憖 Have you spent some time to check if this bug has been raised before?
Are you willing to submit PR?
None
The text was updated successfully, but these errors were encountered: