optimize.curve_fit uses different nomenclature from optimize.leastsq #3172

Closed
alchemyst opened this Issue Dec 27, 2013 · 1 comment

Comments

Projects
None yet
2 participants
@alchemyst
Contributor

alchemyst commented Dec 27, 2013

The curve_fit help uses N for the number of data points and M for the number of parameters, but then calls leastsq which has these reversed. This makes some of the error messages harder to understand.

I suggest simply swapping N and M in the curve_fit help.

@rgommers

This comment has been minimized.

Show comment Hide comment
@rgommers

rgommers Jan 4, 2014

Member

Would make sense that these two functions have consistent docs. Changing the curve_fit docstring should be straightforward.

Member

rgommers commented Jan 4, 2014

Would make sense that these two functions have consistent docs. Changing the curve_fit docstring should be straightforward.

alchemyst added a commit to alchemyst/scipy that referenced this issue Jan 5, 2014

DOC: make curve_fit nomenclature same as leastsq
The curve_fit help used N for the number of data points and M for the number of parameters, but lasts which has these reversed. Fixes issue #3172

@rgommers rgommers closed this Jan 5, 2014

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment