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.

Sign up for GitHub

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

Jump to bottom

BREAKING CHANGES: Workspace feature #484

Draft
wants to merge 123 commits into
base: main
Choose a base branch
from
Draft

BREAKING CHANGES: Workspace feature #484

wants to merge 123 commits into from

Conversation

tonyzhao6
Copy link
Contributor

Reviewer:
Estimate:

Ticket

Fixes: JIRA_TICKET_LINK

Description

Goal

Changes

Future Tasks (optional)

How has this been tested?

To-do before merge (optional)

Checklist

Fill with x for completed.

  • My code follows the style guidelines of this project
  • I have reviewed my own code to ensure good quality
  • I have tested the functionality of my code to ensure it works as intended
  • I have resolved merge conflicts

(Delete any items below that are not relevant)

  • I have updated the automated tests
  • I have updated the scripts in scripts/
  • I have updated the requirements
  • I have updated the README file
  • I have updated affected documentation
  • I have added a blogpost in Latest Updates
  • I have updated the CI/CD scripts in .github/workflows/
  • I have updated the Terraform code

tonyzhao6 added 30 commits January 10, 2025 14:56
Adding termcolor to requirements.
c12c919
Adding new litellm model for chat.
a9eb059
Fixing import path for add dummy data script.
64b0d39
Adding openai/chat litellm config.
d1b0750
Add utils to generate random int32.
81569bc
Adding chat management functionalities.
7a53deb
Adding prompts for ChatHistory.
cb3bc81
Removed zero-shot cot and updated system message passing.
7a11b0e
Updated pre-commit to include types-requests.
bd9dfde
Added chat fallback model.
5a850b3
Added REDIS timeout variable to config.
991a9e9
Updated prompts for ChatHistory.
0a590bf
Passing in paraphrase argument so that paraphrasing can be skipped fo…
7ca49d7 
…r chat histories.
Updat chat management utilities.
6656e9f
Updated prompts for ChatHistory.
e8c0ca8
Added response generation with RAG and chat history.
197edc5
Updated decorators and llm query generation to include chat history. …
13c1b2f 
…Added /chat endpoint. Temporariliy commented out /search endpoint for quick testing.
Updated chat manager functions. Updated parameter name for _ask_llm_a…
57bbf15 
…sync from json to _json.
Updated prompts for spacing.
72ea6b1
Updated prompt and chat management.
9cd02dc
Added utils for generating random int32.
0adec3e
Refactored chat and search endpoints.
ff9da93
Consolidated chat and search endpoints.
ba5e21a
CCs.
6828aa9
Removed termcolor package.
8ee16eb
Adding types-requests to requirements-dev.txt for github workflow.
14d2481
Passing along session ID for QueryResponse.
5873478
Logic shift to query refined template.
eed61dd
CCs.
7b15749
Removing paraphrase argument.
d1a205e
tonyzhao6 added 27 commits February 3, 2025 12:22
Updating workspace BDD tests.
02fad28
Updating workspace BDD tests.
ca69ccb
Merging in frontend changes only for multi-turn conv.
5d90c93
Updating with multi-turn conv frontend PR and pylint fixes.
acb3a93
Adding linting make command.
68bfeff
Merged with topic modeling PR.
5706a1e
Folding in hotfixes to admin_app.
53cd86e
CCs.
f195e76
CCs.
601009b
CCs.
2bfa8d3
CCs.
56a3faf
Folding in hotfixes to admin_app.
390b592
Updated dashboard package for workspace.
601864b
Verified remaining tests.
2591f5e
Updated github workflow for tests. Updated test_urgency_detect.py to …
8fc058c 
…include proper teardown. Updated dashboard filtering logic to point to UrgencyResponseDB instead of ResponseFeedbackDB. CCs.
Updated optional_components for linting and updated httpx dependency …
a4ab171 
…in order to pass github workflow.
Testing reverting back to using type.
d6f49c3
Testing reverting back to using isinstance.
b298c4e
CCs.
dad1043
CCs.
92dc85b
CCs.
e7c8aff
CCs.
ec52e2b
Added accidentally deleted pytest fixture.
37aeed7
Moved archive content test to its own workspace.
8179bd5
login endpoints now return workspace_name in AuthenticatedDetails. lo…
dc8481a 
…gin-workspace now has dependency injection on get_current_user so that access token is required. workspace default quotas changed to env defaults. UserRetrieve now returns list of dicts instead of two separate lists. retrieve_all_users is now retrieve_all_users_in_current_workspace. added get_current_workspace endpoint.
Returning WorkspaceRetrieve after creating workspaces instead of Work…
f6984a9 
…spaceCreate so that workspace_id is available.
Moved login-workspace endpoint to workspace/routers.py and changed to…
1a1021a 
… switch-workspace endpoint due to authentication requirement. Disabled updating workspace quotas on backend.
@tonyzhao6 tonyzhao6 changed the title BREAKING CHANGE: Workspace feature BREAKING CHANGES: Workspace feature Feb 10, 2025
@lickem22 lickem22 force-pushed the tonyzhao6/workspace_feature branch from ef4ac3e to 1a1021a Compare February 11, 2025 06:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sidravi1 sidravi1 Awaiting requested review from sidravi1 sidravi1 will be requested when the pull request is marked ready for review sidravi1 is a code owner

@suzinyou suzinyou Awaiting requested review from suzinyou suzinyou will be requested when the pull request is marked ready for review suzinyou is a code owner

@lickem22 lickem22 Awaiting requested review from lickem22 lickem22 will be requested when the pull request is marked ready for review lickem22 is a code owner

@amiraliemami amiraliemami Awaiting requested review from amiraliemami amiraliemami will be requested when the pull request is marked ready for review amiraliemami is a code owner

@markbotterill markbotterill Awaiting requested review from markbotterill markbotterill will be requested when the pull request is marked ready for review markbotterill is a code owner

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@tonyzhao6