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

Improve the error logging for OAI Issues #1987

Merged

Conversation

sadmuphin
Copy link
Contributor

Background

This is a QoL update for a user facing issue that seems to be pretty common: users not setting up paid accounts and inevitably getting hit with a nasty exception vaguely telling them: "Failed to get response after x retries". This doesn't help anyone.

This PR just improves the logging for this common issue by outputting better logs and directing the user to possible causes for hitting the rate limit.

Changes

Added better logging in llm_utils.py in the create_chat_completion function.

PR Quality Checklist

  • My pull request is atomic and focuses on a single change.
  • I have thoroughly tested my changes with multiple different prompts.
  • I have considered potential risks and mitigations for my changes.
  • I have documented my changes clearly and comprehensively.
  • I have not snuck in any "extra" small tweaks changes

@nponeccop nponeccop added the B7 label Apr 16, 2023
@BillSchumacher BillSchumacher merged commit 64e0577 into Significant-Gravitas:master Apr 17, 2023
@nponeccop nponeccop mentioned this pull request Apr 17, 2023
1 task
sindlinger pushed a commit to Orgsindlinger/Auto-GPT-WebUI that referenced this pull request Sep 25, 2024
…ror-msgs

Improve the error logging for OAI Issues
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.

3 participants