Skip to content

Conversation

@alizenhom
Copy link
Collaborator

@alizenhom alizenhom commented Jul 12, 2024

Description

Please include a summary of the changes and the related issue to help is review the PR better and faster.

Checklist for adding new integration:

  • Defined APIS in constants folder.
  • Updated SERVICE_PROVIDERS in common.py
  • Created a folder under instrumentation with the name of the integration with atleast patch.py and instrumentation.py files.
  • Added instrumentation in all_instrumentations in langtrace.py and to the InstrumentationType in types.py files.
  • Added examples for the new integration in the examples folder.
  • Updated pyproject.toml to install new dependencies
  • Updated the README.md of langtrace-python-sdk to include the new integration in the supported integrations table.
  • Updated the README.md of Langtrace's repository to include the new integration in the supported integrations table.
  • Added new integration page to supported integrations in Langtrace Docs

@alizenhom alizenhom added the WIP Work in progress label Jul 12, 2024
Base automatically changed from development to main July 24, 2024 07:48
@alizenhom alizenhom changed the base branch from main to development August 15, 2024 08:49
@alizenhom alizenhom marked this pull request as draft August 15, 2024 08:49
Base automatically changed from development to main August 15, 2024 16:45
@alizenhom alizenhom changed the base branch from main to development August 27, 2024 10:12
Base automatically changed from development to main August 29, 2024 08:52
@alizenhom alizenhom changed the base branch from main to development September 9, 2024 11:19
@alizenhom alizenhom marked this pull request as ready for review September 11, 2024 10:22
@alizenhom alizenhom added new instrumentation Adding support for a new vendor and removed WIP Work in progress labels Sep 11, 2024
@karthikscale3 karthikscale3 merged commit 77024fb into development Sep 11, 2024
@karthikscale3 karthikscale3 deleted the ali/instrument-autogen branch September 11, 2024 20:15
karthikscale3 added a commit that referenced this pull request Sep 11, 2024
* remove logs

* remove requirements

* Bump version

* Squash

* add sentry integration

* update readme

* hotfix for checking package installed

* Support Autogen (#242)

* autogen kickstart

* enhance autogen

* run autogen

* finish autogen

* add readme

* disable cache

* Support genai and also add token reporting and other data points (#345)

* support genai and also add token reporting

* rename example file

* fix

---------

Co-authored-by: Karthik Kalyanaraman <karthik@scale3labs.com>

* fix: weaviate datetime handling for request and response (#346)

* fix: weaviate datetime handling for request and response

* bump version

* bump version

---------

Co-authored-by: Karthik Kalyanaraman <karthik@scale3labs.com>

---------

Co-authored-by: Rohit Kadhe <rohit@scale3labs.com>
Co-authored-by: Rohit Kadhe <113367036+rohit-kadhe@users.noreply.github.com>
Co-authored-by: Ali Waleed <ali@scale3labs.com>
Co-authored-by: darshit-s3 <119623510+darshit-s3@users.noreply.github.com>
karthikscale3 added a commit that referenced this pull request Sep 11, 2024
* remove logs

* remove requirements

* Bump version

* Squash

* add sentry integration

* update readme

* hotfix for checking package installed

* Support Autogen (#242)

* autogen kickstart

* enhance autogen

* run autogen

* finish autogen

* add readme

* disable cache

* Support genai and also add token reporting and other data points (#345)

* support genai and also add token reporting

* rename example file

* fix

---------

Co-authored-by: Karthik Kalyanaraman <karthik@scale3labs.com>

* fix: weaviate datetime handling for request and response (#346)

* fix: weaviate datetime handling for request and response

* bump version

* bump version

---------

Co-authored-by: Karthik Kalyanaraman <karthik@scale3labs.com>

* Minor bugfix to langchain instrumentation (#348)

* Bugfix

* bump version

---------

Co-authored-by: Rohit Kadhe <rohit@scale3labs.com>
Co-authored-by: Rohit Kadhe <113367036+rohit-kadhe@users.noreply.github.com>
Co-authored-by: Ali Waleed <ali@scale3labs.com>
Co-authored-by: darshit-s3 <119623510+darshit-s3@users.noreply.github.com>
darshit-s3 added a commit that referenced this pull request Sep 12, 2024
* add sentry integration

* update readme

* hotfix for checking package installed

* Support Autogen (#242)

* autogen kickstart

* enhance autogen

* run autogen

* finish autogen

* add readme

* disable cache

* Support genai and also add token reporting and other data points (#345)

* support genai and also add token reporting

* rename example file

* fix

---------

Co-authored-by: Karthik Kalyanaraman <karthik@scale3labs.com>

* fix: weaviate datetime handling for request and response (#346)

* fix: weaviate datetime handling for request and response

* bump version

* bump version

---------

Co-authored-by: Karthik Kalyanaraman <karthik@scale3labs.com>

* Minor bugfix to langchain instrumentation (#348)

* Bugfix

* bump version

* fix: handling for openai NOT_GIVEN default arg value (#350)

* fix: handling for openai NOT_GIVEN default arg value

* style: fix formating

---------

Co-authored-by: Rohit Kadhe <rohit@scale3labs.com>
Co-authored-by: Rohit Kadhe <113367036+rohit-kadhe@users.noreply.github.com>
Co-authored-by: Ali Waleed <ali@scale3labs.com>
Co-authored-by: Karthik Kalyanaraman <karthik@scale3labs.com>
Co-authored-by: Karthik Kalyanaraman <105607645+karthikscale3@users.noreply.github.com>
karthikscale3 added a commit that referenced this pull request Sep 16, 2024
* add sentry integration

* update readme

* hotfix for checking package installed

* Support Autogen (#242)

* autogen kickstart

* enhance autogen

* run autogen

* finish autogen

* add readme

* disable cache

* Support genai and also add token reporting and other data points (#345)

* support genai and also add token reporting

* rename example file

* fix

---------

Co-authored-by: Karthik Kalyanaraman <karthik@scale3labs.com>

* fix: weaviate datetime handling for request and response (#346)

* fix: weaviate datetime handling for request and response

* bump version

* bump version

---------

Co-authored-by: Karthik Kalyanaraman <karthik@scale3labs.com>

* Minor bugfix to langchain instrumentation (#348)

* Bugfix

* bump version

* fix: handling for openai NOT_GIVEN default arg value (#350)

* fix: handling for openai NOT_GIVEN default arg value

* style: fix formating

* fix: tool choice for groq, datetime for cohere (#352)

---------

Co-authored-by: Ali Waleed <ali@scale3labs.com>
Co-authored-by: Karthik Kalyanaraman <karthik@scale3labs.com>
Co-authored-by: darshit-s3 <119623510+darshit-s3@users.noreply.github.com>
Co-authored-by: Karthik Kalyanaraman <105607645+karthikscale3@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

new instrumentation Adding support for a new vendor

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants