Skip to content
This repository has been archived by the owner on Oct 31, 2024. It is now read-only.

Add Termination Conditions to Broyden #45

Merged
merged 4 commits into from
Feb 19, 2023

Conversation

avik-pal
Copy link
Member

Needs to wait on SciML/DiffEqBase.jl#878

Project.toml Outdated Show resolved Hide resolved
@codecov
Copy link

codecov bot commented Feb 19, 2023

Codecov Report

Merging #45 (5a3db2b) into main (f250799) will decrease coverage by 0.17%.
The diff coverage is 90.00%.

@@            Coverage Diff             @@
##             main      #45      +/-   ##
==========================================
- Coverage   91.37%   91.21%   -0.17%     
==========================================
  Files          15       15              
  Lines         638      649      +11     
==========================================
+ Hits          583      592       +9     
- Misses         55       57       +2     
Impacted Files Coverage Δ
ext/SimpleBatchedNonlinearSolveExt.jl 87.50% <88.88%> (-0.88%) ⬇️
src/broyden.jl 91.17% <90.90%> (-1.69%) ⬇️

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

Project.toml Outdated Show resolved Hide resolved
@ChrisRackauckas
Copy link
Member

Make sure to follow up with some docs in NonlinearSolve, and TerminationCondition should get a doc page of its own.

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

Successfully merging this pull request may close these issues.

2 participants