-
Notifications
You must be signed in to change notification settings - Fork 5.6k
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
Incorrect Parameter Type on UNet2DModel #4806
Comments
Nice catch! Would you like to open a PR to correct it? |
May I ? |
My plate is full |
This issue has been automatically marked as stale because it has not had recent activity. If you think this still needs to be addressed please comment on this thread. Please note that issues that do not follow the contributing guidelines are likely to be ignored. |
fpgaminer
added a commit
to fpgaminer/diffusers
that referenced
this issue
Mar 1, 2024
6 tasks
stevhliu
pushed a commit
that referenced
this issue
Mar 4, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Describe the bug
The parameter type for
UNet2DModel::__init__
seems to be incorrect, at least on these three parameters:diffusers/src/diffusers/models/unet_2d.py
Lines 97 to 99 in c4d2823
I believe the correct type is
Tuple[str, ...]
andTuple[int, ...]
which means "A tuple of any size, where all elements are of type str (or int)". The current type,Tuple[str]
means "A tuple that must be of size 1 and that one element is of type str", which causes typing errors when trying to use UNet2DModel.Reproduction
A typical instantiation of UNet2DModel like this:
Logs
System Info
Who can help?
@patrickvonplaten @sayakpaul
The text was updated successfully, but these errors were encountered: