open-interpreter icon indicating copy to clipboard operation
open-interpreter copied to clipboard
verified publisher icon OpenInterpreter

Fix output not str

Open Notnaton opened this issue 1 year ago • 1 comments

output is a list list.lower() errors

Describe the changes you have made:

Reference any relevant issues (e.g. "Fixes #000"):

Pre-Submission Checklist (optional but appreciated):

  • [ ] I have included relevant documentation updates (stored in /docs)
  • [x] I have read docs/CONTRIBUTING.md
  • [x] I have read docs/ROADMAP.md

OS Tests (optional but appreciated):

  • [ ] Tested on Windows
  • [ ] Tested on MacOS
  • [ ] Tested on Linux

Notnaton avatar Mar 22 '24 20:03 Notnaton

FYI I get AttributeError: 'dict' object has no attribute 'lower' with this fix

SgtPooki avatar Mar 24 '24 05:03 SgtPooki

Nice Anton, thanks! this is a super clean fix, I just merged in one that's a little dirtier but slightly more robust: if "traceback" in str(output).lower()". Will close this one, please re-open if this PR catches any cases that the new fix would miss!

KillianLucas avatar Mar 28 '24 02:03 KillianLucas