Fix error handling and clean up client

anarchy-ai / LLM-VM

irresponsible innovation. Try now at https://chat.dev/

https://anarchy.ai/

MIT License

481 stars 145 forks source link

Fix error handling and clean up client #431

Open mmirman opened 10 months ago

mmirman commented 10 months ago

https://github.com/anarchy-ai/LLM-VM/blob/main/src/llm_vm/client.py#L197C31-L197C31

This error handling is chaotic! We loose the stack trace frequently and there is unnecessary redirection. Please keep the stack trace so we can figure out what line things got broken on.

We use rebel even when tools is empty also!

abhigya-sodani commented 10 months ago

Whats an example of using REBEL w/o tools

mmirman commented 10 months ago

Thats the point. If you give it a tools object with no tools what happens?

abhigya-sodani commented 10 months ago

Ok working on this. Pushed a fix that fixes Openai issues

lucylililiwang commented 9 months ago

Hi, Can I please work on this issue? Thank you!

lucylililiwang commented 9 months ago

https://github.com/lucylililiwang/LLM-VM/blob/lucylililiwang-Fix_error_handling_and_clean_up_client/Fix%20error%20handling%20and%20clean%20up%20client