CyanideByte

Results 21 comments of CyanideByte
trafficstars

Ya this seems like a reasonable thing to fix. I made a PR for it: https://github.com/OpenInterpreter/open-interpreter/pull/1203

https://github.com/BerriAI/litellm/pull/3176 Potentially already being done by litellm

batch is supported under the cross platform category named "shell"

> Thank you! Although, I am having difficulty finding anything about "cross platform" or enabling shell in the read me and in the wiki. Can you please share a link...

- Restored the if statement I didn't notice I removed before. - Updated my poetry files to fix the merge conflict.

I tested on Windows and confirmed it does fix the issue. Excellent PR. @imapersonman do you know if there is an issue about this opened on litellm yet?

Good idea. For now I made a fix PR for the latest ones that popped up. https://github.com/BerriAI/litellm/pull/3334 Added a pytest for it to my PR.

Running `pip install --upgrade litellm` will fix this issue.

I verified that its more than just missing output. The code doesn't actually get run either. The code flow gets into the terminal.py run() function but never made it to...

> Does the same error happen when using python 3.10? Happens to me on conda python 3.11 too.