-
Notifications
You must be signed in to change notification settings - Fork 966
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
Paul assistant #1342
base: main
Are you sure you want to change the base?
Paul assistant #1342
Conversation
…e the dropdownlist to a list
…) and display assistant profile in header
backend/chainlit/__init__.py
Outdated
@@ -402,6 +427,9 @@ def acall(self): | |||
"TaskStatus", | |||
"Video", | |||
"ChatSettings", | |||
"AssistantSettings", | |||
# assistant |
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.
remove comment
@@ -0,0 +1,16 @@ | |||
from typing import List, Dict |
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.
if we have multiple assistant file it can be in a dedicated folder/module
backend/chainlit/server.py
Outdated
StaticFiles(directory="public", check_dir=False), | ||
name="public", | ||
) | ||
# app.mount( |
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 this commented?
backend/chainlit/server.py
Outdated
Union[None, User, PersistedUser], Depends(get_current_user) | ||
], | ||
): | ||
# if not current_user: |
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 this commented?
@@ -40,6 +41,7 @@ const Header = memo(() => { | |||
}} | |||
> | |||
<ChatProfiles /> | |||
<SelectedAssistantDisplay /> |
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.
can chat profiles co exist with assistants?
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.
they should not, but it will simply be specified in the README (or/and the doc)
const enableHistory = !!user.accessToken && !!config?.dataPersistence; | ||
|
||
useEffect(() => { | ||
if (isMobile) { | ||
setChatHistoryOpen(false); | ||
} else { | ||
setChatHistoryOpen(enableHistory); | ||
setChatHistoryOpen(true); |
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.
we should use enableHistory and include assistants in it
…assistants features
No description provided.