-
Notifications
You must be signed in to change notification settings - Fork 655
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
Added Multiplication block #2110
Conversation
Hi, the build complains about: Task :integration:pmdMain FAILED
Is there any chance to open this file from this github site? I would not be able to reproduce it locally as I am using Windows and the Windows build seems OK, just not the ubuntu one |
689e10c
to
870fd60
Compare
Codecov ReportBase: 72.08% // Head: 71.42% // Decreases project coverage by
Additional details and impacted files@@ Coverage Diff @@
## master #2110 +/- ##
============================================
- Coverage 72.08% 71.42% -0.67%
- Complexity 5126 6289 +1163
============================================
Files 473 625 +152
Lines 21970 27849 +5879
Branches 2351 2999 +648
============================================
+ Hits 15838 19890 +4052
- Misses 4925 6496 +1571
- Partials 1207 1463 +256
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report at Codecov. |
frankfliu thanks for fixing , i tried "Download log archives" but that's just log files, where can i download the test results (in html form) ? |
@patins1 You can find the report.zip file at the bottom of summary page: https://github.com/deepjavalibrary/djl/actions/runs/3371014936 |
Scrolling down is clearly a useful skill :-$ |
* Added Multiplication block * improved comments * Address PMD issues Co-authored-by: Frank Liu <frankfliu2000@gmail.com>
Adds a new block type that enables to use a custom aggregating function as opposed to summing of Linear block