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.

Sign up for GitHub

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

Jump to bottom

Fix the issue where the parameters passed to VertexAI ignored #5889 #5891

Merged

hwchase17 merged 1 commit into langchain-ai:master from BigicecreamTaken:VertexAI_paramter_not_passed_bugfix

Jun 9, 2023

Contributor

BigicecreamTaken commented Jun 8, 2023

Fixes #5889 and fixes the name of the argument in init_vertexai
@hwchase17
@agola11


          Fix the issue where the parameters passed to VertexAI ignored langcha…

dfeace4

…in-ai#5889

hwchase17 approved these changes

View reviewed changes

Contributor

hwchase17 left a comment

thanks!

hwchase17 added the lgtm label

hwchase17 mentioned this pull request

Fix vertexai.py 'params' generation for 'init_vertexai' #5884

Closed

hwchase17 merged commit 0eb1bc1 into langchain-ai:master

Undertone0809 pushed a commit to Undertone0809/langchain that referenced this pull request


          Fix the issue where the parameters passed to VertexAI ignored langcha…

5df76bf

…in-ai#5889 (langchain-ai#5891)

Fixes langchain-ai#5889 and fixes the name of the argument in init_vertexai
@hwchase17
@agola11

Co-authored-by: Lior Durahly <lior.durahly@superwise.ai>

This was referenced Jun 25, 2023

Zep Authentication #6725

Closed

Zep Authentication #6728

Merged

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

lgtm