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.

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

[241][Copilot] Marketing Text Improvements - Functions and GPT4o #1573

Merged
merged 9 commits into from
Jul 19, 2024

Conversation

qasimikram
Copy link
Contributor

@qasimikram qasimikram commented Jul 15, 2024

Summary

Improvements done to marketing text feature by introduction functions and upgrading the model to gpt4o. The changes were merged into main as part of this PR #1275

This PR backports all the commits related to functions/tools that needs to be backported to make the above changes work in 24. Following are the list of commits that were cherry-picked in the same order:

  1. [Copilot] Add ClearTools and tests #824
  2. [Copilot] Improve API of Function Calling #882
  3. [Copilot] Minor changes to the internals of OperationResponse and FunctionResponse #928
  4. Fix return value for ProcessFunctionCall #945
  5. [Copilot No. Series] Support of JSON mode for chat completion and Tools Calling Results in Message History #719
  6. Model Upgrade and Function Calling added #1275

Work Item(s)

Fixes # AB#521281

darjoo and others added 6 commits July 15, 2024 14:09
<!-- Thank you for submitting a Pull Request. If you're new to
contributing to BCApps please read our pull request guideline below
* https://github.com/microsoft/BCApps/Contributing.md
-->
#### Summary <!-- Provide a general summary of your changes -->
Add tests for deleting Tools and an additional API for clearing all
tools without having to loop through.

#### Work Item(s) <!-- Add the issue number here after the #. The issue
needs to be open and approved. Submitting PRs with no linked issues or
unapproved issues is highly discouraged. -->
Fixes
[AB#522487](https://dynamicssmb2.visualstudio.com/1fcb79e7-ab07-432a-a3c6-6cf5a88ba4a5/_workitems/edit/522487)
…ctionResponse (#928)

<!-- Thank you for submitting a Pull Request. If you're new to
contributing to BCApps please read our pull request guideline below
* https://github.com/microsoft/BCApps/Contributing.md
-->
#### Summary <!-- Provide a general summary of your changes -->
Minor changes to the internals of OperationResponse and
FunctionResponse.
Conforming closer to facade principals 

#### Work Item(s) <!-- Add the issue number here after the #. The issue
needs to be open and approved. Submitting PRs with no linked issues or
unapproved issues is highly discouraged. -->
Fixes
[AB#493212](https://dynamicssmb2.visualstudio.com/1fcb79e7-ab07-432a-a3c6-6cf5a88ba4a5/_workitems/edit/493212)
#### Summary <!-- Provide a general summary of your changes -->

Fix return value for procedure `ProcessFunctionCall`. Also explicit
return value false

#### Work Item(s) <!-- Add the issue number here after the #. The issue
needs to be open and approved. Submitting PRs with no linked issues or
unapproved issues is highly discouraged. -->
Fixes
[AB#525473](https://dynamicssmb2.visualstudio.com/1fcb79e7-ab07-432a-a3c6-6cf5a88ba4a5/_workitems/edit/525473)
…ls Calling Results in Message History (#719)

Enable JSON mode for chat completion, requiring "json" in messages, and
maintain tool call history. This update prepares the system to return
valid JSON objects as chat completions, facilitating better integration
with tools and enhancing the user experience.

<!-- Thank you for submitting a Pull Request. If you're new to
contributing to BCApps please read our pull request guideline below
* https://github.com/microsoft/BCApps/Contributing.md
-->
#### Summary 
1. Adding Json Mode support to the completion
2. When AOAI respond with tool calling message, this message is saved in
the message history, marked as tool. New function implemented to parse
tool message response. When tool was called we can now add the tool
result back to the chat message history in a correct format, so AOAI
will respond back taking into account tool result.

#### Work Item(s)
Fixes #718



Fixes
[AB#503148](https://dynamicssmb2.visualstudio.com/1fcb79e7-ab07-432a-a3c6-6cf5a88ba4a5/_workitems/edit/503148)

---------

Co-authored-by: Jesper Schulz-Wedde <[email protected]>
Co-authored-by: Darrick <[email protected]>
@qasimikram qasimikram changed the title [24X][Copilot] Marketing Text Improvements - Functions and GPT4o [241][Copilot] Marketing Text Improvements - Functions and GPT4o Jul 15, 2024
@github-actions github-actions bot added this to the Version 24.1 milestone Jul 15, 2024
@qasimikram qasimikram requested review from darjoo and msft-sam July 15, 2024 12:19
@qasimikram qasimikram marked this pull request as ready for review July 15, 2024 12:31
@qasimikram qasimikram requested review from a team as code owners July 15, 2024 12:31
darjoo
darjoo previously approved these changes Jul 15, 2024
PredragMaricic
PredragMaricic previously approved these changes Jul 16, 2024
@qasimikram qasimikram dismissed stale reviews from PredragMaricic and darjoo via 450a4e7 July 17, 2024 13:02
@qasimikram qasimikram merged commit e544500 into releases/24.1 Jul 19, 2024
26 checks passed
@qasimikram qasimikram deleted the features/mkt-modelupgrade-improv-gpt4o-241 branch July 19, 2024 12:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants