Skip to content

[Misc] Delay deprecation of CommonAttentionMetadata properties#33801

Merged
vllm-bot merged 1 commit intovllm-project:mainfrom
neuralmagic:lwilkinson/delay-deprecation
Feb 4, 2026
Merged

[Misc] Delay deprecation of CommonAttentionMetadata properties#33801
vllm-bot merged 1 commit intovllm-project:mainfrom
neuralmagic:lwilkinson/delay-deprecation

Conversation

@LucasWilkinson
Copy link
Collaborator

@LucasWilkinson LucasWilkinson commented Feb 4, 2026

Deprecation is delayed due to snags with performance #33771 and CI instability slowing down #32073

Remove version for now until the timeline becomes more clear

Signed-off-by: Lucas Wilkinson <lwilkins@redhat.com>
Copy link
Contributor

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request updates the deprecation messages for the seq_lens_cpu and num_computed_tokens_cpu properties in CommonAttentionMetadata. The change removes the specific version v0.15.0 targeted for removal and replaces it with a more general message urging users to migrate. This aligns with the pull request's goal of delaying the deprecation due to performance and stability concerns. The changes are correct and look good.

@LucasWilkinson LucasWilkinson enabled auto-merge (squash) February 4, 2026 16:11
@github-actions github-actions bot added the ready ONLY add when PR is ready to merge/full CI is needed label Feb 4, 2026
@vllm-bot vllm-bot merged commit 0e92298 into vllm-project:main Feb 4, 2026
13 of 18 checks passed
gameofdimension pushed a commit to gameofdimension/vllm that referenced this pull request Feb 5, 2026
…project#33801)

Signed-off-by: Lucas Wilkinson <lwilkins@redhat.com>
Signed-off-by: felix01.yu <felix01.yu@vipshop.com>
ItzDEXX pushed a commit to ItzDEXX/vllm that referenced this pull request Feb 19, 2026
tunglinwood pushed a commit to tunglinwood/vllm that referenced this pull request Mar 4, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ready ONLY add when PR is ready to merge/full CI is needed v1

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants