einops icon indicating copy to clipboard operation
einops copied to clipboard

Published 20 hours ago • verified publisher icon arogozhnikov

Add PaddlePaddle backend.

Open jm12138 opened this issue 3 years ago • 2 comments

  • Add paddlepaddle deep learning framework support.

jm12138 avatar Jun 06 '21 15:06 jm12138

Hello and thanks for contributing.

I don't merge this PR for two reasons:

  • paddlepaddle doesn't handle zero-dimensional tensor properly, see this issue, einops tests are failing and the same issue with previous versions of mxnet was a lot of headache

  • einops follows license requirements, it should not derive external code - einsum should be either taken from paddle (I see there is no support) or imported from paddlenlp; No derived code should be copied to einops

arogozhnikov avatar Jul 17 '21 23:07 arogozhnikov

@jm12138 @arogozhnikov Hello, It is seemed that if transpose、reshape、stack、tile、unsequeeze supports 0D, this PR can pass CI and paddle will be supported in EinOps. We will support these 5 APIs for 0D firstly, and are there any other questions you met?

zhwesky2010 avatar Sep 15 '22 03:09 zhwesky2010

@jm12138 @arogozhnikov Problem mentioned above has been fix in https://github.com/PaddlePaddle/Paddle/pull/46555. If there are any other prolem and we will fixed as soon as possible.

zhwesky2010 avatar Oct 13 '22 07:10 zhwesky2010