Skip to content

Commit

Permalink
fix: load model fail overlays thread message error
Browse files Browse the repository at this point in the history
  • Loading branch information
louis-jan committed Feb 2, 2024
1 parent 36ad16f commit 22bfe3c
Show file tree
Hide file tree
Showing 4 changed files with 6 additions and 54 deletions.
4 changes: 4 additions & 0 deletions web/hooks/useSetActiveThread.ts
Original file line number Diff line number Diff line change
Expand Up @@ -8,6 8,8 @@ import {

import { useAtomValue, useSetAtom } from 'jotai'

import { loadModelErrorAtom } from './useActiveModel'

import { extensionManager } from '@/extension'
import { setConvoMessagesAtom } from '@/helpers/atoms/ChatMessage.atom'
import {
Expand All @@ -24,6 26,7 @@ export default function useSetActiveThread() {
const setThreadMessage = useSetAtom(setConvoMessagesAtom)
const setThreadModelParams = useSetAtom(setThreadModelParamsAtom)
const setIsGeneratingResponse = useSetAtom(isGeneratingResponseAtom)
const setLoadModelError = useSetAtom(loadModelErrorAtom)

const setActiveThread = async (thread: Thread) => {
if (activeThreadId === thread.id) {
Expand All @@ -32,6 35,7 @@ export default function useSetActiveThread() {
}

setIsGeneratingResponse(false)
setLoadModelError(undefined)
events.emit(InferenceEvent.OnInferenceStopped, thread.id)

// load the corresponding messages
Expand Down
6 changes: 2 additions & 4 deletions web/screens/Chat/ChatBody/index.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 25,6 @@ const ChatBody: React.FC = () => {
const messages = useAtomValue(getCurrentChatMessagesAtom)
const { downloadedModels } = useGetDownloadedModels()
const { setMainViewState } = useMainViewState()
const loadModelError = useAtomValue(loadModelErrorAtom)

if (downloadedModels.length === 0)
return (
Expand Down Expand Up @@ -86,9 85,8 @@ const ChatBody: React.FC = () => {
message.content.length > 0) && (
<ChatItem {...message} key={message.id} />
)}
{!loadModelError &&
(message.status === MessageStatus.Error ||
message.status === MessageStatus.Stopped) &&
{(message.status === MessageStatus.Error ||
message.status === MessageStatus.Stopped) &&
index === messages.length - 1 && (
<ErrorMessage message={message} />
)}
Expand Down
48 changes: 0 additions & 48 deletions web/screens/Chat/LoadModelErrorMessage/index.tsx

This file was deleted.

2 changes: 0 additions & 2 deletions web/screens/Chat/index.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -70,7 70,6 @@ const ChatScreen: React.FC = () => {
const activeModel = useAtomValue(activeModelAtom)

const isGeneratingResponse = useAtomValue(isGeneratingResponseAtom)
const loadModelError = useAtomValue(loadModelErrorAtom)

const { getRootProps, isDragReject } = useDropzone({
noClick: true,
Expand Down Expand Up @@ -213,7 212,6 @@ const ChatScreen: React.FC = () => {
)}

{activeModel && isGeneratingResponse && <GenerateResponse />}
{loadModelError && <LoadModelErrorMessage />}
<ChatInput />
</div>
</div>
Expand Down

0 comments on commit 22bfe3c

Please sign in to comment.