Skip to content

Commit

Permalink
bugfix for mrope
Browse files Browse the repository at this point in the history
Signed-off-by: zouyida <zouyida@huawei.com>
  • Loading branch information
zouyida2002 committed Feb 26, 2025
1 parent d535c1d commit dce979a
Showing 1 changed file with 2 additions and 6 deletions.
8 changes: 2 additions & 6 deletions vllm_ascend/model_runner.py
Original file line number Diff line number Diff line change
Expand Up @@ -721,14 +721,10 @@ def _compute_multi_modal_input(self, inter_data: InterDataForSeqGroup,
mrope_input_positions, mrope_position_delta = \
MRotaryEmbedding.get_input_positions(
token_ids,
hf_config

Check failure on line 724 in vllm_ascend/model_runner.py

View workflow job for this annotation

GitHub Actions / mypy (3.10)

invalid syntax. Perhaps you forgot a comma? [syntax]

Check failure on line 724 in vllm_ascend/model_runner.py

View workflow job for this annotation

GitHub Actions / mypy (3.12)

invalid syntax. Perhaps you forgot a comma? [syntax]
image_grid_thw=image_grid_thw,

Check failure on line 725 in vllm_ascend/model_runner.py

View workflow job for this annotation

GitHub Actions / ruff (3.12)

Ruff

vllm_ascend/model_runner.py:725:25: SyntaxError: Expected ',', found name
video_grid_thw=video_grid_thw,
image_token_id=hf_config.image_token_id,
video_token_id=hf_config.video_token_id,
vision_start_token_id=hf_config.vision_start_token_id,
vision_end_token_id=hf_config.vision_end_token_id,
spatial_merge_size=hf_config.vision_config.
spatial_merge_size,
second_per_grid_ts=None,
context_len=inter_data.context_lens[seq_idx],
seq_len=inter_data.seq_lens[seq_idx],
)
Expand Down

0 comments on commit dce979a

Please sign in to comment.