Opened 10 years ago

Closed 8 months ago

#35 closed defect (migrated)

ClpPackedMatrix::transposeTimes() code and specification mismatch

Reported by: kistamas Owned by: somebody
Priority: major Milestone:
Component: component1 Version:
Keywords: Cc:

Description

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

Change History (1)

comment:1 Changed 8 months ago by stefan

  • Resolution set to migrated
  • Status changed from new to closed

This ticket has been migrated to GitHub and will be resolved there: ​https://github.com/coin-or/Clp/issues/35

Note: See TracTickets for help on using tickets.