search

DrChainsaw / NaiveNASflux.jl

Your local Flux surgeon
MIT License
23 stars 0 forks source link

Replace Zygote adjoints with ChainRulesCore #80

Closed DrChainsaw closed 2 years ago

DrChainsaw commented 2 years ago

Depends on https://github.com/DrChainsaw/NaiveNASlib.jl/pull/94

Don't forget to remove/rewrite all adjoints before merging!

codecov[bot] commented 2 years ago

Codecov Report

Merging #80 (d0276e8) into master (f9b0cdb) will increase coverage by 0.53%. The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master      #80      +/-   ##
==========================================
+ Coverage   91.75%   92.28%   +0.53%     
==========================================
  Files           8        8              
  Lines         594      596       +2     
==========================================
+ Hits          545      550       +5     
+ Misses         49       46       -3
Impacted Files Coverage Δ
src/chainrules.jl 100.00% <100.00%> (ø)
src/neuronutility.jl 97.18% <100.00%> (+0.08%) :arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update f9b0cdb...d0276e8. Read the comment docs.