Repository navigation
[DNM] Add custom triton kernel - #9265
wangxiyuan wants to merge 1 commit into
Conversation
Summary of ChangesHello, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed! This pull request enables platform-specific optimizations for the vLLM-Ascend project by integrating custom Triton kernels. By leveraging the upstream vLLM triton_dispatcher, the changes allow for more efficient execution of expansion operations on NPU hardware, ensuring better alignment with Ascend-specific performance requirements. Highlights
New Features🧠 You can now enable Memory (public preview) to help Gemini Code Assist learn from your team's feedback. This makes future code reviews more consistent and personalized to your project's style. Click here to enable Memory in your admin console. Using Gemini Code AssistThe full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips. Invoking Gemini You can request assistance from Gemini at any point by creating a comment using either
Customization To customize the Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a Limitations & Feedback Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counterproductive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for GitHub and other Google products, sign up here. Footnotes
|
There was a problem hiding this comment.
Code Review
This pull request introduces the registration of Ascend-specific Triton kernels using the triton_dispatcher mechanism, specifically for the expand operation. This change aims to replace manual function patching with a more integrated approach. Feedback indicates that the PR title and summary should be updated to follow the repository's style guide. Additionally, it is recommended to remove the commented-out code in patch_rejection_sampler.py instead of just disabling it, and to verify that the upstream implementation correctly utilizes the dispatcher for the 'expand' kernel to ensure the optimized implementation is actually used.
|
👋 Hi! Thank you for contributing to the vLLM Ascend project. The following points will speed up your PR merge:
If CI fails, you can run linting and testing checks locally according Contributing and Testing. |
b355665 to
08a6a5c
Compare
Signed-off-by: wangxiyuan <wangxiyuan1007@gmail.com>
08a6a5c to
79d97e5
Compare
expand_kernelascend implementation by triton register, instead of patch way.