search

r-barnes / faster-unmixer

A faster implementation of the sediment inverse/unmixing scheme proposed in Lipp et al (2021).
5 stars 1 forks source link

Add CLARABEL solver #53

Closed AlexLipp closed 9 months ago

AlexLipp commented 9 months ago

Issue #51 raised that ECOS is no-longer the default solver of choice for CVXPY. Whilst we do not use ECOS by default, it would be desirable to include the functionality for the replacement solver for ECOS, which is CLARABEL. This PR adds in CLARABEL as a solver of choice in the NetworkUnmixer class. It also adds it as a tested solver in the run-time benchmark.

r-barnes commented 9 months ago

@AlexLipp - Did you run with CLARABEL and observe reasonable outputs?

AlexLipp commented 9 months ago

@AlexLipp - Did you run with CLARABEL and observe reasonable outputs?

Yes. The unit-tests, when "ecos" is replaced with "clarabel" all pass.