Merged
Conversation
Comment out base response variables and update assertion.
|
Thanks for your contribution! |
Contributor
There was a problem hiding this comment.
Pull Request Overview
该 PR 将两个测试文件中的严格字符串匹配断言改为更灵活的子字符串检查，以适应 AI 模型输出的变化性。
- 将精确的响应字符串比对改为仅检查关键词"人工智能"的存在
- 注释掉而非删除原有的基准响应字符串
Reviewed Changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| tests/ci_use/XPU_45T/run_w4a8.py | 修改测试断言从精确匹配改为关键词检查，注释掉两个基准响应字符串 |
| tests/ci_use/XPU_45T/run_45T.py | 修改测试断言从精确匹配改为关键词检查，注释掉两个基准响应字符串 |
Comment on lines
+22
to
+23
| #base_response_110 = "你好！我是一个基于人工智能技术的助手，可以帮你解答问题、提供建议、聊天或者协助完成各种任务。无论是学习、工作还是生活中的疑问，我都可以尽力提供帮助。😊 你有什么想聊的吗？" | ||
| #base_response_104 = "你好！我是一个人工智能助手，可以帮你解答问题、提供建议、聊天或者完成一些任务。无论是学习、工作还是生活中的疑问，我都可以尽力帮忙哦～有什么需要我做的吗？😊" |
There was a problem hiding this comment.
注释格式不一致。在第22行使用了 # 紧贴代码（无空格），而在第24行的注释 # 非流式对话 中使用了空格分隔。建议在 # 后添加空格以保持一致性：# base_response_110 = ...
Suggested change
| #base_response_110 = "你好！我是一个基于人工智能技术的助手，可以帮你解答问题、提供建议、聊天或者协助完成各种任务。无论是学习、工作还是生活中的疑问，我都可以尽力提供帮助。😊 你有什么想聊的吗？" | |
| #base_response_104 = "你好！我是一个人工智能助手，可以帮你解答问题、提供建议、聊天或者完成一些任务。无论是学习、工作还是生活中的疑问，我都可以尽力帮忙哦～有什么需要我做的吗？😊" | |
| # base_response_110 = "你好！我是一个基于人工智能技术的助手，可以帮你解答问题、提供建议、聊天或者协助完成各种任务。无论是学习、工作还是生活中的疑问，我都可以尽力提供帮助。😊 你有什么想聊的吗？" | |
| # base_response_104 = "你好！我是一个人工智能助手，可以帮你解答问题、提供建议、聊天或者完成一些任务。无论是学习、工作还是生活中的疑问，我都可以尽力帮忙哦～有什么需要我做的吗？😊" |
juncaipeng
pushed a commit
to juncaipeng/FastDeploy
that referenced
this pull request
Nov 10, 2025
* Refactor test_45t by commenting out responses Comment out base response variables and update assertion. * Update run_w4a8.py * Fix assertion syntax in run_45T.py
chang-wenbin
pushed a commit
to chang-wenbin/FastDeploy
that referenced
this pull request
Mar 2, 2026
* Refactor test_45t by commenting out responses Comment out base response variables and update assertion. * Update run_w4a8.py * Fix assertion syntax in run_45T.py
This file contains hidden or 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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Motivation
Modifications
Usage or Command
Accuracy Tests
Checklist
[FDConfig],[APIServer],[Engine],[Scheduler],[PD Disaggregation],[Executor],[Graph Optimization],[Speculative Decoding],[RL],[Models],[Quantization],[Loader],[OP],[KVCache],[DataProcessor],[BugFix],[Docs],[CI],[Optimization],[Feature],[Benchmark],[Others],[XPU],[HPU],[GCU],[DCU],[Iluvatar],[Metax]]pre-commitbefore commit.releasebranch, make sure the PR has been submitted to thedevelopbranch, then cherry-pick it to thereleasebranch with the[Cherry-Pick]PR tag.