Skip to content

Conversation

@siju-samuel
Copy link
Member

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 Reviewers.

@yzhliu
Copy link
Member

yzhliu commented Nov 26, 2018

Thanks, please request review.

@siju-samuel
Copy link
Member Author

@MarisaKirisame @jroesch @yzhliu @tqchen please help to review this PR. TIA

@jroesch
Copy link
Member

jroesch commented Nov 27, 2018

Looks good to me, would be good to merge this before #2144 so we can synchronize on the operator implementations.

const Array<Tensor>& inputs,
const Type& out_type,
const Target& target) {
return { topi::full_like(inputs[0], inputs[1]()) };
Copy link
Contributor

Choose a reason for hiding this comment

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

If I am you, i will CHECK inputs size before return.
Maybe we should have a convention for how much CHECKing we should do? idk.

Copy link
Member Author

Choose a reason for hiding this comment

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

input size is verified at the FullLikeRel, I guess again checking will b reduntant. most of new compute func doesnt have this checks.

@tqchen tqchen merged commit 246a38a into apache:master Nov 27, 2018
@siju-samuel siju-samuel deleted the relay_full_comp_schd branch November 27, 2018 06:42
FrozenGene pushed a commit to FrozenGene/tvm that referenced this pull request Dec 27, 2018
wweic pushed a commit to neo-ai/tvm that referenced this pull request Feb 20, 2019
wweic pushed a commit to neo-ai/tvm that referenced this pull request Feb 20, 2019
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