Skip to content

Conversation

pcuenca
Copy link
Member

@pcuenca pcuenca commented Sep 6, 2022

As suggested by @bes-dev. According to the documentation this shouldn't take any memory - it just plays with the strides.

As suggested by @bes-dev. According the documentation this shouldn't
take any memory - it just plays with the strides.
@HuggingFaceDocBuilderDev
Copy link

HuggingFaceDocBuilderDev commented Sep 6, 2022

The documentation is not available anymore as the PR was closed or merged.

@bes-dev
Copy link

bes-dev commented Sep 6, 2022

👍

@patrickvonplaten patrickvonplaten merged commit 56c0037 into main Sep 6, 2022
@patil-suraj patil-suraj deleted the onnx-expand branch September 7, 2022 16:36
natolambert pushed a commit that referenced this pull request Sep 7, 2022
Use `expand` instead of ones to broadcast tensor.

As suggested by @bes-dev. According the documentation this shouldn't
take any memory - it just plays with the strides.
yoonseokjin pushed a commit to yoonseokjin/diffusers that referenced this pull request Dec 25, 2023
Use `expand` instead of ones to broadcast tensor.

As suggested by @bes-dev. According the documentation this shouldn't
take any memory - it just plays with the strides.
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.

5 participants