[Clp-tickets] [COIN-OR Linear Programming Solver] #35: ClpPackedMatrix::transposeTimes() code and specification mismatch

[COIN-OR Linear Programming Solver]

#35: ClpPackedMatrix::transposeTimes() code and specification mismatch
------------------------+---------------------------------------------------
 Reporter:  kistamas    |       Owner:  somebody
     Type:  defect      |      Status:  new     
 Priority:  major       |   Milestone:          
Component:  component1  |     Version:          
 Keywords:              |  
------------------------+---------------------------------------------------
 Hi,

 I have CBC 2.4.
 I wanted to use the function
 ClpPackedMatrix::transposeTimes(const ClpSimplex * model, double scalar,
                               const CoinIndexedVector * rowArray,
                               CoinIndexedVector * y,
                               CoinIndexedVector * columnArray) const

 The specification says:
 /* Return <code>x * A + y</code> in <code>z</code>.
         Squashes small elements and knows about ClpSimplex */

 However, in the body of this function at line 575 there is the following
 check:

 assert (!y->getNumElements());

 In other word, y must be empty.
 Then, it seems that this function is able to compute only x A, contrary to
 the specification.
 Or, have I misunderstood something?

 Thank you for providing feedback,

 Tamas Kis

-- 
Ticket URL: <https://projects.coin-or.org/Clp/ticket/35>
COIN-OR Linear Programming Solver <http://projects.coin-or.org/Clp>
A linear programming solver.