baggepinnen / ControlSystemIdentification.jl

System Identification toolbox, compatible with ControlSystems.jl
https://baggepinnen.github.io/ControlSystemIdentification.jl/dev
MIT License
132 stars 13 forks source link

performance improvements to newpem #92

Closed baggepinnen closed 2 years ago

codecov[bot] commented 2 years ago

Codecov Report

Merging #92 (10b3c76) into master (590f052) will increase coverage by 0.69%. The diff coverage is 84.88%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #92      +/-   ##
==========================================
+ Coverage   83.42%   84.11%   +0.69%     
==========================================
  Files          12       12              
  Lines        2093     2115      +22     
==========================================
+ Hits         1746     1779      +33     
+ Misses        347      336      -11     
Flag Coverage Δ
unittests 84.11% <84.88%> (+0.69%) :arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
src/arx.jl 86.68% <0.00%> (-0.25%) :arrow_down:
src/pem.jl 82.85% <80.00%> (-1.44%) :arrow_down:
src/plotting.jl 92.64% <87.50%> (-0.07%) :arrow_down:
src/ControlSystemIdentification.jl 79.43% <95.45%> (+6.56%) :arrow_up:
src/subspace.jl 84.10% <100.00%> (-0.52%) :arrow_down:
src/subspace2.jl 88.37% <100.00%> (ø)
src/types.jl 75.65% <0.00%> (+5.21%) :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 590f052...10b3c76. Read the comment docs.