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.

Sign up for GitHub

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

Jump to bottom

feat(sdk): client, annotations #2452

Draft
wants to merge 7 commits into
base: main
Choose a base branch
from
Draft

feat(sdk): client, annotations #2452

wants to merge 7 commits into from

Conversation

doronkopit5
Copy link
Contributor

@doronkopit5 doronkopit5 commented Jan 2, 2025 •
edited
Loading

  • I have added tests that cover my changes.
  • If adding a new instrumentation or changing an existing one, I've added screenshots from some observability platform showing the change.
  • PR name follows conventional commits format: feat(instrumentation): ... or fix(instrumentation): ....
  • (If applicable) I have updated the documentation accordingly.

@doronkopit5
feat(sdk): Add report_labeling method and enhance type hints
7a1133f 
- Introduced a new static method `report_labeling` in the Traceloop class to facilitate reporting labeling data with detailed parameters.
- Enhanced type hints across the SDK, including the use of `Optional` for several parameters and improved type definitions for API requests.
- Updated the Fetcher class to support posting to a new API endpoint with better error handling.
- Refactored existing code for clarity and consistency, including adjustments to environment variable handling and method signatures.
@doronkopit5
fmt
cd9e745
@doronkopit5
lint
eecdd5b
@doronkopit5
deprecate use score
cf09fea
@doronkopit5
refactor and creat client instance
208e7e1
@doronkopit5
lint
01149b4
@doronkopit5
Merge branch 'main' of github.com:traceloop/openllmetry into dk/label…
993c67c 
…ings
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@doronkopit5