Closed tmigot closed 1 year ago
Merging #66 (6111885) into main (b2444c1) will decrease coverage by
0.93%
. The diff coverage is98.26%
.:exclamation: Current head 6111885 differs from pull request most recent head c4979b6. Consider uploading reports for the commit c4979b6 to get more accurate results
@@ Coverage Diff @@
## main #66 +/- ##
==========================================
- Coverage 99.18% 98.24% -0.94%
==========================================
Files 7 7
Lines 489 685 +196
==========================================
+ Hits 485 673 +188
- Misses 4 12 +8
Impacted Files | Coverage Δ | |
---|---|---|
src/ADNLPModels.jl | 100.00% <ø> (ø) |
|
src/nlp.jl | 98.29% <98.23%> (-1.71%) |
:arrow_down: |
src/nls.jl | 98.65% <98.27%> (-1.35%) |
:arrow_down: |
src/ad.jl | 94.73% <100.00%> (+0.29%) |
:arrow_up: |
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.
@dpo I rebased after the NLS constructors, so it is ready to be reviewed. This PR now includes:
Please let me know, if you want to split it further. After this one, we are back on track with the unit tests.
Sorry, I forgot that you have a bot for this package :open_mouth:
Should I revert the PR ?
Yes, it cuts both ways that bot...
Thanks for reviewing. We need to move forward with this!
Thanks guys, for supporting this long-term project!
Finally, the update with the linear API. Again this is a large PR so here is a recap':
SparseArrays
in the depsADNLPModel
andADNLSModel
store four vectors for the linear constraint functionAx + blin
lin
keyword.jtprod!
).