From 6d5f7438719e4f6ec9dcfc6a70bad877f0d48d89 Mon Sep 17 00:00:00 2001
From: Avik Pal <avikpal@mit.edu>
Date: Sat, 26 Oct 2024 13:41:44 -0400
Subject: [PATCH] chore: remove unused imports

---
 lib/NonlinearSolveBase/src/jacobian.jl |  3 ++-
 src/NonlinearSolve.jl                  | 15 ++++++---------
 2 files changed, 8 insertions(+), 10 deletions(-)

diff --git a/lib/NonlinearSolveBase/src/jacobian.jl b/lib/NonlinearSolveBase/src/jacobian.jl
index a189bf042..5725d3687 100644
--- a/lib/NonlinearSolveBase/src/jacobian.jl
+++ b/lib/NonlinearSolveBase/src/jacobian.jl
@@ -29,7 +29,8 @@ Construct a cache for the Jacobian of `f` w.r.t. `u`.
   - `jvp_autodiff`: Automatic Differentiation or Finite Differencing backend for computing
     the Jacobian-vector product.
   - `linsolve`: Linear Solver Algorithm used to determine if we need a concrete jacobian
-    or if possible we can just use a `SciMLJacobianOperators.JacobianOperator` instead.
+    or if possible we can just use a [`SciMLJacobianOperators.JacobianOperator`](@ref)
+    instead.
 """
 function construct_jacobian_cache(
         prob, alg, f::NonlinearFunction, fu, u = prob.u0, p = prob.p; stats,
diff --git a/src/NonlinearSolve.jl b/src/NonlinearSolve.jl
index 78c4bc9d1..b5bfa76bc 100644
--- a/src/NonlinearSolve.jl
+++ b/src/NonlinearSolve.jl
@@ -10,13 +10,12 @@ using ConcreteStructs: @concrete
 using DiffEqBase: DiffEqBase # Needed for `init` / `solve` dispatches
 using FastClosures: @closure
 using LazyArrays: LazyArrays, ApplyArray, cache
-using LinearAlgebra: LinearAlgebra, ColumnNorm, Diagonal, I, LowerTriangular, Symmetric,
+using LinearAlgebra: LinearAlgebra, Diagonal, I, LowerTriangular, Symmetric,
                      UpperTriangular, axpy!, cond, diag, diagind, dot, issuccess, istril,
                      istriu, lu, mul!, norm, pinv, tril!, triu!
 using LineSearch: LineSearch, AbstractLineSearchCache, LineSearchesJL, NoLineSearch,
                   RobustNonMonotoneLineSearch, BackTracking, LiFukushimaLineSearch
-using LinearSolve: LinearSolve, QRFactorization, needs_concrete_A, AbstractFactorization,
-                   DefaultAlgorithmChoice, DefaultLinearSolver
+using LinearSolve: LinearSolve
 using MaybeInplace: @bb
 using NonlinearSolveBase: NonlinearSolveBase,
                           nonlinearsolve_forwarddiff_solve, nonlinearsolve_dual_solution,
@@ -47,18 +46,16 @@ using SymbolicIndexingInterface: SymbolicIndexingInterface, ParameterIndexingPro
 
 # AD Support
 using ADTypes: ADTypes, AbstractADType, AutoFiniteDiff, AutoForwardDiff,
-               AutoPolyesterForwardDiff, AutoZygote, AutoEnzyme, AutoSparse,
-               NoSparsityDetector, KnownJacobianSparsityDetector
-using DifferentiationInterface: DifferentiationInterface, Constant
+               AutoPolyesterForwardDiff, AutoZygote, AutoEnzyme, AutoSparse
+using DifferentiationInterface: DifferentiationInterface
 using FiniteDiff: FiniteDiff
 using ForwardDiff: ForwardDiff, Dual
-using SciMLJacobianOperators: AbstractJacobianOperator, JacobianOperator, VecJacOperator,
+using SciMLJacobianOperators: AbstractJacobianOperator, VecJacOperator,
                               JacVecOperator, StatefulJacobianOperator
 
 ## Sparse AD Support
 using SparseArrays: AbstractSparseMatrix, SparseMatrixCSC
-using SparseMatrixColorings: ConstantColoringAlgorithm, GreedyColoringAlgorithm,
-                             LargestFirst
+using SparseMatrixColorings: SparseMatrixColorings # NOTE: This triggers an extension in NonlinearSolveBase
 
 const DI = DifferentiationInterface