-
Notifications
You must be signed in to change notification settings - Fork 446
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 onnx mean #2119
Add onnx mean #2119
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #2119 +/- ##
==========================================
+ Coverage 86.21% 86.23% +0.01%
==========================================
Files 690 691 +1
Lines 88620 88720 +100
==========================================
+ Hits 76406 76506 +100
Misses 12214 12214 ☔ View full report in Codecov by Sentry. |
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 for contributing this operator! 🙏
Implementation looks good, just some very minor changes related to formatting before we can merge.
SG! Applied the fixes |
@antimora you might want to update the original issue |
Pull Request Template
Checklist
run-checks all
script has been executed.Related Issues/PRs
#1714
Changes
Add mean ops
Testing
Added test