Skip to content

fix: typo prefll -> prefill in cudagraph option#1283

Merged
sufubao merged 1 commit into
ModelTC:mainfrom
sufubao:fix/prefll-typo
Apr 30, 2026
Merged

fix: typo prefll -> prefill in cudagraph option#1283
sufubao merged 1 commit into
ModelTC:mainfrom
sufubao:fix/prefll-typo

Conversation

@sufubao

@sufubao sufubao commented Apr 30, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Rename CLI flag --prefll_cudagraph_max_handle_token to --prefill_cudagraph_max_handle_token and update the matching start_args_type field and consumer in prefill_cuda_graph.py.
  • Fix the related Chinese comment in infer_struct.py (prefll cuda graph -> prefill cuda graph).

Test plan

  • python -m lightllm.server.api_cli --help | grep prefill_cudagraph_max_handle_token shows the renamed flag.
  • Smoke-test starting the server with --prefill_cudagraph_max_handle_token 512 and confirm prefill cudagraph still initializes.

…x_handle_token

Fix typo "prefll" -> "prefill" in the prefill cudagraph option name and
the corresponding code/comment references.
@sufubao sufubao merged commit 4208b76 into ModelTC:main Apr 30, 2026
1 check passed

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown
Contributor

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 corrects several instances of the typo 'prefll' to 'prefill' across multiple files, including comments, variable names, command-line arguments, and data structures related to CUDA graph prefilling. I have no feedback to provide.

@sufubao sufubao deleted the fix/prefll-typo branch May 1, 2026 03:44
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