Skip to content
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

Support for PyTorch torch.distributions.Normal #3033

Closed
itssimon opened this issue Sep 28, 2020 · 10 comments
Closed

Support for PyTorch torch.distributions.Normal #3033

itssimon opened this issue Sep 28, 2020 · 10 comments
Labels
enhancement Request for new feature or operator

Comments

@itssimon
Copy link

Feature Request

Would be great if ONNX would implement support for the broadcast_tensors operator in PyTorch.

Currently torch.onnx.export() fails if the model uses torch.distributions.Normal:

RuntimeError: Exporting the operator broadcast_tensors to ONNX opset version 11 is not supported. Please open a bug to request ONNX export support for the missing operator.

Also see pytorch/pytorch#30517.

@itssimon itssimon added the enhancement Request for new feature or operator label Sep 28, 2020
@mxkrn
Copy link

mxkrn commented Oct 21, 2020

Same deal in opset_version=12.

Exporting the operator broadcast_tensors to ONNX opset version 12 is not supported. Please open a bug to request ONNX export support for the missing operator.

The new PyTorch distributions module makes extensive use of this method so it would be great to see support for this.

@Lenbus
Copy link

Lenbus commented Dec 2, 2020

Have the same deal with PyTorch1.7 on every opset (9 to 12).

@rickstaa
Copy link

This feature and/or pytorch/pytorch#29843 are the only ones stopping us from moving over to Pytorch from Tensorflow. In Pytorch, I could not find an (easy) way to deploy any of our Gaussian and Baysian algorithms wherein Tensorflow is extremely easy.

@stale
Copy link

stale bot commented Mar 28, 2022

Is this still relevant? If so, what is blocking it? Is there anything you can do to help move it forward?

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs.

@stale stale bot added the stale label Mar 28, 2022
@stale stale bot closed this as completed Apr 19, 2022
@rickstaa
Copy link

I think this is still relevant.

@rickstaa
Copy link

Why didn't the bot re-open this 🤔?

@andife
Copy link
Member

andife commented Oct 6, 2023

What is the current situation? I'm also interested in "torch.distributions.normal.Normal"

@rickstaa
Copy link

rickstaa commented Oct 7, 2023

Maybe somebody of the @pytorch team can reopen this issue 🤔?

@andife andife reopened this Oct 7, 2023
@rickstaa
Copy link

rickstaa commented Oct 7, 2023

Thanks! 🙏

@github-actions github-actions bot removed the stale label Oct 8, 2023
@andife andife changed the title Support for PyTorch broadcast_tensors operator Support for PyTorch torch.distributions.Normal Oct 11, 2023
@justinchuby
Copy link
Contributor

Replaced and tracked by pytorch/pytorch#111034

@justinchuby justinchuby closed this as not planned Won't fix, can't repro, duplicate, stale Oct 11, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Request for new feature or operator
Projects
None yet
Development

No branches or pull requests

6 participants