Skip to content

Add support for MXNet pad operator, for parameter 'None' in slice operator, 1D Convolution and 1D Deconvolution - #3739

Merged
kevinthesun merged 1 commit into
apache:masterfrom
alexgl-github:mxnet-pad2
Sep 20, 2019
Merged

Add support for MXNet pad operator, for parameter 'None' in slice operator, 1D Convolution and 1D Deconvolution#3739
kevinthesun merged 1 commit into
apache:masterfrom
alexgl-github:mxnet-pad2

Conversation

@alexgl-github

Copy link
Copy Markdown
Contributor

MXNet pad is described at:
https://mxnet.incubator.apache.org/api/python/symbol/symbol.html#mxnet.symbol.pad

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.

struct PadAttrs : public tvm::AttrsNode<PadAttrs> {
double pad_value;
Array<Array<IndexExpr> > pad_width;
std::string pad_mode;

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Can we add test cases for different modes?

Copy link
Copy Markdown
ContributorAuthor

Choose a reason for hiding this comment

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

I've added tests, please review again

@alexgl-githubalexgl-github changed the title Add support for MXNet pad operator.Add support for MXNet pad operator, for parameter 'None' slice operator, 1D Convolution and 1D DeconvolutionAug 21, 2019
@alexgl-githubalexgl-github changed the title Add support for MXNet pad operator, for parameter 'None' slice operator, 1D Convolution and 1D DeconvolutionAdd support for MXNet pad operator, for parameter 'None' in slice operator, 1D Convolution and 1D DeconvolutionAug 21, 2019
@yongwww

Copy link
Copy Markdown
Member

@kevinthesun @icemelon9 @zhiics great to get feedback from any of you guys

Comment threadpython/tvm/relay/frontend/common.py Outdated
Comment threadpython/tvm/relay/frontend/common.py Outdated
@kevinthesun

Copy link
Copy Markdown
Contributor

@alexgl-github Can you resolve the conflict and take a look at the ci issue?

@alexgl-github

Copy link
Copy Markdown
ContributorAuthor

@alexgl-github Can you resolve the conflict and take a look at the ci issue?
done, please review again

@kevinthesun

Copy link
Copy Markdown
Contributor

@alexgl-github looks like sanity check is still failing.

@alexgl-github

Copy link
Copy Markdown
ContributorAuthor

@alexgl-github looks like sanity check is still failing.

@kevinthesun please review again.

@tqchentqchen added the status: need update need update based on feedbacks label Sep 13, 2019
@alexgl-github
alexgl-githubforce-pushed the mxnet-pad2 branch 5 times, most recently from eaad44b to da0bc0eCompareSeptember 13, 2019 22:09
@kevinthesun
kevinthesun merged commit 719d6d4 into apache:masterSep 20, 2019
@kevinthesun

Copy link
Copy Markdown
Contributor

Thanks!

anijain2305 added a commit to anijain2305/tvm that referenced this pull request Sep 21, 2019
tqchen pushed a commit that referenced this pull request Sep 23, 2019
wweic pushed a commit to wweic/tvm that referenced this pull request Sep 30, 2019
wweic pushed a commit to wweic/tvm that referenced this pull request Sep 30, 2019
wweic pushed a commit to wweic/tvm that referenced this pull request Sep 30, 2019
wweic pushed a commit to wweic/tvm that referenced this pull request Sep 30, 2019
wweic pushed a commit to neo-ai/tvm that referenced this pull request Oct 1, 2019
wweic pushed a commit to neo-ai/tvm that referenced this pull request Oct 1, 2019
@alexgl-github
alexgl-github deleted the mxnet-pad2 branch November 13, 2019 01:05
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

status: need updateneed update based on feedbacks

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

@alexgl-github@yongwww@kevinthesun@tqchen