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

Remove max turns from task #274

Merged
merged 1 commit into from
Sep 3, 2024
Merged

Remove max turns from task #274

merged 1 commit into from
Sep 3, 2024

Conversation

jlowin
Copy link
Member

@jlowin jlowin commented Sep 3, 2024

It's very appealing to limit the number of turns for attempting a task by setting Task(max_turns=...) but unfortunately in practice this isn't straightforward. If a single agent is applied to the task, then it can iterate forever because there is no concept of formally "ending" a turn. On the other hand, we can't use LLM invocations because a turn usually requires two invocations - one to use a tool and another to process it. For the moment, enforcing operational limits at the Task level is too complex. We will leave run() limits in place.

@jlowin jlowin merged commit 19ee8a2 into main Sep 3, 2024
3 checks passed
@jlowin jlowin deleted the remove-max-turns branch September 3, 2024 23:26
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