Skip to content

Conversation

@xyang16
Copy link
Contributor

@xyang16 xyang16 commented Oct 2, 2024

Description

Brief description of what this PR is about

  • If this change is a backward incompatible change, why must this change be made?
  • Interesting edge cases to note here

@xyang16 xyang16 requested review from a team and zachgk as code owners October 2, 2024 20:19
@xyang16 xyang16 merged commit 0631414 into deepjavalibrary:master Oct 2, 2024
enable_prefix_caching: Optional[bool] = False
disable_sliding_window: Optional[bool] = False
limit_mm_per_prompt: Optional[Mapping[str, int]] = None
use_v2_block_manager: bool = False
Copy link
Contributor

Choose a reason for hiding this comment

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

FYI: vllm-project/vllm#8678 (not merged yet, but changes this default to True in vLLM).

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.

4 participants