Skip to content

Conversation

@rohit-kadhe
Copy link
Contributor

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

@rohit-kadhe rohit-kadhe force-pushed the rohit/S3EN-2799-langchain_google_genai-support branch from 8d3b769 to 0ddd720 Compare September 11, 2024 18:15
karthikscale3
karthikscale3 previously approved these changes Sep 11, 2024
karthikscale3
karthikscale3 previously approved these changes Sep 11, 2024
@karthikscale3 karthikscale3 merged commit c514af4 into development Sep 11, 2024
@karthikscale3 karthikscale3 deleted the rohit/S3EN-2799-langchain_google_genai-support branch September 11, 2024 20:16
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 <[email protected]>

* 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 <[email protected]>

---------

Co-authored-by: Rohit Kadhe <[email protected]>
Co-authored-by: Rohit Kadhe <[email protected]>
Co-authored-by: Ali Waleed <[email protected]>
Co-authored-by: darshit-s3 <[email protected]>
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 <[email protected]>

* 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 <[email protected]>

* Minor bugfix to langchain instrumentation (#348)

* Bugfix

* bump version

---------

Co-authored-by: Rohit Kadhe <[email protected]>
Co-authored-by: Rohit Kadhe <[email protected]>
Co-authored-by: Ali Waleed <[email protected]>
Co-authored-by: darshit-s3 <[email protected]>
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 <[email protected]>

* 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 <[email protected]>

* 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 <[email protected]>
Co-authored-by: Rohit Kadhe <[email protected]>
Co-authored-by: Ali Waleed <[email protected]>
Co-authored-by: Karthik Kalyanaraman <[email protected]>
Co-authored-by: Karthik Kalyanaraman <[email protected]>
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 <[email protected]>

* 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 <[email protected]>

* 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 <[email protected]>
Co-authored-by: Karthik Kalyanaraman <[email protected]>
Co-authored-by: darshit-s3 <[email protected]>
Co-authored-by: Karthik Kalyanaraman <[email protected]>
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.

3 participants