Skip to content

fix(sum): use pairwise summation for floating-point linalg::Sum

3a5a680
Select commit
Loading
Failed to load commit list.
Draft

fix(sum): use pairwise summation for floating-point linalg::Sum #849

fix(sum): use pairwise summation for floating-point linalg::Sum
3a5a680
Select commit
Loading
Failed to load commit list.
Codecov / codecov/project failed Jun 1, 2026 in 1s

28.70% (-8.20%) compared to 9f0e8fb

View this Pull Request on Codecov

28.70% (-8.20%) compared to 9f0e8fb

Details

Codecov Report

❌ Patch coverage is 96.87500% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 28.70%. Comparing base (9f0e8fb) to head (3a5a680).
⚠️ Report is 192 commits behind head on master.
✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
src/backend/linalg_internal_cpu/pairwise_sum.hpp 96.42% 0 Missing and 1 partial ⚠️

❗ There is a different number of reports uploaded between BASE (9f0e8fb) and HEAD (3a5a680). Click for more details.

HEAD has 1 upload less than BASE
Flag BASE (9f0e8fb) HEAD (3a5a680)
1 0
Additional details and impacted files
@@            Coverage Diff             @@
##           master     #849      +/-   ##
==========================================
- Coverage   36.89%   28.70%   -8.20%     
==========================================
  Files         213      242      +29     
  Lines       31032    35272    +4240     
  Branches    12738    14628    +1890     
==========================================
- Hits        11449    10124    -1325     
- Misses      17719    18161     +442     
- Partials     1864     6987    +5123     
Components Coverage Δ
C++ backend 29.88% <76.19%> (∅)
Python bindings 16.89% <ø> (∅)
Python package 51.07% <ø> (∅)
Files with missing lines Coverage Δ
src/backend/linalg_internal_cpu/Sum_internal.cpp 96.29% <100.00%> (+86.61%) ⬆️
src/backend/linalg_internal_cpu/pairwise_sum.hpp 96.42% <96.42%> (ø)

... and 156 files with indirect coverage changes


Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update d2a0d31...3a5a680. Read the comment docs.

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.