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

Add TVMC Frontend for PaddlePaddle #9083

Merged
merged 48 commits into from
Oct 6, 2021

Conversation

jiangjiajun
Copy link
Contributor

@jiangjiajun jiangjiajun commented Sep 23, 2021

PaddlePaddle is supported to import by relay.frontend.from_paddle, this pull request implements a TVM frontend for paddlepaddle, to provide a more efficient way to users.

This pull request has requirements of paddlepaddle>=2.1.3, so we should merge this after ci docker image updated which can be tracked in #9082

@leandron
Thanks for contributing to TVM! Please refer to guideline https://tvm.apache.org/docs/contribute/ for useful information and tips. After the pull request is submitted, please request code reviews from Reviewers by @ them in the pull request thread.

jiangjiajun and others added 30 commits August 3, 2021 17:14
Add padding mode support for conv/pool2d
Copy link
Contributor

@leandron leandron left a comment

Choose a reason for hiding this comment

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

Generally LGTM - can you add a test case to cover this implementation?

@leandron
Copy link
Contributor

As mentioned in #9098, can you also change gen_requirements.py to add paddlepaddle into the "tvmc" section, together with all other frontends?

@jiangjiajun
Copy link
Contributor Author

As mentioned in #9098, can you also change gen_requirements.py to add paddlepaddle into the "tvmc" section, together with all other frontends?

No problem, I'll modify it and add test cases with this pull request

Copy link
Contributor

@leandron leandron left a comment

Choose a reason for hiding this comment

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

I'm happy with the current implementation, so I think we can merge this as soon as we get CI green. Thanks for this PR @jiangjiajun!

@jiangjiajun
Copy link
Contributor Author

@leandron Sorry, I was handling another PR last week, but I noticed there's a request for updating the ci image, and already passed the testing. After #9199 merged, could you help to rerun the ci for this pull request?

@leandron
Copy link
Contributor

leandron commented Oct 5, 2021

@leandron Sorry, I was handling another PR last week, but I noticed there's a request for updating the ci image, and already passed the testing. After #9199 merged, could you help to rerun the ci for this pull request?

You need to retrigger CI by force pushing on the branch, to "update" this PR, then CI will run automatically.

@masahi masahi merged commit f0bcfa7 into apache:main Oct 6, 2021
ylc pushed a commit to ylc/tvm that referenced this pull request Jan 7, 2022
* fix some problems for matmul

* fix some problems for matmul

* add alpha parameter for matmul

* remove unnecessary condition

* add TranslatedLayer which support model loaded by jit.load

* add mul operator support

* Add padding mode support for conv/pool2d

* support 4 two-tuples

* add paddle test case

* add paddle conv2d  case

* update test_forward.py

* fix paddle convert_matmul

* add paddle multiply and matmul op test case

* add test case and fix bug

* delete import pandas

* add paddlepaddle tests

* modify the variable name of convert_reshape

* formatting

* formatting

* use black to format python code

* pylint check

* Remove fluid api

* black format

* Add Paddle Frontend for TVMC

* refine code format

* add test case for tvmc

* fix pylint check

* gen_requirements add paddlepaddle

* Trigger CI

Co-authored-by: root <root@bjyz-sys-gpu-kongming3.bjyz.baidu.com>
Co-authored-by: wjj19950828 <wjjisloser@163.com>
Co-authored-by: heliqi <1101791222@qq.com>
Co-authored-by: Junru Shao <junrushao1994@gmail.com>
ylc pushed a commit to ylc/tvm that referenced this pull request Jan 13, 2022
* fix some problems for matmul

* fix some problems for matmul

* add alpha parameter for matmul

* remove unnecessary condition

* add TranslatedLayer which support model loaded by jit.load

* add mul operator support

* Add padding mode support for conv/pool2d

* support 4 two-tuples

* add paddle test case

* add paddle conv2d  case

* update test_forward.py

* fix paddle convert_matmul

* add paddle multiply and matmul op test case

* add test case and fix bug

* delete import pandas

* add paddlepaddle tests

* modify the variable name of convert_reshape

* formatting

* formatting

* use black to format python code

* pylint check

* Remove fluid api

* black format

* Add Paddle Frontend for TVMC

* refine code format

* add test case for tvmc

* fix pylint check

* gen_requirements add paddlepaddle

* Trigger CI

Co-authored-by: root <root@bjyz-sys-gpu-kongming3.bjyz.baidu.com>
Co-authored-by: wjj19950828 <wjjisloser@163.com>
Co-authored-by: heliqi <1101791222@qq.com>
Co-authored-by: Junru Shao <junrushao1994@gmail.com>
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.

6 participants