PixArtAlphaPipeline does not handle negative_prompt list correctly

[Trojaner]

Describe the bug

Passing negative_prompt as list to PixArtAlphaPipeline will eventually cause a "list object has no attribute lower" error.

The encode_prompt function documentation shows that it can be a list as well:

diffusers/src/diffusers/pipelines/pixart_alpha/pipeline_pixart_alpha.py

Line 308 in 7404f1e

negative_prompt (`str` or `List[str]`, *optional*):

However the function type hints restrict it to a str instead:

diffusers/src/diffusers/pipelines/pixart_alpha/pipeline_pixart_alpha.py

Line 291 in 7404f1e

negative_prompt: str = "",

These restrictions are completely arbitrary, since
a) prompt is correctly handled for the same case,
b) negative_prompt is only used in one place and there it just gets converted to a list anyway:

diffusers/src/diffusers/pipelines/pixart_alpha/pipeline_pixart_alpha.py

Line 392 in 7404f1e

uncond_tokens = [negative_prompt] * batch_size

Reproduction

Call Pixart-Sigma with negative_prompt as list instead of a single string.

Logs

(sorry for images)

System Info

Windows 11
PyTorch 2.2.2 with CUDA support
Diffusers 0.28.0.dev0 @ 7404f1e

Who can help?

@DN6 @sayakpaul @lawrence-cj

[sayakpaul]

Feel free to drop a PR to fix it :)

[Trojaner]

Feel free to drop a PR to fix it :)

No.

What's the point of bug reports if we must fix it ourselves?
Then just remove the option to report bugs and only accept PRs at this point.

I already contributed here by pinpointing the issue to the point where fixing it is trivial.

[sayakpaul]

I already contributed here by pinpointing the issue to the point where fixing it is trivial.

That is exactly why I asked you to submit a PR. Since you already have a thorough understanding of what's going on and know how to fix the issue. That way, your contributions stay within the repository, and that is very much appreciated.

If you don't prefer that, sure, I am just happy to open a PR on behalf of yourself. What is your commit email id so that I can add your as a co-author of the commit?

[Trojaner]

That is exactly why I asked you to submit a PR.

Sadly your initial post is very open to misunderstandings. If you had included more explanation on why you want me to do it, it would have shown me that it wasn't a toxic comment. I will submit a PR in the next days.

[sayakpaul]

Sorry for not being clear. Appreciate your understanding.