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.

Sign up for GitHub

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

Jump to bottom

Add more fused element-wise operators #743

Open

xadupre wants to merge 17 commits into microsoft:main from xadupre:elem

Member

xadupre commented Jun 10, 2024

No description provided.

xadupre added 5 commits

June 7, 2024 17:13


          Add operator AddAdd, MulMul

07140b4


          add_mul

31d8feb


          add submul

cd66c02


          fix compilation

cf1e47f


          fix merge conflicts

4f03053

xadupre marked this pull request as ready for review

June 10, 2024 12:31

xadupre requested a review from a team as a code owner

June 10, 2024 12:31

wschin reviewed

View reviewed changes

operators/cuda/add_mul.h Show resolved Hide resolved

wschin reviewed

View reviewed changes

operators/cuda/add_mul.h Show resolved Hide resolved

xadupre added 2 commits

June 11, 2024 09:28


          fixed merged conflict


          more comments

8af428a

wschin reviewed

View reviewed changes

operators/cuda/add_mul.h Show resolved Hide resolved

wschin reviewed

View reviewed changes

operators/cuda/add_mul.h Show resolved Hide resolved

wschin reviewed

View reviewed changes

operators/cuda/add_mul.h Show resolved Hide resolved

xadupre and others added 4 commits

June 12, 2024 10:44


          Update operators/cuda/add_mul.h

3bf3cc9

Co-authored-by: Wei-Sheng Chin <[email protected]>


          fix merge conflicts

a4a24a5


          Merge branch 'elem' of https://github.com/xadupre/onnxruntime-extensions

76a8857

 into elem


          Merge branch 'main' of https://github.com/microsoft/onnxruntime-exten…

8c9b408

…sions into elem

xadupre self-assigned this

xadupre and others added 2 commits

June 24, 2024 10:37


          Merge branch 'main' of https://github.com/microsoft/onnxruntime-exten…

fa363ef

…sions into elem


          Merge branch 'main' into elem

afd774e

wschin reviewed

View reviewed changes

operators/cuda/add_mul.h Outdated Show resolved Hide resolved

wschin previously approved these changes

View reviewed changes

Contributor

wschin left a comment

Overall LGTM but please still fix the typo.

xadupre added 3 commits

June 27, 2024 09:47


          Merge branch 'elem' of https://github.com/xadupre/onnxruntime-extensions

16c787a

 into elem


          Merge branch 'main' of https://github.com/microsoft/onnxruntime-exten…

0a2cac8

…sions into elem


          fix documentation

6710e81

xadupre dismissed wschin’s stale review via

6710e81

June 27, 2024 07:51

wschin reviewed

View reviewed changes

operators/cuda/add_mul.h Outdated Show resolved Hide resolved

wschin reviewed

View reviewed changes

Contributor

wschin left a comment

Please make sure the math documentation covers all expected behaviors of the op (not only in this PR). They are the golden standard needed for code quality. Once you fix them, please ping me again. Thanks.


          Merge branch 'main' into elem

5c928b9

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet