Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Retry connecting to openai after hitting rate limit to fix #442 #452

Merged
merged 1 commit into from
Sep 20, 2023
Merged

Conversation

mathiasrw
Copy link
Contributor

Retry connecting to openai after hitting rate limit to fix #442

Describe the changes you have made:

Make sure to keep the process running even if we encounter a rate limit in the openAI api.

Reference any relevant issue (Fixes #000)

Will fix #442

  • I have performed a self-review of my code:

I have tested the code on the following OS:

  • Windows
  • MacOS
  • Linux

AI Language Model (if applicable)

  • GPT4
  • GPT3
  • Llama 7B
  • Llama 13B
  • Llama 34B
  • Huggingface model (Please specify which one)

How it looks

image
image

Note how this process would have been lost if I had not been running on code that included this PR.

@KillianLucas
Copy link
Collaborator

Simple and clean. Thanks @mathiasrw, great contribution!

@KillianLucas KillianLucas merged commit 49f5aa5 into OpenInterpreter:main Sep 20, 2023
joshuavial pushed a commit to joshuavial/open-interpreter that referenced this pull request Nov 16, 2023
Retry connecting to openai after hitting rate limit to fix OpenInterpreter#442

Former-commit-id: 49f5aa5
Former-commit-id: a08dbc84fbca43c49151de42c9e055e952d4bd6f
Former-commit-id: 76da81d9d49e329ec7a2673788f561e41d38d508 [formerly 1354e30c81c9fa164b6253355ec2b2aad4892551]
Former-commit-id: 1eca2b5ce1faf0f11bb26201bc547efa83e45d7b
joshuavial pushed a commit to joshuavial/open-interpreter that referenced this pull request Nov 16, 2023
Retry connecting to openai after hitting rate limit to fix OpenInterpreter#442
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Rate limit will kill the process
2 participants