Skip to content

Comments

[Opal ADK] Adds the ability to send binary data to Opal ADK#7857

Open
walker7734 wants to merge 6 commits intomainfrom
opal-adk-files
Open

[Opal ADK] Adds the ability to send binary data to Opal ADK#7857
walker7734 wants to merge 6 commits intomainfrom
opal-adk-files

Conversation

@walker7734
Copy link
Member

Fixes #<issue_number_goes_here>

It's a good idea to open an issue first for discussion.

  • Tests pass
  • Appropriate changes to documentation are included in the PR

…ing off a session id to resume a previously started agent conversation in Opal ADK.
… technically works. However, tuning will need to be done to the prompts provided by the planner and the substituting logic in Breadboard. The Opal ADK agent gets very confused when an input is referenced but it doesn't see it as part of its input. This is because binary input is stored in artifacts in ADK. I updated some prompts on the backend to indicate it should look in the artifacts to find the uploads and this helps but it has a way to go. This will provide a good place to start as it will still send the data to Opal ADK and you can see the agent bang its head against a wall.
@github-actions
Copy link

📊 Coverage Report

Metric PR Main Delta
Lines 98.91% 98.91% ⚪ +0.00%
Functions 99.54% 99.54% ⚪ +0.00%
Branches 94.66% 94.66% ⚪ +0.00%

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.

1 participant