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

[BUG] DSPyInstrumentor does not work with DSPY 2.6.0 #1250

Closed
juicesharp opened this issue Feb 1, 2025 · 3 comments
Closed

[BUG] DSPyInstrumentor does not work with DSPY 2.6.0 #1250

juicesharp opened this issue Feb 1, 2025 · 3 comments
Assignees
Labels
bug Something isn't working language: python Related to Python integration

Comments

@juicesharp
Copy link

hat happened?

After upgrading to DSPy 2.6.0
DSPyInstrumentor().instrument(tracer_provider=tracer_provider) # throws exception No module named 'dsp'

openinference.instrumentation.dspy (0.1.14) installed in the current venv environment.

https://app.screencast.com/h0JDx5xHrRNhW

Steps to reproduce

create venv with dspy 2.6.0
packages:
dspy
openinference-instrumentation-litellm
openinference.instrumentation.dspy

try to use DSPyInstrumentor
from openinference.instrumentation.dspy import DSPyInstrumentor
from opentelemetry import trace as trace_api
from opentelemetry.exporter.otlp.proto.http.trace_exporter import OTLPSpanExporter
from opentelemetry.sdk import trace as trace_sdk
from opentelemetry.sdk.trace.export import SimpleSpanProcessor

Instrumentation

endpoint = "http://127.0.0.1:6006/v1/traces"
tracer_provider = trace_sdk.TracerProvider()
trace_api.set_tracer_provider(tracer_provider)
tracer_provider.add_span_processor(SimpleSpanProcessor(OTLPSpanExporter(endpoint)))

DSPyInstrumentor().instrument(tracer_provider=tracer_provider) <---- Error

....

DSPy version

2.6.0

@juicesharp juicesharp added bug Something isn't working triage Issues that require triage labels Feb 1, 2025
@mikeldking
Copy link
Contributor

Thanks for reporting @juicesharp. Looks like there was a migration of the lm modules. We'll take a look in the coming week but PRs are always welcome. Thanks again!

@mikeldking mikeldking transferred this issue from Arize-ai/phoenix Feb 3, 2025
@dosubot dosubot bot added the language: python Related to Python integration label Feb 3, 2025
@mikeldking mikeldking removed the triage Issues that require triage label Feb 3, 2025
@pavan4
Copy link
Contributor

pavan4 commented Feb 3, 2025

@mikeldking PR is here #1249

@RogerHYang
Copy link
Contributor

@mikeldking PR is here #1249

updated is released in v0.1.15

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working language: python Related to Python integration
Projects
Status: Done
Development

No branches or pull requests

5 participants