microsoft / chat-copilot

MIT License
1.92k stars 658 forks source link

Chat Copilot Planner Implementation #876

Open samfranklinm opened 3 months ago

samfranklinm commented 3 months ago

Hello, I noticed that the previous implementations of a planner have been removed and sk directly invokes functions from internal or external plugins. Is there any plan to implement either the Function-Calling Stepwise or the Handlebars Planner in the Chat Copilot? If not, is there an alternative planned other than the aforementioned?

Thanks!

daleche-sh commented 3 months ago

My team experienced the same issue recently, any recommendation for such scenarios?

glahaye commented 3 months ago

I am not sure exactly where we sit with regards to putting planners back into Chat Copilot using post 1.0 versions of Semantic Kernel. I'll try to find out.

Are there specific things you are trying to achieve which are no longer possible?

samfranklinm commented 3 months ago

It was nice to see/approve/disapprove the plan before execution. This way, I felt that I had more control over the execution because the plan was not always perfectly executed i.e. certain functions from external plugins were not correctly lined up (at least that was the case with the Sequential Planner). Now, the AutoInvoke feature handles the creation and execution of a plan somewhere I don't have access and that too isn't perfect as it misses a few a functions along the way as well. So, I wanted to try out Handlebars (specifically) this time around within the context of this app, allowing me to be able edit/reuse/etc. a plan at the very least.

tdechanterac commented 2 months ago

@glahaye I thought about this and came out with the follwing idea. I tried it, and can do a PR if you want to have a look at it.

First we would have to add a boolean setting AlwayShowPlan (name need to be adjusted). Then according to this settings we could either :

glahaye commented 2 months ago

@tdechanterac I would love to see that PR, yes!