forked from microsoft/semantic-kernel
-
Notifications
You must be signed in to change notification settings - Fork 0
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Remove PlannerSkill and add SequentialPlanner (microsoft#523)
### Motivation and Context This pull request contains a series of changes that refactor and improve the planning module and tests in the SemanticKernel project. The planning module is responsible for creating and executing plans based on natural language prompts and available skills. The changes aim to simplify the code, enhance the functionality, and increase the test coverage and quality of the planning module. ### Description - Add a new SequentialPlanner class that uses semantic function to generate plans from natural language prompts - Add a new SequentialPlanParser class that parses the XML output of the semantic function and converts it into a Plan object - **Remove PlannerSkill** - Rename PlannerConfig class that contains common configuration options for planner instances, such as the maximum number of tokens, the relevancy threshold, and the excluded or included functions - Refactor the Plan class to make it more consistent and robust, and add support for serialization, named outputs, and nested plans - Add unit tests for the SequentialPlanner, SequentialPlanParser, and Plan classes and refactor the existing planning tests to use mock functions and skills. - Fix a failing web skill test by skipping it due to inconsistent Bing search results
- Loading branch information
1 parent
a73bf9a
commit 77259db
Showing
43 changed files
with
1,996 additions
and
4,352 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
267 changes: 0 additions & 267 deletions
267
dotnet/src/SemanticKernel.IntegrationTests/CoreSkills/PlannerSkillTests.cs
This file was deleted.
Oops, something went wrong.
Oops, something went wrong.