modify optimization_debug_options tests by Sugar920 · Pull Request #1002 · Ascend/sglang

Sugar920 · 2026-07-14T09:09:55Z

CI States

Latest PR Test (Base): ❌ Missing run-ci label -- add it to run CI tests.
Latest PR Test (Extra): ❌ Blocked -- run-ci is required first.

…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>

ascend-robot · 2026-07-14T09:10:05Z

CLA Signature Guide

@Sugar920 , thanks for your pull request.

The following commit(s) are not associated with a signed Contributor License Agreement (CLA).

Commit	Reason
a5a8c7f7 fix: change CI suite from debug-...	the email used in the commit is not linked to a signed CLA! please verify that it matches the email you used when signing the 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 /check-cla to revalidate CLA status.

Co-Authored-By: Claude <noreply@anthropic.com>

ascend-robot · 2026-07-14T09:24:04Z

CLA Signature Guide

@Sugar920 , thanks for your pull request.

The following commit(s) are not associated with a signed Contributor License Agreement (CLA).

Commit	Reason
a5a8c7f7 fix: change CI suite from debug-...	the email used in the commit is not linked to a signed CLA! please verify that it matches the email you used when signing the 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 /check-cla to revalidate CLA status.

…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>

ascend-robot · 2026-07-15T03:56:04Z

CLA Signature Guide

@Sugar920 , thanks for your pull request.

The following commit(s) are not associated with a signed Contributor License Agreement (CLA).

Commit	Reason
a5a8c7f7 fix: change CI suite from debug-...	the email used in the commit is not linked to a signed CLA! please verify that it matches the email you used when signing the 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 /check-cla to revalidate CLA status.

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>

ascend-robot · 2026-07-15T03:58:02Z

CLA Signature Guide

@Sugar920 , thanks for your pull request.

The following commit(s) are not associated with a signed Contributor License Agreement (CLA).

Commit	Reason
a5a8c7f7 fix: change CI suite from debug-...	the email used in the commit is not linked to a signed CLA! please verify that it matches the email you used when signing the 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 /check-cla to revalidate CLA status.

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>

ascend-robot · 2026-07-15T03:59:02Z

CLA Signature Guide

@Sugar920 , thanks for your pull request.

The following commit(s) are not associated with a signed Contributor License Agreement (CLA).

Commit	Reason
a5a8c7f7 fix: change CI suite from debug-...	the email used in the commit is not linked to a signed CLA! please verify that it matches the email you used when signing the 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 /check-cla to revalidate CLA status.

Co-Authored-By: Claude <noreply@anthropic.com>

ascend-robot · 2026-07-15T04:09:30Z

CLA Signature Guide

@Sugar920 , thanks for your pull request.

The following commit(s) are not associated with a signed Contributor License Agreement (CLA).

Commit	Reason
a5a8c7f7 fix: change CI suite from debug-...	the email used in the commit is not linked to a signed CLA! please verify that it matches the email you used when signing the 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 /check-cla to revalidate CLA status.

Co-Authored-By: Claude <noreply@anthropic.com>

ascend-robot · 2026-07-15T04:19:19Z

CLA Signature Guide

@Sugar920 , thanks for your pull request.

The following commit(s) are not associated with a signed Contributor License Agreement (CLA).

Commit	Reason
a5a8c7f7 fix: change CI suite from debug-...	the email used in the commit is not linked to a signed CLA! please verify that it matches the email you used when signing the 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 /check-cla to revalidate CLA status.

… 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>

ascend-robot · 2026-07-17T06:11:43Z

CLA Signature Guide

@Sugar920 , thanks for your pull request.

The following commit(s) are not associated with a signed Contributor License Agreement (CLA).

Commit	Reason
a5a8c7f7 fix: change CI suite from debug-...	the email used in the commit is not linked to a signed CLA! please verify that it matches the email you used when signing the 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 /check-cla to revalidate CLA status.

ascend-robot · 2026-07-17T06:16:44Z

CLA Signature Guide

@Sugar920 , thanks for your pull request.

The following commit(s) are not associated with a signed Contributor License Agreement (CLA).

Commit	Reason
a5a8c7f7 fix: change CI suite from debug-...	the email used in the commit is not linked to a signed CLA! please verify that it matches the email you used when signing the 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 /check-cla to revalidate CLA status.

ascend-robot · 2026-07-17T06:25:58Z

CLA Signature Guide

@Sugar920 , thanks for your pull request.

The following commit(s) are not associated with a signed Contributor License Agreement (CLA).

Commit	Reason
a5a8c7f7 fix: change CI suite from debug-...	the email used in the commit is not linked to a signed CLA! please verify that it matches the email you used when signing the 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 /check-cla to revalidate CLA status.

ascend-robot · 2026-07-27T07:19:12Z

CLA Signature Guide

@Sugar920 , thanks for your pull request.

The following commit(s) are not associated with a signed Contributor License Agreement (CLA).

Commit	Reason
a5a8c7f7 fix: change CI suite from debug-...	the email used in the commit is not linked to a signed CLA! please verify that it matches the email you used when signing the CLA.
d115dc90 Refactor CUDA graph batch size l...	the email used in the commit is not linked to a signed CLA! please verify that it matches the email you used when signing the 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 /check-cla to revalidate CLA status.

Co-Authored-By: Claude <noreply@anthropic.com>

ascend-robot · 2026-07-27T08:15:18Z

CLA Signature Guide

@Sugar920 , thanks for your pull request.

The following commit(s) are not associated with a signed Contributor License Agreement (CLA).

Commit	Reason
a5a8c7f7 fix: change CI suite from debug-...	the email used in the commit is not linked to a signed CLA! please verify that it matches the email you used when signing the CLA.
d115dc90 Refactor CUDA graph batch size l...	the email used in the commit is not linked to a signed CLA! please verify that it matches the email you used when signing the 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 /check-cla to revalidate CLA status.

shun8686 · 2026-07-27T12:25:14Z

            f"  Mean TTFT: {t1:.1f} ms (max_bs=1) vs {t8:.1f} ms (max_bs=8)\n"
            f"  P99  TTFT: {p1:.1f} ms (max_bs=1) vs {p8:.1f} ms (max_bs=8)"
        )
        self.assertGreater(t1, 0)


Is this assertion reasonable?

shun8686 · 2026-07-27T12:36:29Z

+                continue
+            parts = [p.strip() for p in line.split("|")]
+            for part in parts:
+                if part.isdigit() and 3 <= len(part) <= 7:


Is it possible to misjudge?

shun8686 · 2026-07-27T12:38:04Z

+    """Testcase: Verify TP server does not create extra NPU processes beyond TP workers.
+
+    [Test Category] Parameter
+    [Test Target] --cuda-graph-backend-decode; --cuda-graph-backend-prefill


Only test disable?

shun8686 · 2026-07-27T12:39:24Z

+        metrics = run_eval(args)
+        print(f"{metrics=}")
+
+        self.assertGreater(metrics["score"], 0.24)


Is 0.24 normal?

shun8686 · 2026-07-27T12:41:39Z

+register_npu_ci(est_time=10800, suite="full-16-npu-a3", nightly=True)
+
+
+class TestPPAccuracy(unittest.TestCase):


Use CustomTestCase?

shun8686 · 2026-07-27T12:43:21Z

+    @classmethod
+    def setUpClass(cls):
+        cls.model = LLAMA_3_1_8B_INSTRUCT_WEIGHTS_PATH
+        cls.base_url = "http://127.0.0.1:23333"


Use DEFAULT_URL_FOR_TEST instead?

shun8686 · 2026-07-27T12:45:08Z

+        time.sleep(4)
+
+
+class TestFixedBugs(unittest.TestCase):


Use CustomTestCase instead?

shun8686 · 2026-07-27T12:49:05Z

+        time.sleep(4)
+
+
+class TestFixedBugs(unittest.TestCase):


The class name isn't appropriate

ascend-robot added the ascend-cla/no label Jul 14, 2026

github-actions Bot added the npu label Jul 14, 2026

Sugar920 changed the title ~~fix: change CI suite from debug-full to full for optimization_debug_options tests~~ modify optimization_debug_options tests Jul 14, 2026

fix: apply black formatting to optimization_debug_options tests

8ff26d1

Co-Authored-By: Claude <noreply@anthropic.com>

cherryblo reviewed Jul 15, 2026

View reviewed changes

Comment thread test/registered/ascend/basic_function/optimization_debug_options/test_npu_cuda_graph_bs.py Outdated

feat: add PP disaggregation and PP single-node tests from PR Ascend#801

891fda2

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>

feat: add full decode graph GSM8K test from PR Ascend#886

47a23e5

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>

fix: rename suite from debug-full-16-npu-a3 to full-16-npu-a3

1250c09

Co-Authored-By: Claude <noreply@anthropic.com>

style: fix isort and black formatting

27a3ebe

Co-Authored-By: Claude <noreply@anthropic.com>

fix: restore suite name debug-full-1-npu-a3 -> full-1-npu-a3

7cb2d93

fix: apply black/isort formatting to pass lint

7ea63c9

Refactor CUDA graph batch size log parsing

d115dc9

fix: apply black formatting to pass lint

6d3708f

Co-Authored-By: Claude <noreply@anthropic.com>

shun8686 reviewed Jul 27, 2026

View reviewed changes

		register_npu_ci(est_time=10800, suite="full-16-npu-a3", nightly=True)


		class TestPPAccuracy(unittest.TestCase):

Uh oh!

Conversation

Sugar920 commented Jul 14, 2026 • edited by github-actions Bot Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

CI States

Uh oh!

ascend-robot commented Jul 14, 2026

CLA Signature Guide

Uh oh!

ascend-robot commented Jul 14, 2026

CLA Signature Guide

Uh oh!

Uh oh!

ascend-robot commented Jul 15, 2026

CLA Signature Guide

Uh oh!

ascend-robot commented Jul 15, 2026

CLA Signature Guide

Uh oh!

ascend-robot commented Jul 15, 2026

CLA Signature Guide

Uh oh!

ascend-robot commented Jul 15, 2026

CLA Signature Guide

Uh oh!

ascend-robot commented Jul 15, 2026

CLA Signature Guide

Uh oh!

ascend-robot commented Jul 17, 2026

CLA Signature Guide

Uh oh!

ascend-robot commented Jul 17, 2026

CLA Signature Guide

Uh oh!

ascend-robot commented Jul 17, 2026

CLA Signature Guide

Uh oh!

ascend-robot commented Jul 27, 2026

CLA Signature Guide

Uh oh!

ascend-robot commented Jul 27, 2026

CLA Signature Guide

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

Sugar920 commented Jul 14, 2026 •

edited by github-actions Bot

Loading