Skip to content

Conversation

@3coins
Copy link
Contributor

@3coins 3coins commented Nov 11, 2025

No description provided.

@3coins 3coins added the enhancement New feature or request label Nov 11, 2025
Copy link
Contributor

@dlqqq dlqqq left a comment

Choose a reason for hiding this comment

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

@3coins Thanks for making these changes! These look good to me, I just left one minor comment below.

'image': extract_image_data(data) if data else None,
"output_type": output_type,
"text": clean_text(data.get("text/plain")),
"image": extract_image_data(data) if data else None,
Copy link
Contributor

Choose a reason for hiding this comment

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

When the agent runs a tool reading a single cell containing an image output, will this cause the encoded image data to be included in the tool output? I think the image data may need to be passed a different way for multimodal model support.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants