modify optimization_debug_options tests - #1002
Conversation
…ptions tests - test_npu_cuda_graph_bs.py: debug-full-2-npu-a3 -> full-2-npu-a3 - test_npu_embedding_interpolation.py: debug-full-1-npu-a3 -> full-1-npu-a3 - test_npu_no_extra_forked_npu_context.py: debug-full-2-npu-a3 -> full-2-npu-a3 Co-Authored-By: Claude <noreply@anthropic.com>
CLA Signature Guide@Sugar920 , 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 |
Sugar920
changed the title
Co-Authored-By: Claude <noreply@anthropic.com>
CLA Signature Guide@Sugar920 , 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 |
…raph bs tests Remove debug print block in test_npu_cuda_graph_bs.py and stderr assertions in test_npu_msprobe_dump_config.py — both are redundant; the downstream assertions (assertEqual on decode_bs and dump.json file check) already cover the end-to-end validation. Co-Authored-By: Claude <noreply@anthropic.com>
CLA Signature Guide@Sugar920 , 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 |
Cherry-pick test_npu_disaggregation_pp.py and test_npu_pp_single_node.py from debug branch (PR Ascend#801). Co-Authored-By: Claude <noreply@anthropic.com>
CLA Signature Guide@Sugar920 , 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 |
Cherry-pick test_npu_full_decode_graph_gsm8k.py from testcase-npu-model-tokenizer branch (PR Ascend#886). Co-Authored-By: Claude <noreply@anthropic.com>
CLA Signature Guide@Sugar920 , 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 |
Co-Authored-By: Claude <noreply@anthropic.com>
CLA Signature Guide@Sugar920 , 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 |
Co-Authored-By: Claude <noreply@anthropic.com>
CLA Signature Guide@Sugar920 , 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 |
… from debug - test_npu_cuda_graph_bs.py: fix _BS_LOG_RE regex to match NPU prefill CG capture log format num_tokens=[...], migrate to new CG flags - test_npu_model_tokenizer.py: accumulated changes from debug branch Co-Authored-By: Claude <noreply@anthropic.com>
CLA Signature Guide@Sugar920 , 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@Sugar920 , 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@Sugar920 , 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@Sugar920 , 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 |
Co-Authored-By: Claude <noreply@anthropic.com>
CLA Signature Guide@Sugar920 , 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 |
Replace heuristic PID detection with explicit column-index lookup via 'Process id' header. Add psutil.pid_exists() cross-validation to prevent misidentifying memory values (e.g. 4696, 1538) as PIDs. Co-Authored-By: Claude <noreply@anthropic.com>
CLA Signature Guide@Sugar920 , 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 |
Replace two weak assertGreater(x, 0) with a real comparison: max_bs=1 should have higher TTFT than max_bs=8 due to lack of batching. Co-Authored-By: Claude <noreply@anthropic.com>
CLA Signature Guide@Sugar920 , 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 |
4 participants
CI States
Latest PR Test (Base): ❌ Missing
run-cilabel -- add it to run CI tests.Latest PR Test (Extra): ❌ Blocked --
run-ciis required first.