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

Chat Copilot Planner Implementation #876

Open
samfranklinm opened this issue Mar 18, 2024 · 6 comments
Open

Chat Copilot Planner Implementation #876

samfranklinm opened this issue Mar 18, 2024 · 6 comments
Labels
enhancement New feature or request planner

Comments

@samfranklinm
Copy link

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
Copy link

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

@glahaye
Copy link
Collaborator

glahaye commented Mar 25, 2024

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
Copy link
Author

samfranklinm commented Mar 28, 2024

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
Copy link

@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 :

  • Call directly the HandleRequest method
  • or creating a plan and displaying it in a message with two choices (execute / deny)
    Then according to the user response, we could call the HandleRequest.
    The plan will then be executed.

@glahaye
Copy link
Collaborator

glahaye commented Apr 27, 2024

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

@glahaye glahaye added enhancement New feature or request and removed triage labels Jul 24, 2024
@zhangzunke
Copy link

Is there any update for the enhancement, thanks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request planner
Projects
None yet
Development

No branches or pull requests

6 participants