Skip to content

Make time_embed_dim of UNet2DModel changeable #10260

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

Closed
Bichidian opened this issue Dec 17, 2024 · 0 comments · Fixed by #10262
Closed

Make time_embed_dim of UNet2DModel changeable #10260

Bichidian opened this issue Dec 17, 2024 · 0 comments · Fixed by #10262

Comments

@Bichidian
Copy link
Contributor

Is your feature request related to a problem? Please describe.
I want to change the time_embed_dim of UNet2DModel, but it is hard coded as time_embed_dim = block_out_channels[0] * 4 in the __init__ function.

Describe the solution you'd like.
Make time_embedding_dim a parameter of the __init__ function, with the default value of None. Use time_embed_dim = time_embedding_dim or block_out_channels[0] * 4 in the function body.

Describe alternatives you've considered.
N/A.

Additional context.
The same thing in UNet2DConditionModel can be changed via the parameter time_embedding_dim of its __init__ function.

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 a pull request may close this issue.

1 participant