Skip to content

Conversation

hanouticelina
Copy link
Contributor

Related to huggingface/huggingface_hub#3399, some providers or models may ignore this parameter if it's not supported. This PR updates the docstring to make that clearer and align it with the one in huggingface_hub.

Copy link
Member

@pcuenca pcuenca left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you! 🙌 Is there a way to know the providers / models that support it?

Copy link
Contributor

@Wauplin Wauplin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@hanouticelina
Copy link
Contributor Author

Thank you! 🙌 Is there a way to know the providers / models that support it?

@pcuenca no there is no automatic way to check that unfortunately - someone has to look it up in each provider's documentation. Also for a given provider, some models might support it while others don't

@hanouticelina hanouticelina merged commit 9d8cb9a into main Oct 7, 2025
4 of 5 checks passed
@hanouticelina hanouticelina deleted the update-image-to-image-params-docstring branch October 7, 2025 12:54
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