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

Copilotchat: Don't call the planner if there are no functions available #885

Merged
merged 2 commits into from
May 9, 2023

Conversation

adrianwyatt
Copy link
Contributor

Motivation and Context

The planner should not get called if there are no functions available to it.

Description

CopilotChatPlanner now checks if there are functions available before attempting to create a plan. If there are no functions available it returns an empty plan without interacting with the AI.

@adrianwyatt adrianwyatt self-assigned this May 9, 2023
@adrianwyatt adrianwyatt added enhancement PR: ready for review All feedback addressed, ready for reviews labels May 9, 2023
@adrianwyatt adrianwyatt enabled auto-merge (squash) May 9, 2023 17:58
@adrianwyatt adrianwyatt merged commit 01fe119 into microsoft:main May 9, 2023
@adrianwyatt adrianwyatt deleted the skipplanner-nofn branch May 10, 2023 17:41
dehoward pushed a commit to lemillermicrosoft/semantic-kernel that referenced this pull request Jun 1, 2023
…le (microsoft#885)

### Motivation and Context
The planner should not get called if there are no functions available to
it.

### Description
CopilotChatPlanner now checks if there are functions available before
attempting to create a plan. If there are no functions available it
returns an empty plan without interacting with the AI.
golden-aries pushed a commit to golden-aries/semantic-kernel that referenced this pull request Oct 10, 2023
…le (microsoft#885)

### Motivation and Context
The planner should not get called if there are no functions available to
it.

### Description
CopilotChatPlanner now checks if there are functions available before
attempting to create a plan. If there are no functions available it
returns an empty plan without interacting with the AI.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
PR: ready for review All feedback addressed, ready for reviews
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants