highperfocused/danswer
1
0
Fork 0
mirror of https://github.com/danswer-ai/danswer.git synced 2025-04-05 02:20:23 +02:00
Code Issues Packages Projects Releases Wiki Activity

llm provider causing re render in effect ()

Browse Source 
* llm provider causing re render in effect

* clean

* unused

* k
This commit is contained in:
pablodanswer 2024-11-22 08:53:24 -08:00 committed by GitHub
parent 9819aa977a
commit 67bfcabbc5
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
2 changed files with 2 additions and 3 deletions
web/src
app/chat
ChatPage.tsx
lib
hooks.ts

4
web/src/app/chat/ChatPage.tsx

@ -52,6 +52,7 @@ import {
useLayoutEffect,
useRef,
useState,
useMemo,
} from "react";
import { usePopup } from "@/components/admin/connectors/Popup";
import { SEARCH_PARAM_NAMES, shouldSubmitOnLoad } from "./searchParams";
@ -266,7 +267,6 @@ export function ChatPage({
availableAssistants[0];
const noAssistants = liveAssistant == null || liveAssistant == undefined;
// always set the model override for the chat session, when an assistant, llm provider, or user preference exists
useEffect(() => {
const personaDefault = getLLMProviderOverrideForPersona(
@ -282,7 +282,7 @@ export function ChatPage({
);
}
// eslint-disable-next-line react-hooks/exhaustive-deps
}, [liveAssistant, llmProviders, user?.preferences.default_model]);
}, [liveAssistant, user?.preferences.default_model]);
const stopGenerating = () => {
const currentSession = currentSessionId();

1
web/src/lib/hooks.ts

@ -174,7 +174,6 @@ export function useLlmOverride(
modelName: "",
}
);
const [llmOverride, setLlmOverride] = useState<LlmOverride>(
currentChatSession && currentChatSession.current_alternate_model
? destructureValue(currentChatSession.current_alternate_model)