-
Notifications
You must be signed in to change notification settings - Fork 4.6k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[Bug]: agent_react_multimodal_step.py bug and possible fix provided #11389
Comments
🤖 Hey there, @alexoli97! Long time no code 😄. How's everything on your end? Thank you for providing the details of the error you're encountering. The Based on the function definition you provided, def add_user_step_to_reasoning(
generate_chat_message_fn: ChatMessageCallable,
step: TaskStep,
memory: BaseMemory,
current_reasoning: List[BaseReasoningStep],
verbose: bool = False,
) -> None: The error message suggests that To resolve this issue, you'll need to identify where As for the fix you've suggested, without seeing the actual code it's hard to say whether it would resolve the issue. However, based on the information you've provided, it seems like a reasonable approach. Please ensure that your modifications maintain the original functionality of the For more information on how to debug this type of error, you might find this StackOverflow thread helpful. It explains the error in more detail and provides some general advice on how to fix it. I hope this helps! If you have any further questions or need more assistance, please don't hesitate to ask. Sources
|
I am having this issue as well @alexoli97 , could you point out specifically what you changed in the file to fix it? I cannot see any changes when comparing your comment to the current file. |
just delete this i think self._add_user_step_to_reasoning = partial( |
@dosu-bot here is the function
|
Being explicit with the args here got it working for me: self._add_user_step_to_reasoning(
step=step,
memory=task.extra_state["new_memory"],
current_reasoning=task.extra_state["current_reasoning"],
verbose=self._verbose,
) |
Bug Description
in : response = execute_steps(agent, task)
error:
TypeError Traceback (most recent call last)
in <cell line: 2>()
1 # Run this and not the below if you just want to run everything at once.
----> 2 response = execute_steps(agent, task)
6 frames
/usr/local/lib/python3.10/dist-packages/llama_index/core/agent/react_multimodal/step.py in _run_step(self, step, task)
388 # an intermediate step in the middle
389 if step.input is not None:
--> 390 self._add_user_step_to_reasoning(
391 step,
392 task.extra_state["new_memory"],
TypeError: add_user_step_to_reasoning() got multiple values for argument 'generate_chat_message_fn'
Version
v0.10.12
Steps to Reproduce
follow this his guide: https://docs.llamaindex.ai/en/stable/examples/multi_modal/mm_agent.html
Relevant Logs/Tracbacks
The text was updated successfully, but these errors were encountered: