-
Notifications
You must be signed in to change notification settings - Fork 3k
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: add reconnect action. #1519
Conversation
Codecov ReportAttention: Patch coverage is
❗ Your organization needs to install the Codecov GitHub app to enable full functionality. Additional details and impacted files@@ Coverage Diff @@
## main #1519 +/- ##
=======================================
Coverage ? 60.76%
=======================================
Files ? 87
Lines ? 3693
Branches ? 0
=======================================
Hits ? 2244
Misses ? 1449
Partials ? 0 ☔ View full report in Codecov by Sentry. |
Signed-off-by: ifuryst <ifuryst@gmail.com>
…action Signed-off-by: ifuryst <ifuryst@gmail.com> # Conflicts: # opendevin/server/session/msg_stack.py
2722ac6
to
47a7793
Compare
Signed-off-by: ifuryst <ifuryst@gmail.com>
47a7793
to
81e5335
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.
LGTM +1 strongly typed docstring
@iFurySt lgtm! Just need to fix the conflicts |
@rbren okay, i'll handle it tomorrow. |
Hi @iFurySt , I'm just looking through the backlog and was just wondering what you'd like to do with this PR? Fix the conflicts? Close and possibly do in a different PR? Any way is fine, just checking! |
I'll close this PR, it's been behind too long. |
refreshing the page
orsocket reconnect
: try to reconnect to the already initialized agent. if it doesn't exist, auto init.reset button
: init agent includes related envs like docker containers.change settings
: init agent, same as step 2.Screen.Recording.2024-05-02.at.21.42.20.mov