-
Notifications
You must be signed in to change notification settings - Fork 3.4k
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
fix(controller): "Current task state not recognized" #1517
Conversation
This reverts commit f5cbfe1.
…ask-state-not-recognize
@amanape I tried to fix the reset issue; based on my testing, you should be able to reset tasks easily. However, I'm debating regarding whether the PAUSE button should have any effect when the agent is awaiting user input. Awaiting user input is technically a type of "pause" itself? WDYT? |
@xingyaoww You're right, disabling the pause button while the agent is waiting makes more sense. Another alternative would be your suggestions in #1508 Personally, I think
is a good one. |
@xingyaoww @amanape i slightly adjust the reset behavior here. #1519 |
* fix agent task already running bug * attemp to fix "Current task state not recognized" * Revert "attemp to fix "Current task state not recognized"" This reverts commit f5cbfe1. * attemp to fix "Current task state not recognized" * fix invalid state for reset --------- Co-authored-by: Leo <ifuryst@gmail.com>
Prerequisite: merge #1516
Fix #1492 (comment)