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

Create ChatAnyscale #8770

Merged
merged 17 commits into from
Aug 7, 2023
Merged

Conversation

joshuasundance-swca
Copy link
Contributor

@vercel
Copy link

vercel bot commented Aug 4, 2023

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

Name Status Preview Comments Updated (UTC)
langchain ✅ Ready (Inspect) Visit Preview 💬 Add feedback Aug 7, 2023 4:11am

@dosubot dosubot bot added the 🤖:enhancement A large net-new component, integration, or chain. Use sparingly. The largest features label Aug 4, 2023
Copy link
Contributor

@waleedkadous waleedkadous left a comment

Choose a reason for hiding this comment

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

Thank you for putting this together!

Drop me an e-mail at mwk@anyscale.com and we'll send you some swag ;-)

libs/langchain/langchain/chat_models/anyscale.py Outdated Show resolved Hide resolved
Copy link
Contributor

@hwchase17 hwchase17 left a comment

Choose a reason for hiding this comment

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

would LOVE an example notebook (or adding to existing anyscale one)

@joshuasundance-swca
Copy link
Contributor Author

The "Migrate from OpenAI" page on Anyscale Endpoints gives us a sort of roadmap for improvement of this class, but it is functional and it logs to Langsmith right now. Sorry there are so many commits, I had to align my dev process with the formatting and stuff. :)

@baskaryan
Copy link
Collaborator

lgtm, any additional comments @waleedkadous?

@waleedkadous
Copy link
Contributor

LGTM!

@tonyabracadabra
Copy link
Contributor

This is great! Hope we can have js integration soon

@baskaryan
Copy link
Collaborator

awesome, thanks @joshuasundance-swca!!

@baskaryan baskaryan merged commit 7fc07ba into langchain-ai:master Aug 7, 2023
@tonyabracadabra
Copy link
Contributor

Hey @waleedkadous any thoughts on supporting langchain-ai/langchainjs#2195?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🤖:enhancement A large net-new component, integration, or chain. Use sparingly. The largest features
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants