2
94%
master: 94%

Ran 31 Dec 2016 01:13AM UTC

Files 28

Run time 1s

Badge

Embed ▾

Committed 30 Dec 2016 11:38PM UTC coverage: 77.172% (+0.5%) from 76.643%

Job # 152.2

Build Type

push

travis-ci

Committed by

slivingston

Commit Message

Merge pull request #101 "Refactored matrix inversions (see #85 and #91)"
https://github.com/python-control/python-control/pull/101

Changes are from branch `master` of
https://github.com/mp4096/python-control.git

There was merge conflict in how a for-loop was refactored into
`map` (here) vs. list comprehension (from PR #110).

I compared the two alternatives using %timeit of Jupyter for matrices
that would correspond to LTI systems with 10 state dimensions, 2
inputs, 2 outputs (so, the A matrix has shape (10, 10), B matrix has
shape (10,2), etc.), and with 100 state dimensions, 20 inputs,
20 outputs, all using matrices from numpy.random.random((r,c)).

The difference in timing performance does not appear
significant. However, the case of `map` was slightly faster
(approximately 500 to 900 ns less in duration), so I decided to use
that one to resolve the merge conflict.

Run Details

2265 of 2935 relevant lines covered (77.17%)

0.77 hits per line

python-control / python-control / 152 / 2
94%
master: 94%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 152.2

python-control / python-control / 152 / 2 94% master: 94%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 152.2

python-control / python-control / 152 / 2
94%
master: 94%

README BADGES
x