Closed dlcole3 closed 2 years ago
Merging #18 (186a513) into main (1c82649) will increase coverage by
0.07%
. The diff coverage is97.50%
.
@@ Coverage Diff @@
## main #18 +/- ##
==========================================
+ Coverage 93.95% 94.03% +0.07%
==========================================
Files 1 1
Lines 331 352 +21
==========================================
+ Hits 311 331 +20
- Misses 20 21 +1
Impacted Files | Coverage Δ | |
---|---|---|
src/DynamicNLPModels.jl | 94.03% <97.50%> (+0.07%) |
:arrow_up: |
Continue to review full report at Codecov.
Legend - Click here to learn more
Δ = absolute <relative> (impact)
,ø = not affected
,? = missing data
Powered by Codecov. Last update 1c82649...186a513. Read the comment docs.
Added support to
LQDynamicData
andLQDynamicModel
to handle non diagonalS
matrix in the objective function.LQDynamicData
now has an attribute forS
, andLQDynamicModel
now incorporatesS
into the construction ofH
for both sparse and condensed forms. Also updatedsparse_lq_test.jl
so that the JuMP formulation includes theS
matrix and added a new test for the addition of theS
matrix. Also changed type variable in source code so that it usesV
in place ofS
(e.g.,LQDynamicData{T, S, M}
is nowLQDynamicData{T, V, M}
.