env log can be left disabled when initializing multiple `SWEEnv`s

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.88k stars 1.19k forks source link

env log can be left disabled when initializing multiple `SWEEnv`s #536

Closed klieret closed 3 weeks ago

klieret commented 1 month ago

SWEEnv.logger is actually the global variable logger. If one SWEEnv has verbose=False, it will not be corrected for the next one.

Probably only an issue when testing for log messages

klieret commented 1 month ago

Solve together with #501