-
Notifications
You must be signed in to change notification settings - Fork 47
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
Unconstrained Optimization #519
Conversation
Codecov Report
@@ Coverage Diff @@
## develop #519 +/- ##
===========================================
+ Coverage 90.14% 90.22% +0.07%
===========================================
Files 72 72
Lines 4291 4315 +24
===========================================
+ Hits 3868 3893 +25
+ Misses 423 422 -1
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.
I believe this should all work fine, no question. However, I was wondering conceptually whether it may be desirable to make the distinction clearer between startpoint and objective bounds, same as it is done in petab (the difference is not supported yet), e.g. by not using the "ConstraintRemover" but instead allowing to pass both ub and ub_init to the problem class, where the former could then just be inf directly. Having both ub and ub_init will be necessary at some point anyway. What would be your opinion on that? Maybe also @dweindl @jvanhoefer ?
A problem with what I just said might be that in sampling, inf bounds make not so very much sense, so having this "unconstrained optimization" in addition may also work.
Oh that actually makes a lot of sense, implementation would also be easier and less hacky. Wasn't aware this was already an (unsupported) petab feature. Will adapt implementation. |
Yes, would be great to get that into pypesto. Probably relevant to several people.
👍 Besides that, I can't really judge how helpful it will be to have |
So in general I really like @yannikschaelte's idea of having |
I think uniform prior with boundaries should be equivalent to specifying |
Reimplemented using |
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.
Left to do afterwards: Adjust in PEtab import, #522.
Co-authored-by: Yannik Schälte <31767307+yannikschaelte@users.noreply.github.com>
Don't we need to adapt the |
Yep |
Works with all optimizers but
pyswarm
anddlib
. Also some optimizers are terribad ...