fix: minor cleanups

Author: avik-pal

lib/NonlinearSolveBase/src/autodiff.jl

@@ -34,7 +34,8 @@ const ForwardADs = (
 function select_forward_mode_autodiff(
         prob::AbstractNonlinearProblem, ad::AbstractADType; warn_check_mode::Bool = true)
     if warn_check_mode && !(ADTypes.mode(ad) isa ADTypes.ForwardMode) &&
-       !(ADTypes.mode(ad) isa ADTypes.ForwardOrReverseMode)
+       !(ADTypes.mode(ad) isa ADTypes.ForwardOrReverseMode) &&
+       !is_finite_differences_backend(ad)
         @warn "The chosen AD backend $(ad) is not a forward mode AD. Use with caution."
     end
     if incompatible_backend_and_problem(prob, ad)
@@ -60,13 +61,9 @@ end
 function select_reverse_mode_autodiff(
         prob::AbstractNonlinearProblem, ad::AbstractADType; warn_check_mode::Bool = true)
     if warn_check_mode && !(ADTypes.mode(ad) isa ADTypes.ReverseMode) &&
-       !(ADTypes.mode(ad) isa ADTypes.ForwardOrReverseMode)
-        if !is_finite_differences_backend(ad)
-            @warn "The chosen AD backend $(ad) is not a reverse mode AD. Use with caution."
-        else
-            @warn "The chosen AD backend $(ad) is a finite differences backend. This might \
-                   be slow and inaccurate. Use with caution."
-        end
+       !(ADTypes.mode(ad) isa ADTypes.ForwardOrReverseMode) &&
+       !is_finite_differences_backend(ad)
+        @warn "The chosen AD backend $(ad) is not a reverse mode AD. Use with caution."
     end
     if incompatible_backend_and_problem(prob, ad)
         adₙ = select_reverse_mode_autodiff(prob, nothing; warn_check_mode)

lib/SimpleNonlinearSolve/src/SimpleNonlinearSolve.jl

@@ -1,6 +1,7 @@
 module SimpleNonlinearSolve
 
 using Accessors: @reset
+using BracketingNonlinearSolve: BracketingNonlinearSolve
 using CommonSolve: CommonSolve, solve, init, solve!
 using ConcreteStructs: @concrete
 using FastClosures: @closure

src/core/generalized_first_order.jl

@@ -60,7 +60,7 @@ function GeneralizedFirstOrderAlgorithm{concrete_jac, name}(;
         max_shrink_times::Int = typemax(Int)) where {concrete_jac, name}
     return GeneralizedFirstOrderAlgorithm{concrete_jac, name}(
         linesearch, trustregion, descent, max_shrink_times,
-        autodiff, jvp_autodiff, vjp_autodiff)
+        autodiff, vjp_autodiff, jvp_autodiff)
 end
 
 concrete_jac(::GeneralizedFirstOrderAlgorithm{CJ}) where {CJ} = CJ