Skip to content

Conversation

@obinnascale3
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

karthikscale3
karthikscale3 previously approved these changes Feb 25, 2025
The PineconeInstrumentation class represents the Pinecone instrumentation"""

def instrumentation_dependencies(self) -> Collection[str]:
return ["pinecone-client >= 3.1.0"]
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

update version here


def instrumentation_dependencies(self) -> Collection[str]:
return ["pinecone-client >= 3.1.0"]
return ["pinecone >= 3.1.0"]
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

here aswell

@obinnascale3 obinnascale3 merged commit 4bfebd6 into development Feb 26, 2025
4 checks passed
@obinnascale3 obinnascale3 deleted the obinna/S3EN-1111-update-pinecone-integration branch February 26, 2025 06:44
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.

4 participants