-
Notifications
You must be signed in to change notification settings - Fork 14
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
added reranker id in the config of chat bot #52
Conversation
|
src/components/ChatView.tsx
Outdated
@@ -56,6 +56,11 @@ export interface Props { | |||
|
|||
// Defines the name of the summary prompt. Defaults to "vectara-summary-ext-v1.2.0". | |||
summaryPromptName?: string; | |||
|
|||
//Define the reranker Id to be used , Defaults to "272725718" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nit: Can we add a space after the "//" please?
src/types.ts
Outdated
@@ -95,3 +95,5 @@ export type ChatTurn = { | |||
results: DeserializedSearchResult[]; | |||
factualConsistencyScore?: number; | |||
}; | |||
|
|||
export type RerankerIds = 272725717 | 272725719 | 272725718 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This should be singular: RerankerId
src/useChat.ts
Outdated
@@ -68,7 +75,7 @@ export const useChat = ({ | |||
queryValue: query, | |||
rerank: true, | |||
rerankNumResults: 7, | |||
rerankerId: 272725718, | |||
rerankerId: rerankerId, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nit, this can be simply rerankerId
. JS will infer that we're assigning a variable of this name to a field of the same name. This comment applies in a few other places.
No description provided.