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

WIP Liqun/linalg #18621

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

WIP Liqun/linalg #18621

wants to merge 3 commits into from

Conversation

liqunfu
Copy link
Contributor

@liqunfu liqunfu commented Nov 29, 2023

Description

There has been requests from ONNX community to add linalg ops. All those operators can be found in torch.linalg. Because of the number of those operators that need to be added, we are still in discussion on whether to use a domain other that ONNX to host it. Before these ops are added to ONNX, this PR is to first add it in ORT so that users can use torch onnx custome op feature to get unblocked.

Motivation and Context

onnx/sigs#169
onnx/onnx#5570

Signed-off-by: Liqun Fu <[email protected]>
Signed-off-by: Liqun Fu <[email protected]>
Signed-off-by: Liqun Fu <[email protected]>
@@ -0,0 +1,318 @@
import unittest
import onnx

Check notice

Code scanning / CodeQL

Module is imported with 'import' and 'import from' Note test

Module 'onnx' is imported with both 'import' and 'import from'.
Module 'onnxruntime.test.onnx' is imported with both 'import' and 'import from'.
Copy link

@github-advanced-security github-advanced-security bot left a comment

Choose a reason for hiding this comment

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

lintrunner found more than 10 potential problems in the proposed changes. Check the Files changed tab for more details.

@XXXDoraemonslayer
Copy link

Hi Any updates on the above branch ?

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.

2 participants