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

Implement relay nn.bias_add compute in C++ #3027

Merged
merged 3 commits into from
Apr 17, 2019
Merged

Conversation

yinghai
Copy link
Contributor

@yinghai yinghai commented Apr 15, 2019

Relay nn.bias_add now has a compute implementation in Python, which is inconvenient for C++ flow given many of the Relay nodes already are implemented in C++. This PR ports the compute from Python to C++.

@yinghai
Copy link
Contributor Author

yinghai commented Apr 15, 2019

@hlu1 @ajtulloch

@yinghai
Copy link
Contributor Author

yinghai commented Apr 16, 2019

Comments addressed.

Copy link
Contributor

@hlu1 hlu1 left a comment

Choose a reason for hiding this comment

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

LGTM

@Laurawly Laurawly merged commit 12257dd into apache:master Apr 17, 2019
@yinghai yinghai deleted the add_bias branch April 17, 2019 20:09
larroy pushed a commit to larroy/tvm that referenced this pull request Apr 19, 2019
* Implement nn.bias_add compute in C++

* Address comments

* Remove unnecessary check
wweic pushed a commit to wweic/tvm that referenced this pull request May 13, 2019
* Implement nn.bias_add compute in C++

* Address comments

* Remove unnecessary check
wweic pushed a commit to neo-ai/tvm that referenced this pull request May 13, 2019
* Implement nn.bias_add compute in C++

* Address comments

* Remove unnecessary check
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