-
Notifications
You must be signed in to change notification settings - Fork 641
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
First note not played #8
Comments
Thanks for the bug report, I'll take a look into it on the next iteration. I should be back in the MIDI.js code in the next week or so. |
I wonder if anyone reproduced the bug ? Or is it me who misused something ? |
I've updated the app, there was a typo in the Player.js. |
Not sure whether to post here or create a new issue, but this behaviour has been reintroduced? |
@BoopFtang I am seeing this also, take a look here: #165 |
Hi,
It seems that the first note of a loaded file is not played.
When debugging, I found that the first noteOn event of the data array has a length of "0". By length, I mean data[n][1] (see Player.js; Line:226). So queuedTime remains inferior to currentTime, and the event pushing is skipped. I didn't investigate any further to suggest a fix.
The issue can be reproduced also by the first file on "http://mudcu.be/midi-js/"
Regards
The text was updated successfully, but these errors were encountered: