Skip to content

Conversation

AnilAren
Copy link
Contributor

@AnilAren AnilAren commented Oct 8, 2025

Title

Fixed Image Generation

Relevant issues

Pre-Submission checklist

Please complete all items before asking a LiteLLM maintainer to review your PR

  • I have Added testing in the tests/litellm/ directory, Adding at least 1 test is a hard requirement - see details
  • I have added a screenshot of my new test passing locally
  • My PR passes all unit tests on make test-unit
  • My PR's scope is as isolated as possible, it only solves 1 specific problem

Type

🐛 Bug Fix

Changes

The change in utils.py addresses a missed case — we previously didn’t include invalid parameters when handling optional parameters.

Evidences:

This code change has been verified and tested successfully across four different models.

  1. azure/dall-e-3
  2. azure/gpt-image-1
  3. azure_ai/FLUX.1-Kontext-pro
  4. azure_ai/FLUX-1.1-pro

GPT IMAGE 1

Pasted image 20251008155030 Pasted image 20251008155249

DALL E 3

Pasted image 20251008155617 Pasted image 20251008155651

FLUX - azure_ai/FLUX.1-Kontext-pro

Pasted image 20251008155833 Pasted image 20251008155848

azure_ai/FLUX-1.1-pro

image image

Copy link

vercel bot commented Oct 8, 2025

@anilkumarpeesa is attempting to deploy a commit to the CLERKIEAI Team on Vercel.

A member of the Team first needs to authorize it.

@CLAassistant
Copy link

CLAassistant commented Oct 8, 2025

CLA assistant check
All committers have signed the CLA.

@AnilAren

This comment was marked as duplicate.

@AnilAren
Copy link
Contributor Author

AnilAren commented Oct 8, 2025

The test case failures are not related to my code changes.

These Testcases are failing as part of the master branch.

@AnilAren
Copy link
Contributor Author

AnilAren commented Oct 9, 2025

Please note that the lint errors and test case failures are pre-existing issues and not caused by my recent changes.

@AnilAren
Copy link
Contributor Author

@krrishdholakia Kindly review and let me know if there’s anything I might have missed, so we can resolve it and move ahead with the PR merge.

@AnilAren
Copy link
Contributor Author

@krrishdholakia I’ve refreshed my branch with the latest changes. The lint errors are now resolved, and the test case failures are not related to my changes.

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.

4 participants