problem modification fixes

It doesn't look like the problem access/modification was ever tested. The tests still fail:

- The sign of the reduced costs disagree with the tests
- ``addvar!`` ignores the ``rowidx``/``rowcoef`` input to add the variable to existing constraints