Skip to content

Copilot Chat: Add Transform Output Skill#1453

Closed
thomasgobin wants to merge 12 commits intomicrosoft:mainfrom
thomasgobin:feature/pr-transform-output-skill
Closed

Copilot Chat: Add Transform Output Skill#1453
thomasgobin wants to merge 12 commits intomicrosoft:mainfrom
thomasgobin:feature/pr-transform-output-skill

Conversation

@thomasgobin
Copy link
Copy Markdown
Contributor

Motivation and Context

Regarding PR #1138 , I have tried a solution with an intermediate skill.

Description

This intermediate skill, called TransformationOutputSkill can prepare/format an output of a step to fit the expected format of another step input.

Contribution Checklist

Comment thread samples/apps/copilot-chat-app/webapi/CopilotChatWebApi.csproj Outdated
@teresaqhoang teresaqhoang added PR: feedback to address Waiting for PR owner to address comments/questions Copilot Chat labels Jun 19, 2023
@thomasgobin thomasgobin requested a review from a team as a code owner June 20, 2023 14:20
@madsbolaris madsbolaris added core plugin Anything related to core plugins planner Anything related to planner or plans and removed samples core plugin Anything related to core plugins labels Jun 27, 2023
@madsbolaris
Copy link
Copy Markdown
Member

Moving to the Semantic Kernel team's project

@teresaqhoang
Copy link
Copy Markdown
Contributor

Hey @thomasgobin, do you plan to complete this PR? We'd love to have this Skill in SK but would like to see some verification and tests before merge

@shawncal shawncal changed the title Add Transform Output Skill Copilot Chat: Add Transform Output Skill Jul 8, 2023
@thomasgobin thomasgobin requested a review from teresaqhoang July 11, 2023 09:19
@TaoChenOSU
Copy link
Copy Markdown
Contributor

Closing this PR. Issue is tracked by #1138. Please resubmit the feature to the kernel core when ready.

@TaoChenOSU TaoChenOSU closed this Jul 19, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

planner Anything related to planner or plans PR: feedback to address Waiting for PR owner to address comments/questions

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

6 participants