Skip to content

Conversation

@Yikun
Copy link
Collaborator

@Yikun Yikun commented May 6, 2025

What this PR does / why we need it?

Does this PR introduce any user-facing change?

No

How was this patch tested?

CI passed

@Yikun Yikun force-pushed the clean-triton-patch branch from adc857f to 5500fb8 Compare May 6, 2025 08:47
Yikun added 2 commits May 6, 2025 16:48
…ject#753)"

This reverts commit d7e1110.

Signed-off-by: Yikun Jiang <yikunkero@gmail.com>
Signed-off-by: Yikun Jiang <yikunkero@gmail.com>
@Yikun Yikun force-pushed the clean-triton-patch branch from 5500fb8 to bf1fb1c Compare May 6, 2025 08:49
@Yikun Yikun changed the title [Core] Clean triton patch [Core] Clean triton patch which has been fixed in vllm May 6, 2025
@Yikun Yikun changed the title [Core] Clean triton patch which has been fixed in vllm [Core] Cleanup triton patch which has been fixed in vllm May 6, 2025
@Yikun Yikun marked this pull request as ready for review May 6, 2025 10:50
@Yikun
Copy link
Collaborator Author

Yikun commented May 6, 2025

Consider it's a main only change:
image

All CI passed, will merge directly.

@Yikun Yikun merged commit afe1767 into vllm-project:main May 6, 2025
14 of 18 checks passed
chopper0126 pushed a commit to chopper0126/vllm-ascend that referenced this pull request Oct 16, 2025
…t#764)

### What this PR does / why we need it?
- Revert "Re-patch TritonPlaceholder on main to make CI happy (vllm-project#753)"
because upstream main CI already merged:
vllm-project/vllm#17446
- Keep 0.8.5.post1 compatible

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

### How was this patch tested?
CI passed

---------

Signed-off-by: Yikun Jiang <yikunkero@gmail.com>
Angazenn pushed a commit to Angazenn/vllm-ascend that referenced this pull request Oct 21, 2025
…t#764)

### What this PR does / why we need it?
- Revert "Re-patch TritonPlaceholder on main to make CI happy (vllm-project#753)"
because upstream main CI already merged:
vllm-project/vllm#17446
- Keep 0.8.5.post1 compatible

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

### How was this patch tested?
CI passed

---------

Signed-off-by: Yikun Jiang <yikunkero@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants