Skip to content
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

Open
wants to merge 59 commits into
base: main
Choose a base branch
from
Open

Conversation

annalimm
Copy link
Collaborator

No description provided.

annalimm and others added 30 commits February 25, 2025 11:52
@annalimm annalimm requested a review from dani2112 March 12, 2025 13:51
Copy link
Collaborator

@dani2112 dani2112 left a 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/
Copy link
Collaborator

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 || []}
Copy link
Collaborator

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 || []}
Copy link
Collaborator

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';
Copy link
Collaborator

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}) => {
Copy link
Collaborator

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

"dompurify": "^3.2.4",
"esbuild": "^0.24.2",
Copy link
Collaborator

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

@@ -0,0 +1,2 @@
declare module 'sbd'
Copy link
Collaborator

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',
Copy link
Collaborator

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

Copy link
Collaborator

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
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

accidentally generated?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants