-
Notifications
You must be signed in to change notification settings - Fork 1.9k
feat: Easy Install #5461
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
base: main
Are you sure you want to change the base?
feat: Easy Install #5461
Changes from all commits
dc1f3f5
bfb3816
e271633
b851ea1
ef917ae
87b79d8
535c882
b60ca4d
48538e9
d2c6ce4
c352345
a20e7c9
11a0728
058bf59
850f918
e44a636
0d5177d
06ec9f8
264d94c
e905433
8007b37
12ecad0
13a4aa0
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -10,7 +10,7 @@ SKIP_WARM_UP=True | |
|
||
# Always keep these on for Dev | ||
# Logs all model prompts to stdout | ||
LOG_DANSWER_MODEL_INTERACTIONS=True | ||
LOG_ONYX_MODEL_INTERACTIONS=True | ||
# More verbose logging | ||
LOG_LEVEL=debug | ||
|
||
|
@@ -39,8 +39,8 @@ FAST_GEN_AI_MODEL_VERSION=gpt-4o | |
|
||
# For Danswer Slack Bot, overrides the UI values so no need to set this up via UI every time | ||
# Only needed if using DanswerBot | ||
#DANSWER_BOT_SLACK_APP_TOKEN=<REPLACE THIS> | ||
#DANSWER_BOT_SLACK_BOT_TOKEN=<REPLACE THIS> | ||
#ONYX_BOT_SLACK_APP_TOKEN=<REPLACE THIS> | ||
#ONYX_BOT_SLACK_BOT_TOKEN=<REPLACE THIS> | ||
Comment on lines
-42
to
+43
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Beautiful |
||
|
||
|
||
# Python stuff | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -93,7 +93,7 @@ def _is_external_group_sync_due(cc_pair: ConnectorCredentialPair) -> bool: | |
|
||
if cc_pair.access_type != AccessType.SYNC: | ||
task_logger.error( | ||
f"Recieved non-sync CC Pair {cc_pair.id} for external " | ||
f"Received non-sync CC Pair {cc_pair.id} for external " | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Nice catch There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. lmao I wanted to fix a bot comment but I realized this same typo appeared a bunch of times so just fixed it across the board |
||
f"group sync. Actual access type: {cc_pair.access_type}" | ||
) | ||
return False | ||
|
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 long as the onyx name is correctly populated I think this is fine.
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.
Ya it's the default now in the compose script, a lot of things are simplified now