Sglang l3005 - #1033
Conversation
CLA Signature Guide@A6444 , thanks for your pull request. The following commit(s) are not associated with a signed Contributor License Agreement (CLA).
To sign CLA, click here. To check if your email is configured correctly, refer to the FAQs. Once you've signed the CLA or updating your email, please comment |
CLA Signature Guide@A6444 , thanks for your pull request. The following commit(s) are not associated with a signed Contributor License Agreement (CLA).
To sign CLA, click here. To check if your email is configured correctly, refer to the FAQs. Once you've signed the CLA or updating your email, please comment |
CLA Signature Guide@A6444 , thanks for your pull request. The following commit(s) are not associated with a signed Contributor License Agreement (CLA).
To sign CLA, click here. To check if your email is configured correctly, refer to the FAQs. Once you've signed the CLA or updating your email, please comment |
CLA Signature Guide@A6444 , thanks for your pull request. The following commit(s) are not associated with a signed Contributor License Agreement (CLA).
To sign CLA, click here. To check if your email is configured correctly, refer to the FAQs. Once you've signed the CLA or updating your email, please comment |
CLA Signature Guide@A6444 , thanks for your pull request. The following commit(s) are not associated with a signed Contributor License Agreement (CLA).
To sign CLA, click here. To check if your email is configured correctly, refer to the FAQs. Once you've signed the CLA or updating your email, please comment |
There was a problem hiding this comment.
The model used in the test case is Qwen3.6, and the test case file name is Qwen3_5.
There was a problem hiding this comment.
The class variable kl_div_thres belongs to KLDivergenceMixin, so why isn't it inheriting from KLDivergenceMixin?
There was a problem hiding this comment.
The product does not support KLDivergenceMixin at present,and the leader has confirmed it
| # Test configuration constants | ||
| BASE_MODEL = LLAMA_2_7B_HF_WEIGHTS_PATH | ||
| LORA_PATHS = [LLAMA_2_7B_LORA_TRIVIA_QA_WEIGHTS_PATH] | ||
| LORA_BACKEND = "triton" |
There was a problem hiding this comment.
LORA_BACKEND = "ascend" is unstable to the Dense model(LLMA-2-7B),resulting in the incorrect loading of LoRA adapter of some prompt.Triton backend verfied to be stable.
| engine.shutdown() | ||
| self.assertEqual(out1, out2) | ||
|
|
||
| def test_8_engine_async_encode_consistency(self): |
There was a problem hiding this comment.
Test_7 has been deleted,This has been aligned with PL.
There was a problem hiding this comment.
The necessary comments has been added
CLA Signature Guide@A6444 , thanks for your pull request. The following commit(s) are not associated with a signed Contributor License Agreement (CLA).
To sign CLA, click here. To check if your email is configured correctly, refer to the FAQs. Once you've signed the CLA or updating your email, please comment |
CLA Signature Guide@A6444 , thanks for your pull request. The following commit(s) are not associated with a signed Contributor License Agreement (CLA).
To sign CLA, click here. To check if your email is configured correctly, refer to the FAQs. Once you've signed the CLA or updating your email, please comment |
CLA Signature Guide@A6444 , thanks for your pull request. The following commit(s) are not associated with a signed Contributor License Agreement (CLA).
To sign CLA, click here. To check if your email is configured correctly, refer to the FAQs. Once you've signed the CLA or updating your email, please comment |
CLA Signature Guide@A6444 , thanks for your pull request. The following commit(s) are not associated with a signed Contributor License Agreement (CLA).
To sign CLA, click here. To check if your email is configured correctly, refer to the FAQs. Once you've signed the CLA or updating your email, please comment |
| repo_type="dataset", | ||
| ) | ||
|
|
||
| os.environ["SGLANG_SKIP_REMOTE_CODE_REGISTRATION"] = "true" |
CLA Signature Guide@A6444 , thanks for your pull request. The following commit(s) are not associated with a signed Contributor License Agreement (CLA).
To sign CLA, click here. To check if your email is configured correctly, refer to the FAQs. Once you've signed the CLA or updating your email, please comment |
CLA Signature Guide@A6444 , thanks for your pull request. The following commit(s) are not associated with a signed Contributor License Agreement (CLA).
To sign CLA, click here. To check if your email is configured correctly, refer to the FAQs. Once you've signed the CLA or updating your email, please comment |
Motivation
Modifications
Accuracy Tests
Speed Tests and Profiling
Checklist
Review and Merge Process
/tag-and-rerun-ci,/tag-run-ci-label,/rerun-failed-ciCI States
Latest PR Test (Base): ❌ Missing
run-cilabel -- add it to run CI tests.Latest PR Test (Extra): ❌ Blocked --
run-ciis required first.