-
Notifications
You must be signed in to change notification settings - Fork 117
Add Gemini text generations instrumentation #1349
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
Add Gemini text generations instrumentation #1349
Conversation
🦙 MegaLinter status: ✅ SUCCESS
See detailed report in MegaLinter reports |
…c-python-agent into gemini-text-generations
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## feature-gemini-instrumentation #1349 +/- ##
=================================================================
Coverage ? 81.45%
=================================================================
Files ? 205
Lines ? 23088
Branches ? 3650
=================================================================
Hits ? 18807
Misses ? 3055
Partials ? 1226 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
…c-python-agent into gemini-text-generations
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Posting a partial review-I'll review the rest tomorrow.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I was trying to check the code coverage report to make sure we covered all the cases but it seems it's not available. Maybe it will be after you push some changes or maybe just take a look at it locally and double check after.
…lic-python-agent into gemini-text-generations
This PR adds instrumentation and tests for Gemini text generations