-
Notifications
You must be signed in to change notification settings - Fork 3.5k
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
[Frontend][ONNX] Support RandomNormal operator #9493
Conversation
Thanks for contribution! I'm working on upgrading ONNX in our CI. The new version seems to have a few official tests for these operators (even though they've been around since opset 1) which I will run by hopefully by the end of this week. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is an excellent change and LGTM. I'm curious what tests you're referring to @AndrewZhaoLuo?
Huh, you know what, im not sure anymore, looking at this pr: https://github.com/apache/tvm/pull/9519/files, can't seem to find the tests anymore. Must have been mistaken or confused. Go ahead and merge |
Awesome, thanks for this contribution @shengxinhu! |
support RandomUniformLike, RandomNormal and RandomNormalLike