[mirjanic]

This pull request adds a fix for the gradient information of the Symmetric kernel by implementing gradients_X_diag. Any feedback is appreciated!

[codecov]

Codecov Report

Merging #1002 (db439d7) into devel (f63ed48) will decrease coverage by 0.00%.
The diff coverage is 33.33%.

Additional details and impacted files

@@            Coverage Diff             @@
##            devel    #1002      +/-   ##
==========================================
- Coverage   54.49%   54.49%   -0.01%     
==========================================
  Files         210      210              
  Lines       21560    21563       +3     
  Branches     3169     3169              
==========================================
+ Hits        11750    11751       +1     
- Misses       9255     9257       +2     
  Partials      555      555              

[lawrennd]

Thank you for this, @mirjanic.

Under CIP-0004 the Symmetric gradients_X_diag improvement still looks like a small mergeable fix, and it is orthogonal to the recent string-identity work in #1126. We would like to land it after rebase + green CI on current devel.

If you can rebase, great; otherwise maintainers can re-implement from your branch and credit you. A short preference note is enough ... but appreciate it's been a long time and if I don't hear, I'll re-implement from your branch!

[lawrennd]

Thank you @mirjanic — under CIP-0004 we tracked this as a kernel quick win.

We are not merging this branch as-is: gradients_X_diag ended up at module scope (not a class method), and the scaled-Kdiag formula omits the cross-term gradients implied by Kdiag. The VibeSafe backlog item and corrected implementation are in #1155; we will close this as superseded when that lands, with credit to you for identifying the missing API.