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

chatapi: general refactor (fixes #7207) #7208

Merged
merged 16 commits into from
Oct 17, 2023
Merged

chatapi: general refactor (fixes #7207) #7208

merged 16 commits into from
Oct 17, 2023

Conversation

Mutugiii
Copy link
Member

@Mutugiii Mutugiii commented Aug 4, 2023 •
edited
Loading

Fixes #7207

  • Refactor chatapi to to improve code and make it more modularized
  • Update the accepted data to accept more than the prompt content: Accept user, time, team-related data
  • Implement functionality to continue a conversation in the chatapi
  • Introduce new function to chat without saving - for use in the course rating & optional saving
  • Support endpoint changes in the chat frontend
  • Improved error handling
  • Fix context_length exceeded error

@Mutugiii Mutugiii requested review from lmmrssa and dogi August 4, 2023 14:20
@Mutugiii Mutugiii linked an issue Aug 4, 2023 that may be closed by this pull request
chat: allow for conversation continuation #7207
Closed
@Mutugiii Mutugiii mentioned this pull request Aug 31, 2023
Open
@Okuro3499 Okuro3499 self-requested a review October 17, 2023 19:10
@dogi dogi changed the title chatapi restructure (fixes #7207) chatapi: general refactor (fixes #7207) Oct 17, 2023
@dogi dogi merged commit 59876ae into master Oct 17, 2023
11 of 13 checks passed
@Mutugiii Mutugiii mentioned this pull request Oct 19, 2023
Merged
@dogi dogi deleted the 7207-chat-conversation-continuation branch October 19, 2023 13:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@dogi dogi dogi approved these changes

@Okuro3499 Okuro3499 Okuro3499 approved these changes

@lmmrssa lmmrssa Awaiting requested review from lmmrssa

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

Successfully merging this pull request may close these issues.

chat: allow for conversation continuation
3 participants
@Mutugiii @dogi @Okuro3499