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

Merged

wangxiyuan merged 3 commits intovllm-project:mainfrom

MrZ20:lint-ascend-4

Jan 24, 2026

Contributor

MrZ20 commented Jan 19, 2026 •

edited

Loading

What this PR does / why we need it?

Scope of Changes:

File Path
`.../distributed/kv_transfer/kv_pool/ascend_store/ascend_store_connector.py`
`vllm_ascend/distributed/kv_transfer/kv_pool/ascend_store/backend/backend.py`
`.../distributed/kv_transfer/kv_pool/ascend_store/backend/memcache_backend.py`
`.../distributed/kv_transfer/kv_pool/ascend_store/backend/mooncake_backend.py`
`vllm_ascend/distributed/kv_transfer/kv_pool/ascend_store/config_data.py`
`vllm_ascend/distributed/kv_transfer/kv_pool/ascend_store/kv_transfer.py`
`vllm_ascend/distributed/kv_transfer/kv_pool/ascend_store/pool_scheduler.py`
`vllm_ascend/distributed/kv_transfer/kv_pool/ascend_store/pool_worker.py`
`.../distributed/kv_transfer/kv_pool/cpu_offload/cpu_kv_cache_manager.py`
`.../distributed/kv_transfer/kv_pool/cpu_offload/cpu_offload_connector.py`
`vllm_ascend/distributed/kv_transfer/kv_pool/cpu_offload/metadata.py`
`vllm_ascend/distributed/kv_transfer/kv_pool/ucm_connector.py`
`vllm_ascend/distributed/kv_transfer/utils/mooncake_transfer_engine.py`
`vllm_ascend/distributed/kv_transfer/utils/utils.py`
`vllm_ascend/kv_offload/cpu_npu.py`
`vllm_ascend/kv_offload/npu.py`
`vllm_ascend/lora/lora_ops.py`
`vllm_ascend/lora/punica_npu.py`
`vllm_ascend/lora/utils.py`

Does this PR introduce any user-facing change?

How was this patch tested?

vLLM version: v0.13.0
vLLM main: vllm-project/vllm@2c24bc6

MrZ20 requested review from LCAIZJ, MengqingCao, nalinaly, paulyu12 and wangxiyuan as code owners

January 19, 2026 07:03

Contributor

github-actions bot commented Jan 19, 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 applies ruff formatting and linting fixes across a wide range of files in the vllm-ascend/ directory. The changes primarily consist of style and formatting improvements, such as reformatting long lines, standardizing import statements, and updating type hints to modern syntax. Some changes also address linting rules, like adding @abstractmethod to empty methods in abstract base classes and fixing mutable default arguments, which improves code quality and correctness. Overall, these are positive changes that enhance code readability and maintainability. I have reviewed the changes and found no issues of high or critical severity. The pull request is good to merge.

MrZ20 mentioned this pull request

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

Open

MrZ20 force-pushed the lint-ascend-4 branch 2 times, most recently from f5b45d1 to 5ac654a Compare

January 20, 2026 01:29

vllm-ascend-ci added ready ready-for-test labels

MrZ20 force-pushed the lint-ascend-4 branch 3 times, most recently from 162ed5d to 62bfea2 Compare

January 20, 2026 13:09

github-actions bot added the merge-conflicts label

Contributor

github-actions bot commented Jan 20, 2026

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

MrZ20 force-pushed the lint-ascend-4 branch from 62bfea2 to 9d00ae4 Compare

January 21, 2026 02:48

github-actions bot removed the merge-conflicts label

paulyu12 approved these changes

View reviewed changes

Potabk removed 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-4 branch from 9d00ae4 to 4c0da39 Compare

January 24, 2026 02:25

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.

MrZ20 force-pushed the lint-ascend-4 branch from 7af65f5 to e007e55 Compare

January 24, 2026 04:09

github-actions bot removed the merge-conflicts label

MrZ20 force-pushed the lint-ascend-4 branch from e007e55 to 3598870 Compare

January 24, 2026 04:12

github-actions bot added the merge-conflicts label

Contributor

github-actions bot commented Jan 24, 2026

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

MrZ20 added 2 commits

January 24, 2026 20:54


          modify

eccb51c

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

update

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

fix

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

del

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

fix

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


          modify

97e5918

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

MrZ20 force-pushed the lint-ascend-4 branch from 3598870 to 97e5918 Compare

January 24, 2026 12:57

github-actions bot removed the merge-conflicts label

Contributor

github-actions bot commented Jan 24, 2026

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

github-actions bot added the merge-conflicts label


          Merge branch 'main' into lint-ascend-4

c971e1d

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

github-actions bot removed the merge-conflicts label

wangxiyuan merged commit 6ccccad into vllm-project:main

20 checks passed

MrZ20 deleted the lint-ascend-4 branch

January 25, 2026 05:22

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


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

f35d931

…) (vllm-project#5996)

### What this PR does / why we need it?
**Scope of Changes**:
| File Path |
| :--- |
|
`.../distributed/kv_transfer/kv_pool/ascend_store/ascend_store_connector.py`
|
|
`vllm_ascend/distributed/kv_transfer/kv_pool/ascend_store/backend/backend.py`
|
| `
.../distributed/kv_transfer/kv_pool/ascend_store/backend/memcache_backend.py`
|
| `
.../distributed/kv_transfer/kv_pool/ascend_store/backend/mooncake_backend.py`
|
| `
vllm_ascend/distributed/kv_transfer/kv_pool/ascend_store/config_data.py`
|
| `
vllm_ascend/distributed/kv_transfer/kv_pool/ascend_store/kv_transfer.py`
|
| `
vllm_ascend/distributed/kv_transfer/kv_pool/ascend_store/pool_scheduler.py`
|
| `
vllm_ascend/distributed/kv_transfer/kv_pool/ascend_store/pool_worker.py`
|
| `
.../distributed/kv_transfer/kv_pool/cpu_offload/cpu_kv_cache_manager.py`
|
| `
.../distributed/kv_transfer/kv_pool/cpu_offload/cpu_offload_connector.py`
|
| ` vllm_ascend/distributed/kv_transfer/kv_pool/cpu_offload/metadata.py`
|
| ` vllm_ascend/distributed/kv_transfer/kv_pool/ucm_connector.py` |
| `
vllm_ascend/distributed/kv_transfer/utils/mooncake_transfer_engine.py` |
| ` vllm_ascend/distributed/kv_transfer/utils/utils.py` |
| ` vllm_ascend/kv_offload/cpu_npu.py` |
| ` vllm_ascend/kv_offload/npu.py` |
| ` vllm_ascend/lora/lora_ops.py` |
| ` vllm_ascend/lora/punica_npu.py` |
| ` vllm_ascend/lora/utils.py` |

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

### How was this patch tested?

- vLLM version: v0.13.0
- vLLM main:
vllm-project/vllm@2c24bc6

---------

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

08eb666

…) (vllm-project#5996)

### What this PR does / why we need it?
**Scope of Changes**:
| File Path |
| :--- |
|
`.../distributed/kv_transfer/kv_pool/ascend_store/ascend_store_connector.py`
|
|
`vllm_ascend/distributed/kv_transfer/kv_pool/ascend_store/backend/backend.py`
|
| `
.../distributed/kv_transfer/kv_pool/ascend_store/backend/memcache_backend.py`
|
| `
.../distributed/kv_transfer/kv_pool/ascend_store/backend/mooncake_backend.py`
|
| `
vllm_ascend/distributed/kv_transfer/kv_pool/ascend_store/config_data.py`
|
| `
vllm_ascend/distributed/kv_transfer/kv_pool/ascend_store/kv_transfer.py`
|
| `
vllm_ascend/distributed/kv_transfer/kv_pool/ascend_store/pool_scheduler.py`
|
| `
vllm_ascend/distributed/kv_transfer/kv_pool/ascend_store/pool_worker.py`
|
| `
.../distributed/kv_transfer/kv_pool/cpu_offload/cpu_kv_cache_manager.py`
|
| `
.../distributed/kv_transfer/kv_pool/cpu_offload/cpu_offload_connector.py`
|
| ` vllm_ascend/distributed/kv_transfer/kv_pool/cpu_offload/metadata.py`
|
| ` vllm_ascend/distributed/kv_transfer/kv_pool/ucm_connector.py` |
| `
vllm_ascend/distributed/kv_transfer/utils/mooncake_transfer_engine.py` |
| ` vllm_ascend/distributed/kv_transfer/utils/utils.py` |
| ` vllm_ascend/kv_offload/cpu_npu.py` |
| ` vllm_ascend/kv_offload/npu.py` |
| ` vllm_ascend/lora/lora_ops.py` |
| ` vllm_ascend/lora/punica_npu.py` |
| ` vllm_ascend/lora/utils.py` |

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

### How was this patch tested?

- vLLM version: v0.13.0
- vLLM main:
vllm-project/vllm@2c24bc6

---------

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

033677a

…) (vllm-project#5996)

### What this PR does / why we need it?
**Scope of Changes**:
| File Path |
| :--- |
|
`.../distributed/kv_transfer/kv_pool/ascend_store/ascend_store_connector.py`
|
|
`vllm_ascend/distributed/kv_transfer/kv_pool/ascend_store/backend/backend.py`
|
| `
.../distributed/kv_transfer/kv_pool/ascend_store/backend/memcache_backend.py`
|
| `
.../distributed/kv_transfer/kv_pool/ascend_store/backend/mooncake_backend.py`
|
| `
vllm_ascend/distributed/kv_transfer/kv_pool/ascend_store/config_data.py`
|
| `
vllm_ascend/distributed/kv_transfer/kv_pool/ascend_store/kv_transfer.py`
|
| `
vllm_ascend/distributed/kv_transfer/kv_pool/ascend_store/pool_scheduler.py`
|
| `
vllm_ascend/distributed/kv_transfer/kv_pool/ascend_store/pool_worker.py`
|
| `
.../distributed/kv_transfer/kv_pool/cpu_offload/cpu_kv_cache_manager.py`
|
| `
.../distributed/kv_transfer/kv_pool/cpu_offload/cpu_offload_connector.py`
|
| ` vllm_ascend/distributed/kv_transfer/kv_pool/cpu_offload/metadata.py`
|
| ` vllm_ascend/distributed/kv_transfer/kv_pool/ucm_connector.py` |
| `
vllm_ascend/distributed/kv_transfer/utils/mooncake_transfer_engine.py` |
| ` vllm_ascend/distributed/kv_transfer/utils/utils.py` |
| ` vllm_ascend/kv_offload/cpu_npu.py` |
| ` vllm_ascend/kv_offload/npu.py` |
| ` vllm_ascend/lora/lora_ops.py` |
| ` vllm_ascend/lora/punica_npu.py` |
| ` vllm_ascend/lora/utils.py` |

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

### How was this patch tested?

- vLLM version: v0.13.0
- vLLM main:
vllm-project/vllm@2c24bc6

---------

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

a1693ea

…) (vllm-project#5996)

### What this PR does / why we need it?
**Scope of Changes**:
| File Path |
| :--- |
|
`.../distributed/kv_transfer/kv_pool/ascend_store/ascend_store_connector.py`
|
|
`vllm_ascend/distributed/kv_transfer/kv_pool/ascend_store/backend/backend.py`
|
| `
.../distributed/kv_transfer/kv_pool/ascend_store/backend/memcache_backend.py`
|
| `
.../distributed/kv_transfer/kv_pool/ascend_store/backend/mooncake_backend.py`
|
| `
vllm_ascend/distributed/kv_transfer/kv_pool/ascend_store/config_data.py`
|
| `
vllm_ascend/distributed/kv_transfer/kv_pool/ascend_store/kv_transfer.py`
|
| `
vllm_ascend/distributed/kv_transfer/kv_pool/ascend_store/pool_scheduler.py`
|
| `
vllm_ascend/distributed/kv_transfer/kv_pool/ascend_store/pool_worker.py`
|
| `
.../distributed/kv_transfer/kv_pool/cpu_offload/cpu_kv_cache_manager.py`
|
| `
.../distributed/kv_transfer/kv_pool/cpu_offload/cpu_offload_connector.py`
|
| ` vllm_ascend/distributed/kv_transfer/kv_pool/cpu_offload/metadata.py`
|
| ` vllm_ascend/distributed/kv_transfer/kv_pool/ucm_connector.py` |
| `
vllm_ascend/distributed/kv_transfer/utils/mooncake_transfer_engine.py` |
| ` vllm_ascend/distributed/kv_transfer/utils/utils.py` |
| ` vllm_ascend/kv_offload/cpu_npu.py` |
| ` vllm_ascend/kv_offload/npu.py` |
| ` vllm_ascend/lora/lora_ops.py` |
| ` vllm_ascend/lora/punica_npu.py` |
| ` vllm_ascend/lora/utils.py` |

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

### How was this patch tested?

- vLLM version: v0.13.0
- vLLM main:
vllm-project/vllm@2c24bc6

---------

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

802ed24

…) (vllm-project#5996)

### What this PR does / why we need it?
**Scope of Changes**:
| File Path |
| :--- |
|
`.../distributed/kv_transfer/kv_pool/ascend_store/ascend_store_connector.py`
|
|
`vllm_ascend/distributed/kv_transfer/kv_pool/ascend_store/backend/backend.py`
|
| `
.../distributed/kv_transfer/kv_pool/ascend_store/backend/memcache_backend.py`
|
| `
.../distributed/kv_transfer/kv_pool/ascend_store/backend/mooncake_backend.py`
|
| `
vllm_ascend/distributed/kv_transfer/kv_pool/ascend_store/config_data.py`
|
| `
vllm_ascend/distributed/kv_transfer/kv_pool/ascend_store/kv_transfer.py`
|
| `
vllm_ascend/distributed/kv_transfer/kv_pool/ascend_store/pool_scheduler.py`
|
| `
vllm_ascend/distributed/kv_transfer/kv_pool/ascend_store/pool_worker.py`
|
| `
.../distributed/kv_transfer/kv_pool/cpu_offload/cpu_kv_cache_manager.py`
|
| `
.../distributed/kv_transfer/kv_pool/cpu_offload/cpu_offload_connector.py`
|
| ` vllm_ascend/distributed/kv_transfer/kv_pool/cpu_offload/metadata.py`
|
| ` vllm_ascend/distributed/kv_transfer/kv_pool/ucm_connector.py` |
| `
vllm_ascend/distributed/kv_transfer/utils/mooncake_transfer_engine.py` |
| ` vllm_ascend/distributed/kv_transfer/utils/utils.py` |
| ` vllm_ascend/kv_offload/cpu_npu.py` |
| ` vllm_ascend/kv_offload/npu.py` |
| ` vllm_ascend/lora/lora_ops.py` |
| ` vllm_ascend/lora/punica_npu.py` |
| ` vllm_ascend/lora/utils.py` |

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

### How was this patch tested?

- vLLM version: v0.13.0
- vLLM main:
vllm-project/vllm@2c24bc6

---------

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

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


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

47c28cd

…) (vllm-project#5996)

### What this PR does / why we need it?
**Scope of Changes**:
| File Path |
| :--- |
|
`.../distributed/kv_transfer/kv_pool/ascend_store/ascend_store_connector.py`
|
|
`vllm_ascend/distributed/kv_transfer/kv_pool/ascend_store/backend/backend.py`
|
| `
.../distributed/kv_transfer/kv_pool/ascend_store/backend/memcache_backend.py`
|
| `
.../distributed/kv_transfer/kv_pool/ascend_store/backend/mooncake_backend.py`
|
| `
vllm_ascend/distributed/kv_transfer/kv_pool/ascend_store/config_data.py`
|
| `
vllm_ascend/distributed/kv_transfer/kv_pool/ascend_store/kv_transfer.py`
|
| `
vllm_ascend/distributed/kv_transfer/kv_pool/ascend_store/pool_scheduler.py`
|
| `
vllm_ascend/distributed/kv_transfer/kv_pool/ascend_store/pool_worker.py`
|
| `
.../distributed/kv_transfer/kv_pool/cpu_offload/cpu_kv_cache_manager.py`
|
| `
.../distributed/kv_transfer/kv_pool/cpu_offload/cpu_offload_connector.py`
|
| ` vllm_ascend/distributed/kv_transfer/kv_pool/cpu_offload/metadata.py`
|
| ` vllm_ascend/distributed/kv_transfer/kv_pool/ucm_connector.py` |
| `
vllm_ascend/distributed/kv_transfer/utils/mooncake_transfer_engine.py` |
| ` vllm_ascend/distributed/kv_transfer/utils/utils.py` |
| ` vllm_ascend/kv_offload/cpu_npu.py` |
| ` vllm_ascend/kv_offload/npu.py` |
| ` vllm_ascend/lora/lora_ops.py` |
| ` vllm_ascend/lora/punica_npu.py` |
| ` vllm_ascend/lora/utils.py` |

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

### How was this patch tested?

- vLLM version: v0.13.0
- vLLM main:
vllm-project/vllm@2c24bc6

---------

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

paulyu12 paulyu12 approved these changes

wangxiyuan Awaiting requested review from wangxiyuan wangxiyuan is a code owner

nalinaly Awaiting requested review from nalinaly nalinaly is a code owner

MengqingCao Awaiting requested review from MengqingCao MengqingCao is a code owner

LCAIZJ Awaiting requested review from LCAIZJ LCAIZJ is a code owner

+1 more reviewer

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

Labels

None yet