Trust Region - Fan's method

[yash2798]

No description provided.

[codecov]

Codecov Report

Merging #178 (05a3e99) into master (c698d16) will increase coverage by 1.03%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #178      +/-   ##
==========================================
+ Coverage   91.88%   92.92%   +1.03%     
==========================================
  Files           7        7              
  Lines         604      636      +32     
==========================================
+ Hits          555      591      +36     
+ Misses         49       45       -4     

Impacted Files	Coverage Δ
src/trustRegion.jl	97.13% <100.00%> (+2.31%)	⬆️

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

[yash2798]

Hi @ChrisRackauckas , line 307 and 309, is it alright if I calculate jacobian through direct ForwardDiff.jacobian calls? If Hei's scheme is used they'll be calculated only once in the cache init function

[yash2798]

nvm i think jvp will work here too.

[yash2798]

@ChrisRackauckas if this seems alright, this can be merged.

[ChrisRackauckas]

Next step, can we get a test that like 4 iterations of in-place and out-of-place match results?

[yash2798]

Next step, can we get a test that like 4 iterations of in-place and out-of-place match results?

Sure I will make a pr for this soon.