Skip to content

fix(vllm-omni): remove invalid syntax in test.py - #12135

Open
H-XX-D wants to merge 2 commits into
mudler:masterfrom
H-XX-D:fix/vllm-omni-test-syntax
Open

H-XX-D wants to merge 2 commits into
mudler:masterfrom
H-XX-D:fix/vllm-omni-test-syntax

Conversation

@H-XX-D

@H-XX-D H-XX-D commented Sep 19, 2026

Copy link
Copy Markdown

What does this PR do?

Fixes a SyntaxError: invalid decimal literal in backend/python/vllm-omni/test.py.

A stray string fragment (additional_information) was concatenated with seed=42 on line 71 (seed=42additional_information) and placed as an isolated identifier on line 81, preventing the test module from being parsed or executed by Python interpreters.

Changes

  • Cleaned seed=42additional_information to seed=42 in GenerateImageRequest
  • Removed stray additional_information token above if __name__ == "__main__":

Signed-off-by: hendrixx-cnc <tjhendrx@icloud.com>
@localai-org-maint-bot

Copy link
Copy Markdown
Collaborator

@mudler code review looks good at e4315c8. This removes exactly the two stray additional_information tokens still present on latest master: one makes the numeric literal invalid, and the other would raise NameError at module import. No other test behavior changes.

The patch applies cleanly to master 8b01583 and git diff --check passes. DCO passes. I could not run Python parsing or the backend suite here because this environment has no Python interpreter; no substantive CI results are available yet. Good to merge from the code-review side, with runtime validation still pending.

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.

2 participants