Parameterizing `Solver` as `Solver{T}` about madnlp.jl HOT 4 CLOSED

That's a good idea. I think it could also help on the GPU. Also: are you mentioning the linear solvers or the interior point solvers?

I would also suggest to use a different naming between the interior point solver (IPMSolver{KKT}?) and the linear solvers (LinearSolvers{T}?).

from madnlp.jl.

@frapac yes, I think we need some consistent naming convention. Maybe we can do the following.
Solver <: AbstractInteriorPointSolver -> InteriorPointSolver{T} <: AbstractInteriorPointSolver{T}
MadNLPMa27.Solver <: AbstractLinearSolver -> Ma27Solver{T} <: AbstractLinearSolver{T}

Also, now that we aiming to make everything run on GPU, maybe we need to add one more signature and make something like InteriorPointSovler{T,V{T}}, so that we can support both InteriorPointSovler{Float64,Vector{Float64}} and InteriorPonitSolver{Float64,CuVector{Float64}}. How does that sound?

from madnlp.jl.

I think what I wrote might be a bit outdated, but you know what I'm saying 🤣

from madnlp.jl.

Closing this issue as it is deprecated.

from madnlp.jl.