Skip to content

Commit

Permalink
Add display submodule (#196)
Browse files Browse the repository at this point in the history
* first cut at the display module

first cut at the display module

i hate pep8 line limits so much

switched display to reuse existing axes by default

set a default style template for segments

added segment hierarchy plotter

added pitch contours and midi tickers

a thousand curses upon pep8

factored out ticker objects

added base and height controls to labeled_intervals

implemented piano_roll

fixing up ticker positions

disable cents if 0 in note ticker

fixed baseline positioning for ticker_notes

added source separation viz

fixed scipy version requirement to only be 0.16 for display

fixed axis limits in separation plot

fixed a bug in separation definition

switched out display bounds in separation

added multipitch display

docstring fix in display.multipitch

colin is a jerk for making us blindly conform to pep8

docstring cleanups in display

* moved midi_to_hz and hz_to_midi to utils

starting tests for display submodule

suppressing pep8 on tests for now

fixed travis config for display install

fixing travis config for tests, again

fixing file paths because travis is dumb

adding tests for labeled intervals

added hierarchy display tests

added pitch plot tests

improved test cases for pitch display

tests for multipitch display

multipitch display test outputs

added pianoroll tests

added cent test for ticker midi display

added separation display tests

pep8 is still stupid

added pep8 back to travis

fixed typo in display test

fixed display test cases

nbagg is necessary

disabling pep8 for matplotlib line

fixed coverage of labeled_intervals

pep8 pedantry

pylint display.py

cleaning up limit setting and segment plot code

added pep8ed mpl_ic

fixed a legacy python compatiblity error in display

* updating gitignore for test images

* integrated craffel comments

* allow height sharing in labeled_intervals

* added events()

added tests for events

added test fixture for display.events

added labels to events

added tests for labeled event display

* removed unused variables

* fixed text_kw default behavior

* fixed an edge case in note ticker

* fixed another rounding error in note ticks

* fixed octave numbering in midi ticker

* fixed axis limits on separation display

* Fixed some copypaste docstring issues

* rewrote labeled_intervals using BrokenBarHCollection

* integrating craffel comments on display

* display support empty data

* fixed auto ranging for display.events

* do not display separation legends by default

* hobgoblins.

* exposed alpha parameter in display.separation

* fixed events axis range logic
  • Loading branch information
bmcfee authored and craffel committed Jun 30, 2016
1 parent 13b3a35 commit 5887679
Show file tree
Hide file tree
Showing 29 changed files with 1,613 additions and 4 deletions.
5 changes: 4 additions & 1 deletion .gitignore
Original file line number Diff line number Diff line change
Expand Up @@ -42,4 +42,7 @@ Thumbs.db
.idea/*

# docs
docs/_build/*
docs/_build/*

# matplotlib tsets
tests/result_images/*
6 changes: 3 additions & 3 deletions .travis.yml
Original file line number Diff line number Diff line change
Expand Up @@ -20,17 +20,17 @@ before_install:
- conda config --set always_yes yes --set changeps1 no
- conda update -q conda
- conda info -a
- deps='pip atlas numpy scipy sphinx nose six future pep8'
- deps='pip atlas numpy scipy sphinx nose six future pep8 matplotlib decorator'
- conda create -q -n test-environment "python=$TRAVIS_PYTHON_VERSION" $deps
- source activate test-environment
- pip install python-coveralls
- pip install numpydoc

install:
- pip install -e .
- pip install -e .[display]

before_script:
- pep8 mir_eval tests evaluators
- pep8 mir_eval evaluators tests

script:
- nosetests -v --with-coverage --cover-package=mir_eval
Expand Down
Loading

0 comments on commit 5887679

Please sign in to comment.