-
Notifications
You must be signed in to change notification settings - Fork 35
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 getter for NLPModelMeta
and NLSMeta
#371
Add getter for NLPModelMeta
and NLSMeta
#371
Conversation
Codecov Report
@@ Coverage Diff @@
## main #371 +/- ##
=======================================
Coverage 99.65% 99.66%
=======================================
Files 12 13 +1
Lines 585 589 +4
=======================================
+ Hits 583 587 +4
Misses 2 2
Continue to review full report at Codecov.
|
|
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.
Great! Thanks!
Hope it was ok to merge! |
Yes, it was ok to merge. The |
It appears as if 1. The names have changed to `x0, y0` 2. The getters have been introduced JuliaSmoothOptimizers/NLPModels.jl#371
It appears as if 1. The names have changed to `x0, y0` 2. The getters have been introduced JuliaSmoothOptimizers/NLPModels.jl#371
This way calling
get_minimize(meta)
would no longer allocate the 2nd call, whilemeta.minimize
would always.For each attribute
foo
in the meta, there is aget_foo
function.Note that there is a priority for NLS over NLP for copies in NLPMeta and NLSMeta, e.g.
get_x0(nls)
will look fornls.nls_meta.x0
.