Auto(Sparse)ReverseDiff not supported #287

gdalle · 2024-03-31T14:58:06Z

Problem:

julia> SparseDiffTools.__test_backend_loaded(ADTypes.AutoForwardDiff())
ERROR: AutoForwardDiff{nothing, Nothing}(nothing) requires ForwardDiff.jl to be loaded. Please load it.
Stacktrace:
 [1] error(s::String)
   @ Base ./error.jl:35
 [2] __test_backend_loaded(ad::AutoForwardDiff{nothing, Nothing})
   @ SparseDiffTools ~/.julia/packages/SparseDiffTools/zWIg7/src/highlevel/common.jl:347
 [3] top-level scope
   @ REPL[41]:1

julia> SparseDiffTools.__test_backend_loaded(ADTypes.AutoReverseDiff())
ERROR: AutoReverseDiff(false) requires nothing.jl to be loaded. Please load it.
Stacktrace:
 [1] error(s::String)
   @ Base ./error.jl:35
 [2] __test_backend_loaded(ad::AutoReverseDiff)
   @ SparseDiffTools ~/.julia/packages/SparseDiffTools/zWIg7/src/highlevel/common.jl:347
 [3] top-level scope
   @ REPL[42]:1

Solution: edit the following lines to add the ReverseDiff case

SparseDiffTools.jl/src/highlevel/common.jl

Lines 344 to 355 in d2f35b7

    
           # Check Backend has been loaded 
        
           ## We pay a small compile time cost for this, but it avoids cryptic error messages 
        
           @inline function __test_backend_loaded(ad::AbstractADType) 
        
               error("$(ad) requires $(__backend(ad)).jl to be loaded. Please load it.") 
        
           end 
        
           @inline __backend(ad) = nothing 
        
           @inline __backend(::Union{AutoEnzyme, AutoSparseEnzyme}) = :Enzyme 
        
           @inline __backend(::Union{AutoZygote, AutoSparseZygote}) = :Zygote 
        
           @inline __backend(::Union{AutoForwardDiff, AutoSparseForwardDiff}) = :ForwardDiff 
        
           @inline __backend(::Union{AutoPolyesterForwardDiff, AutoSparsePolyesterForwardDiff}) = :PolyesterForwardDiff 
        
           @inline __backend(::Union{AutoFiniteDiff, AutoSparseFiniteDiff}) = :FiniteDiff

gdalle mentioned this issue Mar 31, 2024

Issues & PRs in other repos JuliaDiff/DifferentiationInterface.jl#99

Closed

gdalle linked a pull request Apr 3, 2024 that will close this issue

Missing __test_backend_loaded dispatch #288

Open

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Auto(Sparse)ReverseDiff not supported #287

Auto(Sparse)ReverseDiff not supported #287

gdalle commented Mar 31, 2024 •

edited

Loading

Auto(Sparse)ReverseDiff not supported #287

Auto(Sparse)ReverseDiff not supported #287

Comments

gdalle commented Mar 31, 2024 • edited Loading

gdalle commented Mar 31, 2024 •

edited

Loading