jjhelmus / leastsqbound-scipy

Constrained multivariate least-squares optimizer for scipy
Other
21 stars 8 forks source link

fails if no bounds are set #1

Closed matthewjpeel closed 12 years ago

matthewjpeel commented 12 years ago

Creating the i2e/e2i functions before checking if bounds=None causes an error if no bounds are passed. Need to move these lines below the sanity checks.

I think in line 239 leastsqbound should be leastsq or you end up in a loop.

Perhaps the test script should include the unbound case?

jjhelmus commented 12 years ago

Fixed position of bounds check and changed line 239. Also added a test to run leastsqbound with default bounds. Closing issue.