Skip to content
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

Closed
prbzrg opened this issue Nov 1, 2023 · 9 comments

Comments

@prbzrg
Copy link
Member

prbzrg commented Nov 1, 2023

┌ SimpleNonlinearSolve [727e6d20-b764-4bd8-a329-72de5adea6c7]
│  ┌ Warning: NLSolveTerminationCondition has been deprecated in favor of the new dispatch based termination conditions. Please use the new termination conditions API!
│  │   caller = SimpleNonlinearSolve.Broyden() at broyden.jl:20
│  └ @ SimpleNonlinearSolve ~/.julia/packages/SimpleNonlinearSolve/cW4pm/src/broyden.jl:20
│  ┌ Warning: NLSolveTerminationCondition has been deprecated in favor of the new dispatch based termination conditions. Please use the new termination conditions API!
│  │   caller = SimpleNonlinearSolve.SimpleDFSane() at dfsane.jl:67
│  └ @ SimpleNonlinearSolve ~/.julia/packages/SimpleNonlinearSolve/cW4pm/src/dfsane.jl:67

https://github.com/SciML/NonlinearSolve.jl/actions/runs/6716303943/job/18252333439#step:6:744

@ChrisRackauckas
Copy link
Member

I think @avik-pal is handling all of these?

@avik-pal
Copy link
Member

avik-pal commented Nov 1, 2023

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)

@ChrisRackauckas
Copy link
Member

(But it is quite low on my priority list)

Are you sure? 😅 😅 😅 😅 😅 😅 😅

@avik-pal
Copy link
Member

avik-pal commented Nov 3, 2023

It works... Also I don't want a breaking release for NonlinearSolve rn

@ChrisRackauckas
Copy link
Member

We shouldn't default in ways that are throwing warnings though.

@avik-pal
Copy link
Member

avik-pal commented Nov 3, 2023

Only on CI because depwarn is on

@bpfel
Copy link

bpfel commented Nov 6, 2023

Is it possible that 0.1.24 is a breaking change?

We have a few packages relying on DifferentialEquations and now I cannot precompile julia unless I pin SimpleNonlinearSolve to 0.1.23.

Not sure if this is the right place to post this, but the change, as this issue were very recent.

@ChrisRackauckas
Copy link
Member

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.

@ChrisRackauckas
Copy link
Member

Deprecations completed by the v1.0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants