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

Enhance API error handling & add coverage #206

Open
wants to merge 8 commits into
base: main
Choose a base branch
from

Conversation

devin-ai-integration[bot]
Copy link

  • Improve model validation to return proper 501 status for unsupported models
  • Add NLTK resource download script for test dependencies
  • Fix test configuration and improve error handling
  • Enhance API error messages for better debugging

Link to Devin run: https://app.devin.ai/sessions/830d987bcd9d4ab0a6a73e1e2b98a6d7

devin-ai-integration bot and others added 6 commits December 21, 2024 23:12
- Add retry mechanism with exponential backoff
- Improve error handling for timeouts and memory issues
- Add negative test scenarios for API endpoints
- Update test configuration for better async handling

Co-Authored-By: Stan Girard <[email protected]>
- Update model validation to return 501 for unsupported models
- Add NLTK resource download script
- Fix test configuration and improve error handling

Co-Authored-By: Stan Girard <[email protected]>
Copy link
Author

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR. Add "(aside)" to your comment to have me ignore it.
  • Look at CI failures and help fix them

⚙️ Control Options:

  • Disable automatic comment and CI monitoring

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.

0 participants