Skip to content

Fix build vllm kernel for the first time may encounter failure#54

Open
zichen-kuuga wants to merge 2 commits intoMooreThreads:mainfrom
zichen-kuuga:fix/issue_14
Open

Fix build vllm kernel for the first time may encounter failure#54
zichen-kuuga wants to merge 2 commits intoMooreThreads:mainfrom
zichen-kuuga:fix/issue_14

Conversation

@zichen-kuuga
Copy link
Copy Markdown
Contributor

The vLLM kernel uses header file references with relative paths that need to be modified from csrc/xxx to csrc/xxx_musa. If the default directory order is used, the operation that copies csrc to csrc_musa may execute before changing csrc/xxx to csrc/xxx_musa, causing header files to be missing during the initial installation. To avoid this, we sort directories by depth (deepest first).

All tests in tests/test_cpp_extension.py and tests/test_extension_build.py have passed.

@augmentcode
Copy link
Copy Markdown

augmentcode bot commented Apr 10, 2026

🤖 Augment PR Summary

Summary: Fixes first-time vLLM kernel extension builds by ensuring CUDA source directories are ported in a deterministic order (deepest paths first) so header references resolve correctly.
Also: Bumps torchada version to 0.1.49 across package metadata and docs.

🤖 Was this summary useful? React with 👍 or 👎

Copy link
Copy Markdown

@augmentcode augmentcode bot left a comment

Choose a reason for hiding this comment

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

Review completed. 1 suggestion posted.

Fix All in Augment

Comment augment review to trigger a new review at any time.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant