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

Fix callback handlers in tracing.py #245

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Conversation

lbliii
Copy link

@lbliii lbliii commented Nov 9, 2024

The imported observability tools don't seem to have been included and cause the chain-server container to crash stating that the module can't be found. As a workaround, I've added locally defined callbacks to unblock the use of the "Try it Now!" demo experience. I can now successfully load this demo from my Docker Desktop with these changes.

from RAG.tools.observability.langchain import opentelemetry_callback as langchain_otel_cb
from RAG.tools.observability.llamaindex import opentelemetry_callback as llama_index_otel_cb

Error in container:

from RAG.tools.observability.langchain import opentelemetry_callback as langchain_otel_cb
2024-11-09 11:38:58 ModuleNotFoundError: No module named 'RAG.tools.observability'

The imported observability tools don't seem to have been included and cause the `chain-server` container to crash stating that the module can't be found.  As a workaround, I've added locally defined callbacks to unblock the use of the "Try it Now!" demo experience. 

```
from RAG.tools.observability.langchain import opentelemetry_callback as langchain_otel_cb
from RAG.tools.observability.llamaindex import opentelemetry_callback as llama_index_otel_cb
```

Error in container: 

```
from RAG.tools.observability.langchain import opentelemetry_callback as langchain_otel_cb
2024-11-09 11:38:58 ModuleNotFoundError: No module named 'RAG.tools.observability'
```
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.

1 participant