Skip to content

Conversation

@mnlscn
Copy link
Contributor

@mnlscn mnlscn commented Dec 24, 2025

Problem

  • The execute() method in PydanticTransducerVLLM class was referencing an undefined variable state instead of the parameter input when handling single string inputs.
  • Packages Imports were not defined at the beginning of the file. There were some duplicate imports in both async_executor.py and pyproject.toml

…cerVLLM.execute(). The execute() method was referencing an undefined variable 'state' instead of parameter 'input'. Also, removed double dependency reference in pyproject.toml and fixed package imports in async_executor.py
Copy link
Collaborator

Choose a reason for hiding this comment

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

I think we will need jsonfinder, not sure about crewai[google-gebain], @gliozzo do you know if it's required?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Hi, I removed jsonfinder and crewai[google-genai] from lines 39 and 41 because they are duplicates. They are already defined on lines 21 and 38.

Copy link
Collaborator

Choose a reason for hiding this comment

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

OK, that makes sense why there's no updates on the uv lock. Thanks

@D3f0 D3f0 merged commit 378e4f9 into IBM:main Jan 6, 2026
2 of 3 checks passed
abhigoel25 pushed a commit to Chaitya0623/Agentics-Research that referenced this pull request Jan 14, 2026
…ecutor-and-dependencies-management

Fix: correct undefined variable 'state' to 'input' in PydanticTransducerVLLM.execute() and dependencies imports
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