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

track langchain usage for Rockset #9229

Merged
merged 2 commits into from Aug 14, 2023

Conversation

kwadhwa18
Copy link
Contributor

@kwadhwa18 kwadhwa18 commented Aug 14, 2023

Add ability to track langchain usage for Rockset. Rockset's new python client allows setting this. To prevent old clients from failing, it ignore if setting throws exception (we can't track old versions)

Tested locally with old and new Rockset python client

cc @baskaryan

@vercel
Copy link

vercel bot commented Aug 14, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

1 Ignored Deployment
Name Status Preview Comments Updated (UTC)
langchain ⬜️ Ignored (Inspect) Visit Preview Aug 14, 2023 11:16pm

@dosubot dosubot bot added Ɑ: doc loader Related to document loader module (not documentation) 🤖:improvement Medium size change to existing code to handle new use-cases labels Aug 14, 2023
@baskaryan baskaryan added the lgtm PR looks good. Use to confirm that a PR is ready for merging. label Aug 14, 2023
@baskaryan
Copy link
Collaborator

thanks @kwadhwa18!

@baskaryan baskaryan merged commit a69cb95 into langchain-ai:master Aug 14, 2023
44 checks passed
@kwadhwa18 kwadhwa18 deleted the kw_telemetry branch August 14, 2023 23:37
zizhong pushed a commit to opaque-systems/langchain that referenced this pull request Aug 18, 2023
Add ability to track langchain usage for Rockset. Rockset's new python
client allows setting this. To prevent old clients from failing, it
ignore if setting throws exception (we can't track old versions)

Tested locally with old and new Rockset python client

cc @baskaryan
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Ɑ: doc loader Related to document loader module (not documentation) 🤖:improvement Medium size change to existing code to handle new use-cases lgtm PR looks good. Use to confirm that a PR is ready for merging.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants