-
Notifications
You must be signed in to change notification settings - Fork 660
【Hackathon 9th No.76】supplementary unit test for XGrammarChecker #4075
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
Conversation
|
Thanks for your contribution! |
因为上一个PR的commit太乱了,所以重新提交。 @kevincheng2 pls review~ thx |
|
coverage ci安装xgrammar 太慢了,会导致ci超时,我已经联系同事把xgrammar直接打到ci镜像中了。这个需要全面的测试一下新增这个环境会不会影响其他功能,稍等几天哈 |
收到!辛苦老师们🫡 |
|
/re-run all-failed |
|
/re-run all-failed |
Objective
Test Results
Analysis
guided_json: Valid schemas pass correctly. Invalid schemas correctly return an error.
guided_grammar: Valid grammar strings generate the corresponding Grammar object. Invalid grammar strings return errors as expected.
guided_json_object: Simplified JSON object patterns are generated correctly.
guided_choice: Valid choice lists are correctly converted into grammar rules. Invalid choices (e.g., containing
None) trigger exceptions.structural_tag: Correct structured tags are parsed successfully. Missing trigger fields trigger a
KeyError.regex: Regular expressions are passed through directly without validation errors.