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

Jacques/http integration #1

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

Jacques/http integration #1

wants to merge 10 commits into from

Conversation

jverre
Copy link

@jverre jverre commented Oct 3, 2024

Title

Added Comet's Opik open-source observability integration for logging and evaluation.

For more information on Opik, see: https://github.com/comet-ml/opik

Hope you find this useful!

Type

🆕 New Feature

Changes

The integration was implemented following the Langsmith and Datadog integration.

  1. Adds documentation: docs/my-website/docs/observability/opik_integration.md
  2. Lists Opik in sidebar
  3. Adds the integration: litellm/integrations/opik/opik.py
  4. Added tests: tests/local_testing/test_opik.py

UI Changes

Screenshot 2024-10-04 at 13 07 40 a screenshot/GIF of working UI fixes

Testing

All tests pass:

pytest tests/local
Screenshot 2024-10-04 at 12 59 04

litellm/integrations/opik/opik.py Outdated Show resolved Hide resolved
litellm/integrations/opik/opik.py Outdated Show resolved Hide resolved
litellm/integrations/opik/opik.py Show resolved Hide resolved
litellm/integrations/opik/opik.py Outdated Show resolved Hide resolved
litellm/integrations/opik/opik.py Show resolved Hide resolved
litellm/integrations/opik/utils.py Show resolved Hide resolved
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.

3 participants