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

add openllmetry-powered auto instrumentation to python sdk #8455

Merged
merged 18 commits into from
May 3, 2024

Conversation

Vadman97
Copy link
Member

@Vadman97 Vadman97 commented May 3, 2024

Summary

  • Adds openllmetry-powered auto instrumentation to the highlight python sdk.
  • Adds opentelemetry context argument to the H.trace method for threaded trace context propagation.
  • Adds threaded span recording example.
  • Refactors python sdk to separate e2e examples to the top level e2e directory.

How did you test this change?

new docs on vercel
unit tests
manual testing
Screenshot from 2024-05-03 14-50-05

Are there any deployment considerations?

python sdk version bump

Does this work require review from our design team?

no

Copy link

changeset-bot bot commented May 3, 2024

⚠️ No Changeset found

Latest commit: 682ed28

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

💥 An error occurred when fetching the changed packages and changesets in this PR
Some errors occurred when validating the changesets config:
The package or glob expression "rrdom" is specified in the `ignore` option but it is not found in the project. You may have misspelled the package name or provided an invalid glob expression. Note that glob expressions must be defined according to https://www.npmjs.com/package/micromatch.
The package or glob expression "rrdom-nodejs" is specified in the `ignore` option but it is not found in the project. You may have misspelled the package name or provided an invalid glob expression. Note that glob expressions must be defined according to https://www.npmjs.com/package/micromatch.
The package or glob expression "rrweb" is specified in the `ignore` option but it is not found in the project. You may have misspelled the package name or provided an invalid glob expression. Note that glob expressions must be defined according to https://www.npmjs.com/package/micromatch.
The package or glob expression "rrweb-player" is specified in the `ignore` option but it is not found in the project. You may have misspelled the package name or provided an invalid glob expression. Note that glob expressions must be defined according to https://www.npmjs.com/package/micromatch.
The package or glob expression "rrweb-snapshot" is specified in the `ignore` option but it is not found in the project. You may have misspelled the package name or provided an invalid glob expression. Note that glob expressions must be defined according to https://www.npmjs.com/package/micromatch.
The package or glob expression "@rrweb/types" is specified in the `ignore` option but it is not found in the project. You may have misspelled the package name or provided an invalid glob expression. Note that glob expressions must be defined according to https://www.npmjs.com/package/micromatch.
The package or glob expression "@rrweb/web-extension" is specified in the `ignore` option but it is not found in the project. You may have misspelled the package name or provided an invalid glob expression. Note that glob expressions must be defined according to https://www.npmjs.com/package/micromatch.
The package or glob expression "rrvideo" is specified in the `ignore` option but it is not found in the project. You may have misspelled the package name or provided an invalid glob expression. Note that glob expressions must be defined according to https://www.npmjs.com/package/micromatch.

e2e/openllmetry/main.py Fixed Show fixed Hide fixed
@Vadman97 Vadman97 changed the title add openllmetry auto instrumentation add openllmetry-powered auto instrumentation to python sdk May 3, 2024
@Vadman97 Vadman97 requested review from a team and SpennyNDaJets and removed request for a team May 3, 2024 20:36
e2e/python/highlight_openai/main.py Dismissed Show dismissed Hide dismissed
e2e/python/highlight_script/thread.py Dismissed Show dismissed Hide dismissed
@Vadman97 Vadman97 requested a review from jay-khatri May 3, 2024 21:49
@Vadman97 Vadman97 marked this pull request as ready for review May 3, 2024 21:49
Copy link
Contributor

github-actions bot commented May 3, 2024

☂️ Python Coverage

current status: ✅

Overall Coverage

Lines Covered Coverage Threshold Status
570 506 89% 0% 🟢

New Files

File Coverage Status
sdk/highlight-py/highlight_io/integrations/all.py 100% 🟢
sdk/highlight-py/highlight_io/integrations/anthropic.py 100% 🟢
sdk/highlight-py/highlight_io/integrations/bedrock.py 100% 🟢
sdk/highlight-py/highlight_io/integrations/chromadb.py 83% 🟢
sdk/highlight-py/highlight_io/integrations/cohere.py 100% 🟢
sdk/highlight-py/highlight_io/integrations/haystack.py 100% 🟢
sdk/highlight-py/highlight_io/integrations/langchain.py 100% 🟢
sdk/highlight-py/highlight_io/integrations/llamaindex.py 83% 🟢
sdk/highlight-py/highlight_io/integrations/openai.py 83% 🟢
sdk/highlight-py/highlight_io/integrations/pinecone.py 83% 🟢
sdk/highlight-py/highlight_io/integrations/qdrant.py 83% 🟢
sdk/highlight-py/highlight_io/integrations/replicate.py 100% 🟢
sdk/highlight-py/highlight_io/integrations/transformers.py 83% 🟢
sdk/highlight-py/highlight_io/integrations/vertexai.py 100% 🟢
sdk/highlight-py/highlight_io/integrations/watsonx.py 100% 🟢
sdk/highlight-py/highlight_io/integrations/weaviate.py 100% 🟢
TOTAL 94% 🟢

Modified Files

File Coverage Status
sdk/highlight-py/highlight_io/integrations/init.py 88% 🟢
sdk/highlight-py/highlight_io/integrations/boto.py 100% 🟢
sdk/highlight-py/highlight_io/integrations/boto3sqs.py 100% 🟢
sdk/highlight-py/highlight_io/integrations/celery.py 100% 🟢
sdk/highlight-py/highlight_io/integrations/django.py 60% 🟢
sdk/highlight-py/highlight_io/integrations/fastapi.py 100% 🟢
sdk/highlight-py/highlight_io/integrations/flask.py 66% 🟢
sdk/highlight-py/highlight_io/integrations/redis.py 100% 🟢
sdk/highlight-py/highlight_io/integrations/requests.py 100% 🟢
sdk/highlight-py/highlight_io/integrations/serverless.py 87% 🟢
sdk/highlight-py/highlight_io/integrations/sqlalchemy.py 100% 🟢
sdk/highlight-py/highlight_io/sdk.py 90% 🟢
TOTAL 91% 🟢

updated for commit: 682ed28 by action🐍

@Vadman97 Vadman97 merged commit b5d444f into main May 3, 2024
23 checks passed
@Vadman97 Vadman97 deleted the vadim/openllmetry branch May 3, 2024 23:52
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.

None yet

3 participants