-
Notifications
You must be signed in to change notification settings - Fork 487
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 is_close and all_close tensor operators #1389
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #1389 +/- ##
==========================================
+ Coverage 85.52% 85.54% +0.01%
==========================================
Files 594 595 +1
Lines 68252 68280 +28
==========================================
+ Hits 58374 58409 +35
+ Misses 9878 9871 -7 ☔ 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.
LGTM 😄
@antimora Seems there's a conflict with main? Can you fix it then we can merge |
Pull Request Template
Checklist
run-checks all
script has been executed.Related Issues/PRs
all_close
andis_close
tensor operator #1379all
operator results is incorrect on burn-wgpu when using withlower_equal
#1388 bugChanges
Add is_close and all_close tensor operators.
Testing
New unit testing.