-
Notifications
You must be signed in to change notification settings - Fork 3
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 error handling for API access issues #214
base: dev
Are you sure you want to change the base?
Improve error handling for API access issues #214
Conversation
- Add specific handling for model access errors in OpenAIOnlineRequestProcessor - Provide clearer error messages with actionable information - Enhance 'all requests failed' error message with better context - Fixes #213
- Enhanced error messages when all API requests fail - Added specific error details to failure messages - Added comprehensive tests for error handling - Maintained backward compatibility with existing functionality - Improved user feedback for API key and model access issues
🤖 Devin AI EngineerI'll be helping with this pull request! Here's what you should know: ✅ I will automatically:
⚙️ Control Options:
Additional ControlsYou can control how I interact with specific messages:
|
Fixes #213 |
There actually looks like there is another error - looking closer at #213
Even though it said it wrote to response file, no response file was written and this leads to that value error (which is raised before getting to your new code) |
- Fix status tracking in handle_single_request_with_retries - Add robust file writing with fsync in append_generic_response - Improve retry queue processing with batching - Add proper error handling for file operations - Ensure failed requests are written to response files
Improve Error Handling for API Access Issues
This PR improves error handling in the curator library by making error messages more informative when API requests fail, particularly for API access and model permission issues.
Changes
Testing
✅ Changes have been tested with:
Implementation Details
Link to Devin run: https://preview.devin.ai/devin/56e42bcae3954c0aa5e6316239b5f717