Skip to content

add TerminationCondition support to NewtonRaphson #162

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Conversation

oscardssmith
Copy link
Member

(and eventually all the other solvers). @ChrisRackauckas can you take a look at this to make sure I'm not doing dumb stuff here?

@codecov
Copy link

codecov bot commented Mar 19, 2023

Codecov Report

Merging #162 (5734827) into master (88d92cb) will decrease coverage by 33.62%.
The diff coverage is 31.70%.

❗ Current head 5734827 differs from pull request most recent head d0dcc4c. Consider uploading reports for the commit d0dcc4c to get more accurate results

@@             Coverage Diff             @@
##           master     #162       +/-   ##
===========================================
- Coverage   93.52%   59.90%   -33.62%     
===========================================
  Files           7        7               
  Lines         649      656        +7     
===========================================
- Hits          607      393      -214     
- Misses         42      263      +221     
Impacted Files Coverage Δ
src/jacobian.jl 60.25% <ø> (-26.93%) ⬇️
src/utils.jl 30.00% <0.00%> (-44.29%) ⬇️
src/trustRegion.jl 46.09% <11.70%> (-51.46%) ⬇️
src/NonlinearSolve.jl 100.00% <100.00%> (ø)
src/raphson.jl 81.70% <100.00%> (-15.56%) ⬇️

... and 2 files with indirect coverage changes

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

@avik-pal
Copy link
Member

@oscardssmith the upstream PR SciML/DiffEqBase.jl#903 has been merged

@oscardssmith oscardssmith force-pushed the add-termination-condition-control-to-newtonraphson branch from b2ed52b to 444620b Compare April 25, 2023 13:35
@oscardssmith
Copy link
Member Author

PR updated.

@ChrisRackauckas
Copy link
Member

Tests fail

@oscardssmith oscardssmith force-pushed the add-termination-condition-control-to-newtonraphson branch 2 times, most recently from 5734827 to a379410 Compare June 7, 2023 19:16
@oscardssmith oscardssmith force-pushed the add-termination-condition-control-to-newtonraphson branch from a379410 to d0dcc4c Compare June 7, 2023 19:17
@ChrisRackauckas
Copy link
Member

Supersceded by #208

avik-pal pushed a commit that referenced this pull request Nov 1, 2024
…_msg

tweak warning message for when tspan isn't an enclosing interval
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants