search

cvxgrp / qcml

A Python parser for generating Python/C/Matlab solver interfaces
Other
42 stars 9 forks source link

Bugfix: don't lose unset dimensions on setter! #33

Closed chinasaur closed 10 years ago

chinasaur commented 10 years ago

At the moment this is cosmetic as the underlying shapes don't lose anything. But definitely a bug!

echu commented 10 years ago

Do you mind fixing the test case? Just so "Travis" is happy (it's kind of creepy giving that build tool a human-ish name).

chinasaur commented 10 years ago

I fixed the test case. Had some trouble finding the right test based on travis / nosetests output; where should I be looking?

echu commented 10 years ago

usually the tests are under the "src/test" folder.

the error message usually tells you which file it is, but it didn't in this one case. i'll have to fix this. :)

On Wed, Sep 18, 2013 at 12:39 PM, Peter H. Li notifications@github.comwrote:

I fixed the test case. Had some trouble finding the right test based on travis / nosetests output; where should I be looking?

— Reply to this email directly or view it on GitHubhttps://github.com/cvxgrp/qcml/pull/33#issuecomment-24693018 .