Skip to content

Revert "[Build] Make bundled DeepGEMM wheel portable across Python versions"#41512

Merged
vllm-bot merged 1 commit intomainfrom
revert-41476-fix-deepgemm-py-limited-api
May 2, 2026
Merged

Revert "[Build] Make bundled DeepGEMM wheel portable across Python versions"#41512
vllm-bot merged 1 commit intomainfrom
revert-41476-fix-deepgemm-py-limited-api

Conversation

@mgoin
Copy link
Copy Markdown
Member

@mgoin mgoin commented May 2, 2026

Reverts #41476 as the approach is not actually viable for pybind11-based bindings

Copy link
Copy Markdown

@claude claude Bot left a comment

Choose a reason for hiding this comment

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

Claude Code Review

This repository is configured for manual code reviews. Comment @claude review to trigger a review and subscribe this PR to future pushes, or @claude review once for a one-time review.

Tip: disable this comment in your organization's Code Review settings.

@mergify mergify Bot added the ci/build label May 2, 2026
Copy link
Copy Markdown
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 simplifies the CMake configuration for the deepgemm external project by removing the logic for free-threaded Python checks and Stable ABI (SABI) support. It also removes the explicit library output directory for the _deep_gemm_C target and associated compile options that were used to manage Py_LIMITED_API. I have no feedback to provide.

@vllm-bot vllm-bot merged commit 0a9362d into main May 2, 2026
12 of 13 checks passed
@vllm-bot vllm-bot deleted the revert-41476-fix-deepgemm-py-limited-api branch May 2, 2026 16:42
joa-stdn pushed a commit to joa-stdn/vllm that referenced this pull request May 4, 2026
…rsions" (vllm-project#41512)

Signed-off-by: Joachim Studnia <joachim@mistral.ai>
chaojun-zhang pushed a commit to chaojun-zhang/vllm that referenced this pull request May 6, 2026
Copilot AI pushed a commit to hongbolv/vllm that referenced this pull request May 7, 2026
…rsions" (vllm-project#41512)

Co-authored-by: hongbolv <33214277+hongbolv@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants