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

.Net: Fixing Example48_GroundednessChecks + Tuning Variable usage in Handlebars Planner #4063

Merged
merged 4 commits into from
Dec 7, 2023

Conversation

teresaqhoang
Copy link
Contributor

Motivation and Context

This PR addresses bug #3837, where the plan didn't produce expected Groundedness result.

Fix was to add the GroundingText to the ask (used as the Goal in the planner).

Description

PR also contains small fixes to label parameters in relevant semantic functions as required + some tuning of the CreatePlan prompt to emphasize proper variable usage.

Contribution Checklist

@teresaqhoang teresaqhoang self-assigned this Dec 7, 2023
@teresaqhoang teresaqhoang requested a review from a team as a code owner December 7, 2023 00:44
@shawncal shawncal added .NET Issue or Pull requests regarding .NET code samples labels Dec 7, 2023
@github-actions github-actions bot changed the title Fixing Example48_Example48_GroundednessChecks .Net: Fixing Example48_Example48_GroundednessChecks Dec 7, 2023
@teresaqhoang teresaqhoang changed the title .Net: Fixing Example48_Example48_GroundednessChecks .Net: Fixing Example48_GroundednessChecks + Tuning Variable usage in Handlebars Planner Dec 7, 2023
@teresaqhoang teresaqhoang added this pull request to the merge queue Dec 7, 2023
Merged via the queue into microsoft:main with commit 8e820b6 Dec 7, 2023
18 checks passed
@teresaqhoang teresaqhoang deleted the fixing-kernel-example-48 branch December 7, 2023 18:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
.NET Issue or Pull requests regarding .NET code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

.Net: Kernel Syntax Example 48: Plan didn't produce the expected output
5 participants