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

5858- Conversion from Completion API end point to Chat Completion API… #146

Merged
merged 3 commits into from
Jul 31, 2023
Merged

5858- Conversion from Completion API end point to Chat Completion API… #146

merged 3 commits into from
Jul 31, 2023

Conversation

ArpitaisAn0maly
Copy link
Contributor

Changes:

  • Chatconversation is managed with Messagebuilder Class.
  • Got rid of stuffing of source and Chat History from Prompt. Instead use messagebuilder method to construct history.
  • New folder "core" with two python helper class. One for Message building and one for managing Token count.
  • Few shot prompting for both Keyword search and Response.
  • Token Count for prompt, content and response length is now managed with Token Count and Response Length

@ArpitaisAn0maly ArpitaisAn0maly merged commit db7ef36 into vNext-Dev Jul 31, 2023
2 checks passed
@ArpitaisAn0maly ArpitaisAn0maly deleted the aparmar/5858-ChatCompletionAPI branch July 31, 2023 15:19
lmwilki pushed a commit that referenced this pull request Aug 7, 2023
@ArpitaisAn0maly
Merge pull request #146 from microsoft/aparmar/5858-ChatCompletionAPI
ccd6fc3 
5858- Conversion from Completion API end point to Chat Completion API…
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@dayland dayland dayland approved these changes

@georearl georearl Awaiting requested review from georearl

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

Successfully merging this pull request may close these issues.
2 participants
@ArpitaisAn0maly @dayland