• Home
  • Features
  • Pricing
  • Docs
  • Announcements
  • Sign In

f-dangel / curvlinops / 23122505326
91%

Build:
DEFAULT BRANCH: main
Ran 16 Mar 2026 12:18AM UTC
Jobs 1
Files 50
Run time 1min
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst

16 Mar 2026 12:07AM UTC coverage: 91.447% (-0.004%) from 91.451%
23122505326

push

github

web-flow
[ADD] Weight tying support for (E)KFAC make_fx backend (#264)

* [ADD] Weight tying support for KFAC make_fx backend

Support parameters used in multiple affine operations (weight tying)
in the FX-based KFAC backend. Per-usage inputs/gradients are formatted
individually, then concatenated along the shared dimension before
computing covariances, so the normalization is correct.

Also replace `append_ones_for_bias` with `bias_pad: int | None` in
`input_to_weight_sharing_format` to support weight-tied layers where
some usages have bias and others don't.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

* [DOC] Add PR link to changelog entry

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

* [ADD] Weight tying support for EKFAC make_fx backend

Restructure eigenvalue correction in MakeFxEKFACComputer to collect
per-usage formatted tensors and concatenate along the shared dimension
before computing corrections, matching the KFAC backend approach.

Extract SplitConcatModel to test/utils.py for reuse. Add EKFAC
weight-tying test and parametrize both tests by loss reduction.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

* [FIX] Address Copilot review: bias validation and formatting

- Filter NOT_A_PARAM from bias conflict check (could cause false positives)
- Add check that a tracked bias is not reused across different weights
- Extract _verify_weight_bias_compatibility to reduce complexity
- Fix line-too-long formatting in ekfac.py

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

* [FIX] Remove overly restrictive bias-reuse check

A bias appearing with multiple weights is valid when weights and biases
are treated separately (each becomes its own parameter group).

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

* [TST] Parametrize weight tying tests by bias and separate_weight_and_bias

Add bias parameter to SplitConcatModel and parametrize KFAC/EK... (continued)

53 of 55 new or added lines in 4 files covered. (96.36%)

2459 of 2689 relevant lines covered (91.45%)

0.91 hits per line

New Missed Lines in Diff

Lines Coverage ∆ File
1
98.44
-1.56% curvlinops/computers/ekfac_make_fx.py
1
98.68
-1.32% curvlinops/computers/kfac_make_fx.py
Jobs
ID Job ID Ran Files Coverage
1 23122505326.1 16 Mar 2026 12:18AM UTC 50
91.45
GitHub Action Run
Source Files on build 23122505326
  • Tree
  • List 50
  • Changed 5
  • Source Changed 0
  • Coverage Changed 5
Coverage ∆ File Lines Relevant Covered Missed Hits/Line
  • Back to Repo
  • 192b6420 on github
  • Prev Build on main (#23111723348)
  • Next Build on main (#23174555294)
  • Delete
STATUS · Troubleshooting · Open an Issue · Sales · Support · CAREERS · ENTERPRISE · START FREE · SCHEDULE DEMO
ANNOUNCEMENTS · TWITTER · TOS & SLA · Supported CI Services · What's a CI service? · Automated Testing

© 2026 Coveralls, Inc