Feature/add vertex ai #713
Conversation
…ect credentials. Update GoogleVertexAIEmbedding to handle both cases
| @@ -0,0 +1,117 @@ | |||
| import { ICommonObject, INode, INodeData, INodeParams } from '../../../src/Interface' | |||
There was a problem hiding this comment.
can we keep the file name same camecase? GoogleVertextAI.ts
There was a problem hiding this comment.
Sure!
thank you for pointing out!
| @@ -0,0 +1,115 @@ | |||
| import { ICommonObject, INode, INodeData, INodeParams } from '../../../src/Interface' | |||
There was a problem hiding this comment.
can we rename to file and folder name to ChatGoogleVertextAI, just to prevent same name as the one in llms folder
|
last one would be a video of using GoogleEmbeddings and ChatVertext for ConversationalRetrievalQAChain would be awesome, will be shown on Twitter demo when push to new release ! |
|
@HenryHengZJ here! thank you for sharing my work with Demo video! |
|
@Yongtae723 thanks so much for this awesome contribution! Will be using this immediately 🙏 |
definitely! do you have a high resolution video in google drive like the one you shared last time? |
|
thank you! you can find here |
Do Vertex AI support stream with chromadb (typing effect)? |
|
Hi @kientv from my understanding, LangchainJS right now try to implement stream on Vertex AI. langchain-ai/langchainjs#2625 |
|
Thanks, I want it to speed up flowise, because It need more than 5s to response. Seem that they already merged. |
|
alright! |
|
Oh you mean langchain-ai/langchain#10470 ? Anyway, I will！ |
|
Oh wait, I confused because PR you send me is python libraly. Flowise is using langchain does it make sense to you? |
|
Thanks you, |
|
Look like that they already merged :) |
Feature/add vertex ai
this pr is combine of #679, #680 and #681.
thank you @HenryHengZJ @chungyau97 for long discussion!
I finally make good PR for adding Vertex AI