-
Notifications
You must be signed in to change notification settings - Fork 102
[Doc] Emphasize the importance of the prompts and update BatchGrammarMatcher. #468
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
[Doc] Emphasize the importance of the prompts and update BatchGrammarMatcher. #468
Conversation
Signed-off-by: Yuchuan <[email protected]>
0a6c982 to
3127e00
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR adds a best practices notice to the Quick Start documentation, recommending that users describe the expected output structure in their prompts when using constrained decoding.
- Adds a new section highlighting the importance of prompt engineering when using constrained decoding
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Signed-off-by: Yuchuan <[email protected]>
Signed-off-by: Yuchuan <[email protected]>
e1c7804 to
fcf3828
Compare
Signed-off-by: Yuchuan <[email protected]>
fcf3828 to
bcd8ca8
Compare
Ubospica
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. Thanks
The PR emphasizes the importance of describing the constraint in the prompts. It adds a notice about it in the
Quick Startsection, and adds the introduction toBatchGrammarMatcher.