Skip to content

Add Zygote v0.7 compat - #174

Merged
lkdvos merged 1 commit into
Jutho:masterfrom
leburgel:lb/zygote07
Oct 1, 2026
Merged

lkdvos merged 1 commit into
Jutho:masterfrom
leburgel:lb/zygote07

Conversation

@leburgel

Copy link
Copy Markdown
Contributor

Adds some missing unthunks needed to use Zygote v0.7. Some vectors still arrived thunked, giving method errors like

eigsolve: MethodError: Cannot `convert` an object of type ChainRulesCore.Thunk{ComposedFunction{ChainRulesCore.ProjectTo{AbstractArray, …}, ChainRules.var"#rrule##107#rrule##108"{…}}} to an object of type Vector{Float64}
svdsolve: MethodError: Cannot `convert` an object of type ChainRulesCore.Thunk{ComposedFunction{ChainRulesCore.ProjectTo{AbstractArray, …}, ChainRules.var"#rrule##107#rrule##108"{…}}} to an object of type Vector{Float64}

@codecov

codecov Bot commented Sep 30, 2026 •

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 86.65%. Comparing base (4808454) to head (7d04e22).
⚠️ Report is 1 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #174      +/-   ##
==========================================
- Coverage   88.47%   86.65%   -1.83%     
==========================================
  Files          36       36              
  Lines        3965     3918      -47     
==========================================
- Hits         3508     3395     -113     
- Misses        457      523      +66     

☔ 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.

Comment thread test/ad/eigsolve.jl
@lkdvos
lkdvos merged commit 9253654 into Jutho:master Oct 1, 2026
12 of 15 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants