-
-
Notifications
You must be signed in to change notification settings - Fork 23
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
Warning: NLSolveTerminationCondition has been deprecated in favor of the new dispatch based termination conditions. #92
Comments
I think @avik-pal is handling all of these? |
The easiest way is to do a breaking change of SimpleNonlinearSolve for this (and that would mean another breaking change for NonlinearSolve). (But it is quite low on my priority list) |
Are you sure? 😅 😅 😅 😅 😅 😅 😅 |
It works... Also I don't want a breaking release for NonlinearSolve rn |
We shouldn't default in ways that are throwing warnings though. |
Only on CI because depwarn is on |
Is it possible that 0.1.24 is a breaking change? We have a few packages relying on Not sure if this is the right place to post this, but the change, as this issue were very recent. |
It's not a breaking change. It just required v1.9 because of changes in SciMLBase that required v1.9, so it was yanked and the new v0.1.25 fixes the lower bound. |
Deprecations completed by the v1.0 |
https://github.com/SciML/NonlinearSolve.jl/actions/runs/6716303943/job/18252333439#step:6:744
The text was updated successfully, but these errors were encountered: