-
-
Notifications
You must be signed in to change notification settings - Fork 83
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
Remove lb and ub from kwargs before passing to solver function in lbfgsb #780
Conversation
2bd79f3
to
0c89091
Compare
@Vaibhavdixit02, I Pkg.rm'd Optimization 3.26.0 and re-added this morning (still 3.26.0). I reran the same code and I'm still getting. Is there a new version that includes the fix?
|
Yup you need to update to 3.26.1 |
@Vaibhavdixit02, I think you need to release it. Pkg.add("Optimization") installs 3.26.0 and Pkg.add(Pkg.PackageSpec(;name="Optimization", version="3.26.1")) leads to
|
It was released an hour ago, you registry is probably not updated, do |
@Vaibhavdixit02 thanks for your help! |
@Vaibhavdixit02, I am getting a retcode of failure across many seeds, but when I look at the warning, it says,
I think that should actually be coded as success in STOP_REASON_MAP. |
Yes, can you do a PR? |
#781 I'll merge and release tomorrow morning |
Thanks for handling! Didn't get to yesterday. |
@Vaibhavdixit02, I see you updatedSTOP_REASON_MAP with entry r"Unrecognized stop reason: CONVERGENCE: REL_REDUCTION_OF_F_<=_FACTR*EPSMCH" => ReturnCode.Success. I did ]up and reinstalled Optimization.jl v3.26.1. When I try this:
I get
I can't tell what the issue is.
For the p0 I select in snippet above, the warning is triggered
|
There was no release after this PR, I have triggered it now so you should be able to see it in a few hours |
thanks! |
@Vaibhavdixit02, I gave it a whirl this morning and I don't believe it was triggered, as I got the same result. Will try again tomorrow or later in the day. |
Your registry might be outdated, can you try updating and restart julia https://github.com/SciML/Optimization.jl/releases/tag/v3.26.2 |
@Vaibhavdixit02 something is still off here. I'll try it again tomorrow. I'm not sure of the issue. |
@Vaibhavdixit02, I was able to get the new release
and all my seeds are converging with message
And yet this is still giving me retcode failure when I run:
I am not sure what's the issue as you updated STOP_REASON_MAP. I don't believe there's a typo as I compared entry in STOP_REASON_MAP.
matches the warning
Let me know if you have ideas on what could be the trouble. |
You were absolutely right, sorry for the delay. Created a PR that should fix it (for real!) #782 |
@Vaibhavdixit02, thanks for your help! I'll look for 3.26.3! |
Checklist
contributor guidelines, in particular the SciML Style Guide and
COLPRAC.
Additional context
Add any other context about the problem here.