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.

Sign up for GitHub

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

Jump to bottom

Separate timeout for commands without output #9

Open
klieret opened this issue Oct 16, 2024 · 0 comments · May be fixed by #152
Open

Separate timeout for commands without output #9

klieret opened this issue Oct 16, 2024 · 0 comments · May be fixed by #152
Labels
enhancement

Comments

@klieret
Copy link
Member

klieret commented Oct 16, 2024

No description provided.

@klieret klieret added feature New feature or request and removed feature New feature or request labels Oct 16, 2024
@klieret klieret added this to the 1.0.0 milestone Oct 21, 2024
@klieret klieret removed this from the 1.0.0 milestone Nov 11, 2024
erkinalp added a commit to erkinalp/SWE-ReX that referenced this issue Dec 20, 2024
@devin-ai-integration @erkinalp
Add separate timeout for commands without output
c82ba4e 
When a command produces no output, it might be hanging or stuck. This
change adds a separate timeout specifically for this case, allowing
more granular control over command execution timeouts.

Changes:
- Add no_output_timeout field to BashAction
- Implement no-output timeout handling in _run_normal
- Update timeout logic to handle both timeouts
- Improve error messages for timeout scenarios

Fixes SWE-agent#9

Co-Authored-By: Erkin Alp Güney <[email protected]>
@erkinalp erkinalp linked a pull request Dec 20, 2024 that will close this issue
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Add separate timeout for commands without output erkinalp/SWE-ReX
1 participant
@klieret