1
68%
master: 68%

Ran 01 Jun 2019 02:46AM UTC

Files 6

Run time 0s

Badge

Embed ▾

Committed 01 Jun 2019 02:44AM UTC coverage: 68.217% (+0.8%) from 67.444%

Job # 123.1

Build Type

Pull #169

travis-ci

Committed by

web-flow

Commit Message

Fix integrated piano roll division by zero

When supplying a time array to piano roll, if the time array
exceeds the length of the "native" piano roll generated based
on when the last note finishes, the mean of empty arrays is
computed, resulting in warnings and populating the outputted
piano roll with nan. Similarly, if the time array has a greater
sampling rate than the "native" piano roll, more empty arrays
are generated, resulting in nan when numpy.mean is computed.

- Fixed the above issues as long as time array is increasing
  and greater than or equal to 0.
- Added unit test for the fix
- Made casting to integer explicit in instrument.py to stop
  the numpy deprecation warning

Pull Request Pull Request #169: Fix integrated piano roll division by zero

Run Details

616 of 903 relevant lines covered (68.22%)

0.68 hits per line

craffel / pretty-midi / 123 / 1
68%
master: 68%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 123.1

craffel / pretty-midi / 123 / 1 68% master: 68%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 123.1

craffel / pretty-midi / 123 / 1
68%
master: 68%

README BADGES
x