@@ -15,9 +15,22 @@ class FileMIDI @sequence.tracks << (@track = MIDI::Track.new(@sequence)) @track.events << Tempo.new(Tempo.bpm_to_mpq(options[:tempo])) if options[:tempo] @track.events << MetaEvent.new(META_SEQ_NAME, options[:name]) if options[:name] + + # I'm not sure if this is actually necessary (?) + @track.events << Controller.new(0, CC_VOLUME, 127) + @track.events << ProgramChange.new(0, 1, 0) end def play(note) - @track.events << note + @track.events << NoteOnEvent.new(note.channel, + note.number, + note.velocity, + 0) # this number here should carry an offset representing the + # amount of time since the last message in this stream. it's still + # not clear to me how to handle simultaneous notes, however. + @track.events << NoteOffEvent.new(note.channel, + note.number, + note.velocity, + @sequence.note_to_delta("16th")) # yeah, well, whatever end end lib/midi/file_output/file_midi.rb @@ -21,9 +21,9 @@ describe FileMIDI do end it "adds notes to itself" do 2.times {@midi.play(@create_note[])} - @midi.instance_variable_get("@track").events.size.should == 2 - @midi.instance_variable_get("@track").events.collect(&:class).should == [Note, Note] - # note: I don't understand how this spec passes - I don't think I'm requiring ActiveSupport anywhere + (@midi.instance_variable_get("@track").events.select do |note| + note.class == MIDI::NoteOnEvent + end).size.should == 2 end it "has a MIDI sequence and track" do @midi.instance_variable_get("@sequence").should be_an_instance_of MIDI::Sequence spec/midi_files_spec.rb d5c19c49db20f18d144501a9b2b7ed2ef8a12a4e Giles Bowkett gilesb@gmail.com http://github.com/gilesbowkett/archaeopteryx/commit/347d7cd2fee1f3ced93a0af267f9a9fbe4f5ba72 347d7cd2fee1f3ced93a0af267f9a9fbe4f5ba72 2009-05-05T22:32:58-07:00 2009-05-05T22:32:58-07:00 this is some broken shit on the pathway to awesome shit c5f4ab3e4c90ee79c2c5d9d3a16141d223b9537d Giles Bowkett gilesb@gmail.com