djfroofy/beatlounge — Issue #1: Make NotePlayer and ChordPlayer simple wrappers or factories for SchedulePlayer
Author: djfroofy
Created: 2012-03-01 04:01:30+00:00
State: open

NotePlayer and ChordPlayer classes have some silly interfaces. Why can notes be a generator, a list or a callable, but velocity has to be callable. The "stop" argument can be an integer or a callable ... not a generator? ... ugh

So yeah ... bad.

SchedulePlayer was written after more thought and is much more consistently designed. The current interface for NotePlayer and ChordPlayer should be obsoleted and these should be replaced with either classes wrapping or factories for a SchedulePlayer.