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

[NNVM][TOP] broadcast versions corresponding to topi: mod, max, min, pow, left_shift, right_shift greater, less, equal, not_equal, greater_equal and less_equal. #1383

Merged
merged 4 commits into from
Jul 8, 2018

Conversation

srkreddy1238
Copy link
Contributor

- pow operator support for onnx frontend.

Thanks for contributing to TVM! Please refer to guideline https://docs.tvm.ai/contribute/ for useful information and tips. After the pull request is submitted, please request code reviews from others in the community.

…n, pow left_shift, right_shift greater, less
@srkreddy1238 srkreddy1238 changed the title [NNVM[TOP] broadcast versions corresponding to topi: mod, max, min, pow, left_shift, right_shift greater, less. [NNVM][TOP] broadcast versions corresponding to topi: mod, max, min, pow, left_shift, right_shift greater, less. Jul 5, 2018
@srkreddy1238
Copy link
Contributor Author

@zhreshold @masahi welcome for a review.

@srkreddy1238
Copy link
Contributor Author

@masahi thanks for the review.

@tqchen
Copy link
Member

tqchen commented Jul 5, 2018

Copy link
Member

@masahi masahi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@srkreddy1238 ok, please add test case for all new ops you introduced.

@srkreddy1238
Copy link
Contributor Author

@tqchen thanks for asking test cases. I came across a bug while adding test cases.

@masahi please review.

@srkreddy1238 srkreddy1238 changed the title [NNVM][TOP] broadcast versions corresponding to topi: mod, max, min, pow, left_shift, right_shift greater, less. [NNVM][TOP] broadcast versions corresponding to topi: mod, max, min, pow, left_shift, right_shift greater, less, equal, not_equal, greater_equal and less_equal. Jul 6, 2018
@srkreddy1238
Copy link
Contributor Author

@masahi ready for review now. Thanks

Copy link
Member

@masahi masahi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good.

@tqchen tqchen merged commit 0d57086 into apache:master Jul 8, 2018
@tqchen
Copy link
Member

tqchen commented Jul 8, 2018

thanks @masahi for the helpful reviews, thanks @srkreddy1238 for the contribution, this is now merged!

tqchen pushed a commit to tqchen/tvm that referenced this pull request Aug 4, 2018
…pow, left_shift, right_shift greater, less, equal, not_equal, greater_equal and less_equal. (apache#1383)
sergei-mironov pushed a commit to sergei-mironov/tvm that referenced this pull request Aug 8, 2018
…pow, left_shift, right_shift greater, less, equal, not_equal, greater_equal and less_equal. (apache#1383)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants