Skip to content

Teacher-forced (parallel) TB loss recompute for recurrent estimators - #526

Open
josephdviviano wants to merge 1 commit into
feat/kv-cachefrom
feat/kv-cache-teacher-forced-loss
Open

Teacher-forced (parallel) TB loss recompute for recurrent estimators#526
josephdviviano wants to merge 1 commit into
feat/kv-cachefrom
feat/kv-cache-teacher-forced-loss

Teacher-forced (parallel) TB loss recompute for recurrent estimators

25c24c2
Select commit
Loading
Failed to load commit list.
Codecov / codecov/project succeeded Jul 11, 2026 in 0s

74.54% (+0.04%) compared to d9cfd34

View this Pull Request on Codecov

74.54% (+0.04%) compared to d9cfd34

Details

Codecov Report

❌ Patch coverage is 91.42857% with 3 lines in your changes missing coverage. Please review.
✅ Project coverage is 74.54%. Comparing base (d9cfd34) to head (25c24c2).

Files with missing lines Patch % Lines
src/gfn/utils/prob_calculations.py 91.17% 2 Missing and 1 partial ⚠️
Additional details and impacted files
@@                Coverage Diff                @@
##           feat/kv-cache     #526      +/-   ##
=================================================
+ Coverage          74.49%   74.54%   +0.04%     
=================================================
  Files                 59       59              
  Lines              10692    10720      +28     
  Branches            1558     1561       +3     
=================================================
+ Hits                7965     7991      +26     
- Misses              2222     2223       +1     
- Partials             505      506       +1     
Files with missing lines Coverage Δ
src/gfn/estimators.py 86.14% <100.00%> (+0.02%) ⬆️
src/gfn/utils/prob_calculations.py 83.15% <91.17%> (+1.56%) ⬆️
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.