Feat: add add azure endpoint for image endpoints#11482
Conversation
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
|
Looks like there are a few issues preventing this PR from being merged!
If you'd like me to help, just leave a comment, like
Feel free to include any additional details that might help me get this PR into a better state. You can manage your notification settings |
There was a problem hiding this comment.
Pull Request Overview
This PR implements new Azure-compatible endpoints for image generation and image edits while adding corresponding tests to verify proper routing and behavior.
- Added tests for Azure endpoints in tests/test_litellm/proxy/image_endpoints/test_azure_routes.py
- Updated image endpoints in litellm/proxy/image_endpoints/endpoints.py to accept an optional "model" parameter and adjust model selection accordingly
Reviewed Changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| tests/test_litellm/proxy/image_endpoints/test_azure_routes.py | Added tests for Azure endpoints for image generation and image edits |
| litellm/proxy/image_endpoints/endpoints.py | Introduced new POST endpoints for Azure image generation and image edits with updated model selection logic |
Comments suppressed due to low confidence (2)
litellm/proxy/image_endpoints/endpoints.py:67
- Consider updating the function's docstring to document the new 'model' parameter for clarity on its purpose and usage.
model: Optional[str] = None,
litellm/proxy/image_endpoints/endpoints.py:204
- Document the new 'model' parameter in the function's docstring to clearly explain how it affects the endpoint behavior.
model: Optional[str] = None,
| sys.path.insert(0, os.path.abspath("../../..")) | ||
|
|
There was a problem hiding this comment.
[nitpick] Consider configuring PYTHONPATH or using a test runner configuration instead of modifying sys.path directly to improve test maintainability.
| sys.path.insert(0, os.path.abspath("../../..")) |
Feat: add add azure endpoint for image endpoints
This PR implements new Azure-compatible endpoints for image generation and image edits while adding corresponding tests to verify proper routing and behavior.
Relevant issues
Pre-Submission checklist
Please complete all items before asking a LiteLLM maintainer to review your PR
tests/litellm/directory, Adding at least 1 test is a hard requirement - see detailsmake test-unitType
🆕 New Feature
🐛 Bug Fix
🧹 Refactoring
📖 Documentation
🚄 Infrastructure
✅ Test
Changes