-
Notifications
You must be signed in to change notification settings - Fork 12
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
add linear API in NLS constructor #72
Conversation
af23057
to
664505f
Compare
|
||
ADNLSModel is an Nonlinear Least Squares model using automatic differentiation to | ||
compute the derivatives. | ||
The problem is defined as | ||
|
||
min ½‖F(x)‖² | ||
s.to lcon ≤ c(x) ≤ ucon | ||
s.to lcon ≤ ( Ax ) ≤ ucon | ||
( c(x) ) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe just to make it clear, state that the linear constraints comes first in the list of constraints.
And so, do you still need the keyword linequ
below?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
On second reading, maybe linequ
is to indicate linear residuals. If so, that could be explained more clearly.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
the linear part of the residual is a bit work to be done, so I will had to the todo-list this clarification. Thanks
Excellent, thank you! |
This is the NLS-version of #69
Again this is extracted from the large PR #66 (so I refer to #66 for the unit tests)