Closed avik-pal closed 4 months ago
Attention: Patch coverage is 0%
with 134 lines
in your changes are missing coverage. Please review.
Project coverage is 80.82%. Comparing base (
2899de4
) to head (0ab2af7
). Report is 1 commits behind head on main.:exclamation: Current head 0ab2af7 differs from pull request most recent head 65797aa. Consider uploading reports for the commit 65797aa to get more accurate results
Files | Patch % | Lines |
---|---|---|
ext/LuxForwardDiffExt.jl | 0.00% | 78 Missing :warning: |
ext/LuxZygoteExt.jl | 0.00% | 51 Missing :warning: |
src/utils.jl | 0.00% | 5 Missing :warning: |
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
Zygote.gradient
ForwardDiff.gradient
(Zygote by default drops gradients for parameters here)Zygote.jacobian
(by default Zygote gradient and jacobian cannot be nested together)ForwardDiff.jacobian
(Zygote by default drops gradients for parameters here)I will integrate it will DifferentiationInterface later on.
This must go in before #591 because that breaks almost all Zygote over Zygote support.
Simple Benchmarks
New Timings
Old Timings