Skip to content

test PT on 23 test problems #280

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

Merged
merged 2 commits into from
Nov 7, 2023

Conversation

yonatanwesen
Copy link
Contributor

I forgot to add PT to 23 test problems. This PR fixes that issue

Copy link

codecov bot commented Nov 6, 2023

Codecov Report

Merging #280 (28c2137) into master (c6c875f) will increase coverage by 44.73%.
Report is 4 commits behind head on master.
The diff coverage is n/a.

@@             Coverage Diff             @@
##           master     #280       +/-   ##
===========================================
+ Coverage   48.45%   93.19%   +44.73%     
===========================================
  Files          19       19               
  Lines        1814     1821        +7     
===========================================
+ Hits          879     1697      +818     
+ Misses        935      124      -811     

see 18 files with indirect coverage changes

📣 Codecov offers a browser extension for seamless coverage viewing on GitHub. Try it in Chrome or Firefox today!

@avik-pal
Copy link
Member

avik-pal commented Nov 7, 2023

Run the formatter

@ChrisRackauckas
Copy link
Member

@yonatanwesen
Copy link
Contributor Author

https://github.com/SciML/NonlinearSolve.jl/actions/runs/6776687678/job/18418713412?pr=280#step:6:598 test failure, is it one of the introduced ones?

No. I don't see any failures in the tests I introduced

@ChrisRackauckas ChrisRackauckas merged commit fb2f19a into SciML:master Nov 7, 2023
# 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.

3 participants