NLPModelsJuMP.jl
NLPModelsJuMP.jl copied to clipboard
Combine this package with SparseMatricesCOO
trafficstars
We could probably simplify a few things using https://github.com/JuliaSmoothOptimizers/SparseMatricesCOO.jl
- [ ]
SparseMatrixCOOinstead ofCOOhttps://github.com/JuliaSmoothOptimizers/NLPModelsJuMP.jl/blob/c05a30f503d990823407159492fd40b27fc2ee36/src/utils.jl#L28 - [ ] Move
coo_sym_dothttps://github.com/JuliaSmoothOptimizers/NLPModelsJuMP.jl/blob/c05a30f503d990823407159492fd40b27fc2ee36/src/utils.jl#L74 to https://github.com/JuliaSmoothOptimizers/SparseMatricesCOO.jl/blob/main/src/coo_linalg.jl - [ ] Move
coo_sym_add_mul!https://github.com/JuliaSmoothOptimizers/NLPModelsJuMP.jl/blob/c05a30f503d990823407159492fd40b27fc2ee36/src/utils.jl#L74 to https://github.com/JuliaSmoothOptimizers/SparseMatricesCOO.jl/blob/main/src/coo_linalg.jl
It's an excellent idea @tmigot, I didn't find the time to look at SparseMatricesCOO.jl but we should use it as a dependency of NLPModelsJuMP.jl.
I think it should be a dependency of NLPModels to begin with.
Agreed, this is connected to https://github.com/JuliaSmoothOptimizers/NLPModels.jl/issues/389