-
Notifications
You must be signed in to change notification settings - Fork 706
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
Upgrade to Semantic Kernel 1.4 #817
Conversation
Thanks!!! |
Do you remove planner's kernel? |
Yes. The idea is that OpenAI's function-calling can figure out when to call functions without requiring a plan to do so. Let me know if you think the lack of explicit planner will be a problem. |
@glahaye I have tried to use the Karna and Websercher plugins, but since the plan does not activate, they do not launch these and OpenAI responds directly. Is this the expected workflow? I had to correct some exceptions that were thrown when trying to use the chat. For example, the duplication of the chatid in the list. Resolved using last commit. Thank you very much because thanks to this project I can understand much more about the functioning of SK. |
I am trying to fix the automatic function-calling today. When it works, plugins should automatically be invoked when they are relevant.
If you encounter problems, please feel free to enter them in this PR so that I can address them.
My pleasure! |
I am trying to use the websearcher plugin that has the solution, but it seems that it is not able to invoke it. Before migrating to SK 1.4 using the planner, I made the request. |
@ezar I'll see what I can do about the websearcher plugin. I'm not officially given any time for this project anymore so it might take a bit :/ |
### Motivation and Context Chat Copilot hasn't been upgraded to SK 1 and this hampers development. ### Description Upgrade to SK 1.4 ### Contribution Checklist - [ ] The code builds clean without any errors or warnings - [ ] The PR follows the [Contribution Guidelines](https://github.com/microsoft/chat-copilot/blob/main/CONTRIBUTING.md) and the [pre-submission formatting script](https://github.com/microsoft/chat-copilot/blob/main/CONTRIBUTING.md#development-scripts) raises no violations - [ ] All unit tests pass, and I have added new tests where possible - [ ] I didn't break anyone 😄
### Motivation and Context Chat Copilot hasn't been upgraded to SK 1 and this hampers development. ### Description Upgrade to SK 1.4 ### Contribution Checklist - [ ] The code builds clean without any errors or warnings - [ ] The PR follows the [Contribution Guidelines](https://github.com/microsoft/chat-copilot/blob/main/CONTRIBUTING.md) and the [pre-submission formatting script](https://github.com/microsoft/chat-copilot/blob/main/CONTRIBUTING.md#development-scripts) raises no violations - [ ] All unit tests pass, and I have added new tests where possible - [ ] I didn't break anyone 😄
Motivation and Context
Chat Copilot hasn't been upgraded to SK 1 and this hampers development.
Description
Upgrade to SK 1.4. It has OpenAI's function calling working but, other than that, no planning functionality.
Decision to remove planners:
Contribution Checklist