isalin / TBbard

Automated FFXIV Bard Performer (Now with MIDI support!)
MIT License
35 stars 10 forks source link

Update MidiParser.java #10

Closed hyskylord closed 6 years ago

hyskylord commented 6 years ago

Fix the double click issue

isalin commented 6 years ago

Hmm, this seems to cause notes to be filtered out in some files.

For an example, in the attached file, line 11 should've parsed to "hG".

fountain.zip

In this file 1 tick is approximately 5 ms, so filtering out events with spaces of 10 ticks will filter out anything longer than 50 ms here. Because there's a release event for the previous note happening 31 ms after "hG" it's filtered out.

I think you're on to something though. I think I originally just chose "1" randomly there to filter out multiple events that happen on the same tick back when the original perform system was a bit laggy.