Skip to content
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

Feat/dynamic agent input #18

Closed
wants to merge 2 commits into from
Closed

Feat/dynamic agent input #18

wants to merge 2 commits into from

Conversation

Q1w1N
Copy link
Collaborator

@Q1w1N Q1w1N commented Jan 24, 2025

This will allow for dynamic input generation. So the context can be aggregated outside of the flow and accessed in the runtime.

@grabbou
Copy link
Contributor

grabbou commented Jan 26, 2025

In the current shape, this is unlikely to get merged, as it breaks serializability. Some other agents expect input to be a string, so they can possibly tweak it, e.g. the evaluator may tweak the prompt to ask you to "repair it".

I would consider turning it into an issue / following up on Discord, where we can discuss how to tackle this best!I believe this can be solved with a tool call (if you need to access some dynamic data) or just create your agent as a closure with access to this.

@Q1w1N Q1w1N closed this Jan 27, 2025
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