-
Notifications
You must be signed in to change notification settings - Fork 3.5k
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
[Hexagon] Softmax slice op initial version #11559
Conversation
@gayatripk1 can you fix the merge conflict? |
@gayatripk1 also please add some detail to the PR description. |
2e29537
to
5b9bd4b
Compare
@gayatripk1 please rebase with main to resolve the conflict. |
Resolve merge conflict in utils.py
5b9bd4b
to
5b0933d
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks!
|
||
if __name__ == "__main__": | ||
|
||
sys.exit(pytest.main(sys.argv)) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
we can do this in a follow up:
- remove the new line
- replace
sys.exit(pytest.main(sys.argv))
with tvm.testing.main()
@@ -0,0 +1,140 @@ | |||
# Licensed to the Apache Software Foundation (ASF) under one |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Follow up PR: move this to under test_hexagon/topi
Resolve merge conflict in utils.py
Resolve merge conflict in utils.py
Resolve merge conflict in utils.py
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.
This patch adds the initial python implementation softmax slice op for hexagon.
cc @mehrdadh