Skip to content

Comments

Revert "[Bugfix] Add init_workspace_manager to moe kernel benchmarks …#23

Open
MitchLewis930 wants to merge 1 commit intobench/PR_009_basefrom
bench/PR_009_bug__CodeRabbit
Open

Revert "[Bugfix] Add init_workspace_manager to moe kernel benchmarks …#23
MitchLewis930 wants to merge 1 commit intobench/PR_009_basefrom
bench/PR_009_bug__CodeRabbit

Conversation

@MitchLewis930
Copy link
Collaborator

…(vllm-project#31042)"

This reverts commit e1cd7a5.

PLEASE FILL IN THE PR DESCRIPTION HERE ENSURING ALL CHECKLIST ITEMS (AT THE BOTTOM) HAVE BEEN CONSIDERED.

Purpose

Test Plan

Test Result


Essential Elements of an Effective PR Description Checklist
  • The purpose of the PR, such as "Fix some issue (link existing issues this PR will resolve)".
  • The test plan, such as providing test command.
  • The test results, such as pasting the results comparison before and after, or e2e results
  • (Optional) The necessary documentation update, such as updating supported_models.md and examples for a new model.
  • (Optional) Release notes update. If your change is user facing, please update the release notes draft in the Google Doc.

BEFORE SUBMITTING, PLEASE READ https://docs.vllm.ai/en/latest/contributing (anything written below this line will be removed by GitHub Actions)

@coderabbitai
Copy link

coderabbitai bot commented Jan 21, 2026

Warning

Rate limit exceeded

@MitchLewis930 has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 8 minutes and 37 seconds before requesting another review.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

📥 Commits

Reviewing files that changed from the base of the PR and between e1cd7a5 and 2f33fa9.

📒 Files selected for processing (3)
  • benchmarks/kernels/benchmark_cutlass_fp4_moe.py
  • benchmarks/kernels/benchmark_cutlass_moe_fp8.py
  • benchmarks/kernels/benchmark_grouped_gemm_cutlass.py

Comment @coderabbitai help to get the list of available commands and usage tips.

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