diffusers icon indicating copy to clipboard operation
diffusers copied to clipboard

Fix SVD bug (shape of `time_context`)

Open KimbingNg opened this issue 1 year ago • 0 comments

What does this PR do?

Fixes issue

The shape of time_context should match the shape of hidden_states in this line. Thus, it should be (b h w) n c instead of (h w b) n c.

Before submitting

  • [ ] This PR fixes a typo or improves the docs (you can dismiss the other checks if that's the case).
  • [x] Did you read the contributor guideline?
  • [x] Did you read our philosophy doc (important for complex PRs)?
  • [ ] Was this discussed/approved via a GitHub issue or the forum? Please add a link to it if that's the case.
  • [ ] Did you make sure to update the documentation with your changes? Here are the documentation guidelines, and here are tips on formatting docstrings.
  • [ ] Did you write any new necessary tests?

Who can review?

Anyone in the community is free to review the PR once the tests have passed. Feel free to tag members/contributors who may be interested in your PR.

KimbingNg avatar Mar 10 '24 16:03 KimbingNg