forked from onyx-dot-app/onyx
-
Notifications
You must be signed in to change notification settings - Fork 0
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
Eea merge #1
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* validate dropdown * validate * update organization * move to utils
) * all done except routing * fixed initial changes * added backend endpoint for duplicating a chat session from Slack * got chat duplication routing done * got login routing working * improved answer handling * finished all checks * finished all! * made sure it works with google oauth * dont remove that lol * fixed weird thing * bad comments
* Change external permissions behavior * fixed behavior * added error handling * LLM the goat * comment * simplify * fixed * done * limits increased * added a ton of logging * uhhhh
…p#3264) * use indexing flag in db for manually trigger indexing * add comment. * only try to release the lock if we actually succeeded with the lock * ensure we don't trigger manual indexing on anything but the primary search settings * comment usage of primary search settings * run check for indexing immediately after indexing triggers are set * reorder fix
* checkpoint * add celery termination of the task * rename to RedisConnectorPermissionSyncPayload, add RedisLock to more places, add get_active_search_settings * rename payload * pretty sure these weren't named correctly * testing in progress * cleanup * remove space * merge fix * three dots animation on Pausing * improve messaging when connector is stopped or killed and animate buttons --------- Co-authored-by: Richard Kuo <[email protected]>
* Properly find context window for ollama llama * Better ollama support + upgrade litellm * Ugprade OpenAI as well * Fix mypy
* proper no assistant typing + no assistant modal * updated chat flow * k * updates * update * k * clean up * fix mystery reorg * cleanup * update scroll * default * update logs * push fade * scroll nit * finalize tags * updates * k * various updates * viewport height update * source types update * clean up unused components * minor cleanup * cleanup complete * finalize changes * badge up * update filters * small nit * k * k * address comments * quick unification of icons * minor date range clarity * minor nit * k * update sidebar line * update for all screen sizes * k * k * k * k * rm shs * fix memoization * fix memoization * slack chat * k * k * build org
…3301) * combine for clarity once and for all * remove logs * k
* first pass * k * k * finalize * minor cleanup * k * address * minor typing updates
* added docs example and test * updated docs * needed to make the tests run * updated docs
* hard refresh on auth * k * k * comment for clarity
* add text embedding * post rebase cleanup * fully functional post rebase * rm logs * rm ' * quick clean up * k
* fix typing * add filters display
* remove / update search tests * minor update
* improve error logging on task failure. * add db exception hardening to the indexing watchdog * log on db exception
Co-authored-by: hagen-danswer <[email protected]>
* editable refresh and prune for connectors * add extra validations on pruning/refresh frequency * fix validation * fix icon usage * fix TextFormField error formatting * nit --------- Co-authored-by: Richard Kuo (Danswer) <[email protected]> Co-authored-by: pablodanswer <[email protected]>
* clean up csv prompt + frontend * nit * nit * detect uploading * upload
onyx-dot-app#3417) * better handling around index attempts that don't exist and remove unnecessary index attempt deletions * don't delete index attempts, just update them --------- Co-authored-by: Richard Kuo (Danswer) <[email protected]>
…yx-dot-app#3421) * Mismatch issue of Documents shown and Citation number in text fix When document order presented to LLM differs from order shown to user, wrong doc numbers are cited. Fix: - SearchTool.get_search_result returns now final and initial ranking - initial ranking is passed through a few objects and used for replacement in citation processing Notes: - the citation_num in the CitationInfo() object has not been changed. * PR fixes - linting - removed erroneous tab - added a substitution test case - adjusted original citation extraction use case * Included a key test and * Fixed extra spaces * Updated test documentation Updated: - test_citation_substitution (changed description) - test_citation_processing (removed data only relevant for the substitution)
* migrate hashing to run faster v1 * k
* seed * initial steps * clean up * fully clickable
…ation Silence Slack Permission Sync test flakiness
* update * nit
…nyx-dot-app#3390) * fixed fk issue for slack bot deletion * Added text for empty table and cascade delete for slack bot deletion
* remove slack workspace * update client tokens * fix up * clean up docs * fix up tests
…ents cloud improvements
* let users specify resourcing caps * functioanl resource limits * improve defaults * k * update * update comment + refer to proper resource * self nit * update var names
Enable Vercel Preview
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
[Provide a brief description of the changes in this PR]
How Has This Been Tested?
[Describe the tests you ran to verify your changes]
Accepted Risk (provide if relevant)
N/A
Related Issue(s) (provide if relevant)
N/A
Mental Checklist:
Backporting (check the box to trigger backport action)
Note: You have to check that the action passes, otherwise resolve the conflicts manually and tag the patches.