papilo icon indicating copy to clipboard operation
papilo copied to clipboard

Missing implementation of a SparseStorage constructor overload

Open hlinsen opened this issue 3 months ago • 1 comments

Hi,

The implementation of this constructor is missing https://github.com/scipopt/papilo/blob/main/src/papilo/core/SparseStorage.hpp#L79-L82. This is relevant when the user already has the CSR matrix in memory and wants to store it directly into the sparse object.

hlinsen avatar Sep 03 '25 23:09 hlinsen

The declaration is already in SparseStorage.hpp referred by the link in https://github.com/scipopt/papilo/issues/72#issue-3381623711. So the implementation for C-array input is missing. Since it is not used inside PaPILO, a concrete demonstration of its usecase would still be helpful because this requires a test and more might be needed to be useful.

DominikKamp avatar Sep 05 '25 18:09 DominikKamp

As there have been no further updates or questions, I’m closing this issue. Please reopen it if additional concerns arise.

alexhoen avatar Nov 17 '25 18:11 alexhoen