Skip to content

fix(tf): validate flattened multi-device op widths - #5827

Draft
njzjz-bot wants to merge 3 commits into
deepmodeling:masterfrom
njzjz-bot:fix/tf-multidevice-shapes-5623
Draft

fix(tf): validate flattened multi-device op widths#5827
njzjz-bot wants to merge 3 commits into
deepmodeling:masterfrom
njzjz-bot:fix/tf-multidevice-shapes-5623

test(tf): cover multi-device width guards

e736a90
Select commit
Loading
Failed to load commit list.
Codecov / codecov/project succeeded Jul 27, 2026 in 0s

78.62% (+0.04%) compared to 1c5ff68

View this Pull Request on Codecov

78.62% (+0.04%) compared to 1c5ff68

Details

Codecov Report

❌ Patch coverage is 32.35294% with 23 lines in your changes missing coverage. Please review.
✅ Project coverage is 78.62%. Comparing base (1c5ff68) to head (e736a90).
⚠️ Report is 44 commits behind head on master.

Files with missing lines Patch % Lines
source/op/tf/custom_op.h 31.25% 7 Missing and 4 partials ⚠️
source/op/tf/prod_force_grad_multi_device.cc 33.33% 4 Missing ⚠️
source/op/tf/prod_force_multi_device.cc 33.33% 4 Missing ⚠️
source/op/tf/prod_virial_grad_multi_device.cc 33.33% 4 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master    #5827      +/-   ##
==========================================
+ Coverage   78.57%   78.62%   +0.04%     
==========================================
  Files        1049     1054       +5     
  Lines      120650   121779    +1129     
  Branches     4348     4409      +61     
==========================================
+ Hits        94806    95746     +940     
- Misses      24285    24462     +177     
- Partials     1559     1571      +12     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.