-
Notifications
You must be signed in to change notification settings - Fork 44.8k
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(backend/executor): Add TERMINATED
execution status
#9185
feat(backend/executor): Add TERMINATED
execution status
#9185
Conversation
✅ Deploy Preview for auto-gpt-docs-dev canceled.
|
✅ Deploy Preview for auto-gpt-docs canceled.
|
TERMINATED
execution status
Formerly known as "FAILED" with error message "TERMINATED".
e3d6906
to
0df733d
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good,
Will this part of the code also capture the TERMINATED signal?
https://github.com/Significant-Gravitas/AutoGPT/pull/9185/files#diff-1b278ebf10a9da0fb5030010222b3a6df2b05a5463cad428cd6c38a1541b0f73R204-R220
If yes then it could be the terminated status is overridden? if not then it's all good :)
I was having such an issue at first, which is why I did this: I tested it and now it works. :) |
Formerly known as
FAILED
with error messageTERMINATED
.Changes 🏗️
TERMINATED
toAgentExecutionStatus
enum in DB schema (and its mirror in the front end)TERMINATED
instead ofFAILED
/COMPLETED
TERMINATED
case to status checks referencingAgentExecutionStatus
Checklist 📋
For code changes: