-
Notifications
You must be signed in to change notification settings - Fork 2
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
Feature/answer explainability #96
base: main
Are you sure you want to change the base?
Conversation
…s, first steps in highlighting
…ntermediate result)
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.
see comments. focus on integration for now :-)
.gitignore
Outdated
@@ -13,3 +13,7 @@ dummy-package/ | |||
.vscode/ | |||
|
|||
examples/llama-index/frontend/rag-ui/ | |||
|
|||
pdfs/ |
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.
is the global gitignore the right place for this or is there one closer to the pdfs folder? make clear which pdf fodler is ignored here
@@ -199,6 +199,7 @@ const ChatWindow: React.FC<ChatWindowProps> = ({ | |||
markdown={markdown} | |||
showCopy={showCopy} | |||
showFeedback={showFeedback} | |||
ideas={msg.ideas || []} |
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.
as citation mechanism is more generic consider more generic name for ideas. maybe simply chunks?
@@ -214,6 +215,7 @@ const ChatWindow: React.FC<ChatWindowProps> = ({ | |||
icon={assistantIcon || undefined} | |||
markdown={markdown} | |||
isStreaming={true} | |||
ideas={currentStream.ideas || []} |
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.
does current stream have the idea prop? verify ans also rename prop to chunks or so
@@ -8,6 +8,8 @@ import "./AssistantMarkdownContent.css"; | |||
import {ClipboardIcon, ClipboardDocumentIcon} from "@heroicons/react/24/outline"; | |||
import {scaleFontSize} from "../../utils/scaleFontSize.tsx"; | |||
import { MessageFeedback } from "./MessageFeedback.tsx"; | |||
import { HighlightedIdea } from '../../types'; |
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.
rename that type to something more generic as well
@@ -27,7 +29,7 @@ interface AssistantMarkdownContentProps { | |||
* | |||
* @internal | |||
*/ | |||
const AssistantMarkdownContent: React.FC<AssistantMarkdownContentProps> = ({content, style}) => { | |||
export const AssistantMarkdownContent: React.FC<AssistantMarkdownContentProps> = ({content, style}) => { |
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.
why is export added? valid reason, else remove
lexio/package.json
Outdated
"dompurify": "^3.2.4", | ||
"esbuild": "^0.24.2", |
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.
where does esbuild come from here? is already installed i guess in dev deps
lexio/src/sbd.d.ts
Outdated
@@ -0,0 +1,2 @@ | |||
declare module 'sbd' |
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.
why?
], | ||
build: { | ||
copyPublicDir: false, | ||
outDir: 'dist', |
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.
comment what you changed here
package-lock.json
Outdated
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.
accidentally generated?
package.json
Outdated
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.
accidentally generated?
No description provided.