-
Notifications
You must be signed in to change notification settings - Fork 173
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 op ir #270
base: main
Are you sure you want to change the base?
Add op ir #270
Conversation
[frontend] refine implementation of `op_def`
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.
Failed Tests (13):
BUDDY :: Python/test_addmm.py
BUDDY :: Python/test_amax.py
BUDDY :: Python/test_arith_sub.py
BUDDY :: Python/test_convert_element_type.py
BUDDY :: Python/test_embedding.py
BUDDY :: Python/test_exp.py
BUDDY :: Python/test_expand.py
BUDDY :: Python/test_permute.py
BUDDY :: Python/test_reshape.py
BUDDY :: Python/test_select.py
BUDDY :: Python/test_sum.py
BUDDY :: Python/test_tanh.py
BUDDY :: Python/test_var_mean.py
I encountered above failed when I run the ninja check-buddy
. Please have a check.
Thank you for your comment. This because tosa's op set have not adapted for current graph IR. We can add tosa support later in this PR. |
…an run on new graph ir frontend now.
[frontend] Add tosa support for graph ir
Failed to build LLaMA example:
|
@zhanghb97 I have fix this error. Please review it again. Thank you ! |
|
@zhanghb97 Sorry about that. I have fix it and run ninja check-buddy on my machine. I found it will output this. Unresolved Tests (1): Testing Time: 0.47s Total Discovered Tests: 88 |
@weilinquan No worries. Fixed the bert test: fc2948a |
@zhanghb97 Thank you for your comment ! I have add it and now ninja check-buddy is ok on my machine. |
@weilinquan @xTayEx Thank you! Now, it works well. I am still reviewing the details. |
You can also run llama by BuddyLlama.