-
Notifications
You must be signed in to change notification settings - Fork 360
feat(llmobs): support manual instrumentation of prompts #7257
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
base: master
Are you sure you want to change the base?
Conversation
…r/llmobs-prompts-support
…r/llmobs-prompts-support
…r/llmobs-prompts-support
Overall package sizeSelf size: 4.4 MB Dependency sizes| name | version | self size | total size | |------|---------|-----------|------------| | import-in-the-middle | 2.0.0 | 68.46 kB | 797.03 kB | | dc-polyfill | 0.1.10 | 26.73 kB | 26.73 kB |🤖 This report was automatically generated by heaviest-objects-in-the-universe |
Codecov Report❌ Patch coverage is Additional details and impacted files@@ Coverage Diff @@
## master #7257 +/- ##
==========================================
- Coverage 85.19% 85.08% -0.12%
==========================================
Files 532 532
Lines 22778 22863 +85
==========================================
+ Hits 19405 19452 +47
- Misses 3373 3411 +38 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
This comment has been minimized.
This comment has been minimized.
BenchmarksBenchmark execution time: 2026-01-16 18:49:08 Comparing candidate commit 63c3724 in PR branch Found 0 performance improvements and 0 performance regressions! Performance is the same for 230 metrics, 30 unstable metrics. |
What does this PR do?
Adds support for manually instrumenting prompts from the LLM Observability SDK. Additionally, uses this new logic in the
taggerin the existing OpenAI auto-instrumentation, which previously annotated prompts manually.Motivation
Feature parity with the Python SDK.
MLOB-5073
Testing
Ran these agains recently-merged system tests locally: