[Lint]Style: Convert `vllm-ascend/` to ruff format(Batch #11)#6176

Merged

wangxiyuan merged 2 commits intovllm-project:mainfrom

MrZ20:lint-ascend-10

Feb 6, 2026

Contributor

MrZ20 commented Jan 23, 2026 •

edited by github-actions bot

Loading

What this PR does / why we need it?

Scope of Changes:

File Path
`vllm_ascend/ops/fused_moe/comm_utils.py`
`vllm_ascend/ops/fused_moe/experts_selector.py`
`vllm_ascend/ops/fused_moe/fused_moe.py`
`vllm_ascend/ops/fused_moe/moe_comm_method.py`
`vllm_ascend/ops/fused_moe/moe_mlp.py`
`vllm_ascend/ops/fused_moe/prepare_finalize.py`
`vllm_ascend/ops/fused_moe/token_dispatcher.py`

Does this PR introduce any user-facing change?

How was this patch tested?

vLLM version: v0.14.0
vLLM main: vllm-project/vllm@d682094

MrZ20 requested review from realliujiaxu and zzzzwwjj as code owners

January 23, 2026 02:55

github-actions bot added the module:ops label

Contributor

github-actions bot commented Jan 23, 2026

👋 Hi! Thank you for contributing to the vLLM Ascend project. The following points will speed up your PR merge:‌‌

A PR should do only one thing, smaller PRs enable faster reviews.
Every PR should include unit tests and end-to-end tests ‌to ensure it works and is not broken by other future PRs.
Write the commit message by fulfilling the PR description to help reviewer and future developers understand.

If CI fails, you can run linting and testing checks locally according Contributing and Testing.

gemini-code-assist bot reviewed

View reviewed changes

Contributor

gemini-code-assist bot left a comment

Code Review

This pull request primarily focuses on reformatting the code to adhere to the ruff linter's style guidelines, as indicated by the PR title. The changes observed across the files are consistently related to code formatting, import organization, and modern Python type hint syntax. No functional changes, bugs, or critical issues were introduced. The changes align well with the stated objective of converting the codebase to the ruff format.

MrZ20 mentioned this pull request

[RFC]: Gradual migration of linting and formatting to Ruff #5845

Open

Potabk added ready ready-for-test labels

github-actions bot added the merge-conflicts label

Contributor

github-actions bot commented Jan 23, 2026

This pull request has conflicts, please resolve those before we can evaluate the pull request.

MrZ20 force-pushed the lint-ascend-10 branch from 1ec2251 to 6357ccb Compare

January 24, 2026 08:19

github-actions bot added merge-conflicts and removed merge-conflicts labels

Contributor

github-actions bot commented Jan 24, 2026

This pull request has conflicts, please resolve those before we can evaluate the pull request.

Potabk removed ready ready-for-test labels

MrZ20 force-pushed the lint-ascend-10 branch from 6357ccb to 83d0c7d Compare

January 25, 2026 06:08

github-actions bot added merge-conflicts and removed merge-conflicts labels

Contributor

github-actions bot commented Jan 26, 2026

This pull request has conflicts, please resolve those before we can evaluate the pull request.

MrZ20 force-pushed the lint-ascend-10 branch from 83d0c7d to cb8eea7 Compare

January 26, 2026 12:17

MrZ20 requested a review from whx-sjtu as a code owner

January 26, 2026 12:17

github-actions bot removed the merge-conflicts label

MrZ20 force-pushed the lint-ascend-10 branch from cb8eea7 to 9c5d535 Compare

January 28, 2026 10:22

Contributor

github-actions bot commented Jan 29, 2026

This pull request has conflicts, please resolve those before we can evaluate the pull request.

github-actions bot added the merge-conflicts label

MrZ20 force-pushed the lint-ascend-10 branch from 9c5d535 to 8735068 Compare

February 1, 2026 14:18

github-actions bot added merge-conflicts and removed merge-conflicts labels

Contributor

github-actions bot commented Feb 1, 2026

This pull request has conflicts, please resolve those before we can evaluate the pull request.

github-actions bot removed the merge-conflicts label

MrZ20 force-pushed the lint-ascend-10 branch 2 times, most recently from b72a714 to 768a438 Compare

February 2, 2026 13:42

github-actions bot added the merge-conflicts label

Contributor

github-actions bot commented Feb 5, 2026

This pull request has conflicts, please resolve those before we can evaluate the pull request.


          all-auto

28a17ae

Signed-off-by: MrZ20 <2609716663@qq.com>

fix

Signed-off-by: MrZ20 <2609716663@qq.com>

fix

Signed-off-by: MrZ20 <2609716663@qq.com>

fix

Signed-off-by: MrZ20 <2609716663@qq.com>

MrZ20 force-pushed the lint-ascend-10 branch from 768a438 to 28a17ae Compare

February 6, 2026 06:53

github-actions bot added merge-conflicts and removed merge-conflicts labels

Contributor

github-actions bot commented Feb 6, 2026

This pull request has conflicts, please resolve those before we can evaluate the pull request.


          Merge branch 'main' into lint-ascend-10

1371a07

Signed-off-by: SILONG ZENG <2609716663@qq.com>

github-actions bot removed the merge-conflicts label

wangxiyuan merged commit 65b7f71 into vllm-project:main

25 checks passed

MrZ20 deleted the lint-ascend-10 branch

February 6, 2026 07:41

845473182 pushed a commit to 845473182/vllm-ascend that referenced this pull request


          Merge branch 'main' of https://github.com/vllm-project/vllm-ascend in…

222b7f8

…to qwen3next_rebase

* 'main' of https://github.com/vllm-project/vllm-ascend:
  [Patch] Remove the patch of MiniCPM (vllm-project#5975)
  [P/D] layerwise connector support recompute scheduler (vllm-project#5900)
  [CI] Add workflow support for lint image build (vllm-project#6489)
  [Bugfix] Fix problematic dummy_run & improper input_batch_size in eagle (vllm-project#6517)
  [Refactor]310p_e2e test case update (vllm-project#6539)
  [Refactor]refactor p2p connector (vllm-project#6551)
  [Refactor]refactor 310p attention impl and add ut (vllm-project#6579)
  [Refactor]refactor 310p ops and add ut (vllm-project#6591)
  [Ops][Refactor] Remove custom rotary_embedding operator (vllm-project#6523)
  [Lint]Style: Convert `vllm-ascend/` to ruff format(new Batch vllm-project#8) (vllm-project#6604)
  [Test] Add initial multi modal cases of Qwen2.5-VL-7B-Instruct for disaggregated encoder  (vllm-project#5301)
  [CI] Fix broken CI (vllm-project#6599)
  [Lint]Style: Convert `vllm-ascend/` to ruff format(Batch vllm-project#10) (vllm-project#6173)
  [Lint]Style: Convert `vllm-ascend/` to ruff format(Batch vllm-project#11) (vllm-project#6176)
  [Lint]Style: Convert `vllm-ascend/` to ruff format(Batch vllm-project#8) (vllm-project#6129)
  [Lint]Style: Convert `vllm-ascend/` to ruff format(Batch vllm-project#7) (vllm-project#6023)
  [CI][Misc] Some improvement for github action (vllm-project#6587)
  [Image] Bump mooncake version to v0.3.8.post1 (vllm-project#6428)

chenchuw886 pushed a commit to chenchuw886/vllm-ascend that referenced this pull request


          [Lint]Style: Convert vllm-ascend/ to ruff format(Batch vllm-project#11

7d805b3

) (vllm-project#6176)

### What this PR does / why we need it?
**Scope of Changes**:
| File Path |
| :--- |
| `vllm_ascend/ops/fused_moe/comm_utils.py` |
| `vllm_ascend/ops/fused_moe/experts_selector.py` |
| `vllm_ascend/ops/fused_moe/fused_moe.py` |
| `vllm_ascend/ops/fused_moe/moe_comm_method.py` |
| `vllm_ascend/ops/fused_moe/moe_mlp.py` |
| `vllm_ascend/ops/fused_moe/prepare_finalize.py` |
| `vllm_ascend/ops/fused_moe/token_dispatcher.py` |

### Does this PR introduce _any_ user-facing change?

### How was this patch tested?

- vLLM version: v0.14.0
- vLLM main:
vllm-project/vllm@d682094

Signed-off-by: MrZ20 <2609716663@qq.com>
Signed-off-by: SILONG ZENG <2609716663@qq.com>
Signed-off-by: momochenchuw <chenchuw@huawei.com>

wangxiyuan mentioned this pull request

[Misc]: test #6787

Closed

ZRJ026 pushed a commit to ZRJ026/vllm-ascend that referenced this pull request


          [Lint]Style: Convert vllm-ascend/ to ruff format(Batch vllm-project#11

70bb90d

) (vllm-project#6176)

### What this PR does / why we need it?
**Scope of Changes**:
| File Path |
| :--- |
| `vllm_ascend/ops/fused_moe/comm_utils.py` |
| `vllm_ascend/ops/fused_moe/experts_selector.py` |
| `vllm_ascend/ops/fused_moe/fused_moe.py` |
| `vllm_ascend/ops/fused_moe/moe_comm_method.py` |
| `vllm_ascend/ops/fused_moe/moe_mlp.py` |
| `vllm_ascend/ops/fused_moe/prepare_finalize.py` |
| `vllm_ascend/ops/fused_moe/token_dispatcher.py` |

### Does this PR introduce _any_ user-facing change?

### How was this patch tested?

- vLLM version: v0.14.0
- vLLM main:
vllm-project/vllm@d682094

Signed-off-by: MrZ20 <2609716663@qq.com>
Signed-off-by: SILONG ZENG <2609716663@qq.com>
Signed-off-by: zrj026 <zhangrunjiang026@gmail.com>

maoxx241 pushed a commit to maoxx241/vllm-ascend that referenced this pull request


          [Lint]Style: Convert vllm-ascend/ to ruff format(Batch vllm-project#11

2ca09d1

) (vllm-project#6176)

### What this PR does / why we need it?
**Scope of Changes**:
| File Path |
| :--- |
| `vllm_ascend/ops/fused_moe/comm_utils.py` |
| `vllm_ascend/ops/fused_moe/experts_selector.py` |
| `vllm_ascend/ops/fused_moe/fused_moe.py` |
| `vllm_ascend/ops/fused_moe/moe_comm_method.py` |
| `vllm_ascend/ops/fused_moe/moe_mlp.py` |
| `vllm_ascend/ops/fused_moe/prepare_finalize.py` |
| `vllm_ascend/ops/fused_moe/token_dispatcher.py` |

### Does this PR introduce _any_ user-facing change?

### How was this patch tested?

- vLLM version: v0.14.0
- vLLM main:
vllm-project/vllm@d682094

Signed-off-by: MrZ20 <2609716663@qq.com>
Signed-off-by: SILONG ZENG <2609716663@qq.com>

ZRJ026 pushed a commit to ZRJ026/vllm-ascend that referenced this pull request


          [Lint]Style: Convert vllm-ascend/ to ruff format(Batch vllm-project#11

fd61c8c

) (vllm-project#6176)

### What this PR does / why we need it?
**Scope of Changes**:
| File Path |
| :--- |
| `vllm_ascend/ops/fused_moe/comm_utils.py` |
| `vllm_ascend/ops/fused_moe/experts_selector.py` |
| `vllm_ascend/ops/fused_moe/fused_moe.py` |
| `vllm_ascend/ops/fused_moe/moe_comm_method.py` |
| `vllm_ascend/ops/fused_moe/moe_mlp.py` |
| `vllm_ascend/ops/fused_moe/prepare_finalize.py` |
| `vllm_ascend/ops/fused_moe/token_dispatcher.py` |

### Does this PR introduce _any_ user-facing change?

### How was this patch tested?

- vLLM version: v0.14.0
- vLLM main:
vllm-project/vllm@d682094

Signed-off-by: MrZ20 <2609716663@qq.com>
Signed-off-by: SILONG ZENG <2609716663@qq.com>
Signed-off-by: zrj026 <zhangrunjiang026@gmail.com>

LCAIZJ pushed a commit to LCAIZJ/vllm-ascend that referenced this pull request


          [Lint]Style: Convert vllm-ascend/ to ruff format(Batch vllm-project#11

069e05e

) (vllm-project#6176)

### What this PR does / why we need it?
**Scope of Changes**:
| File Path |
| :--- |
| `vllm_ascend/ops/fused_moe/comm_utils.py` |
| `vllm_ascend/ops/fused_moe/experts_selector.py` |
| `vllm_ascend/ops/fused_moe/fused_moe.py` |
| `vllm_ascend/ops/fused_moe/moe_comm_method.py` |
| `vllm_ascend/ops/fused_moe/moe_mlp.py` |
| `vllm_ascend/ops/fused_moe/prepare_finalize.py` |
| `vllm_ascend/ops/fused_moe/token_dispatcher.py` |

### Does this PR introduce _any_ user-facing change?

### How was this patch tested?

- vLLM version: v0.14.0
- vLLM main:
vllm-project/vllm@d682094

Signed-off-by: MrZ20 <2609716663@qq.com>
Signed-off-by: SILONG ZENG <2609716663@qq.com>

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

zzzzwwjj Awaiting requested review from zzzzwwjj zzzzwwjj is a code owner

realliujiaxu Awaiting requested review from realliujiaxu realliujiaxu is a code owner

whx-sjtu Awaiting requested review from whx-sjtu whx-sjtu is a code owner

1 more reviewer

gemini-code-assist[bot] gemini-code-assist[bot] left review comments

Labels