search

princeton-nlp / SWE-agent

SWE-agent takes a GitHub issue and tries to automatically fix it, using GPT-4, or your LM of choice. It solves 12.47% of bugs in the SWE-bench evaluation set and takes just 1 minute to run.
https://princeton-nlp.github.io/SWE-agent/
MIT License
11.86k stars 1.18k forks source link

Fix: Process done marker not found in read with timeout #596

Closed klieret closed 1 week ago

klieret commented 1 week ago

Closes #595

Fix: Process done marker not found in read with timeout

This happened when both stderr and stdout were written to. There was no guarantee that the suffix that prints the exit status would actually be last.

Fix:

Either of these should reasonably fix this.

codecov[bot] commented 1 week ago

Codecov Report

Attention: Patch coverage is 80.00000% with 1 line in your changes missing coverage. Please review.

Project coverage is 76.28%. Comparing base (aa94a2e) to head (1307e4d). Report is 3 commits behind head on main.

Files Patch % Lines
sweagent/environment/utils.py 75.00% 1 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #596 +/- ## ========================================== + Coverage 76.20% 76.28% +0.08% ========================================== Files 21 21 Lines 3320 3319 -1 ========================================== + Hits 2530 2532 +2 + Misses 790 787 -3 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.