Skip to content
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

Midi files with multi channels doesn't work using MIDI.Player.Loadfile #256

Open
KomeijiSatori opened this issue Jun 20, 2020 · 3 comments

Comments

@KomeijiSatori
Copy link

KomeijiSatori commented Jun 20, 2020

I have a midi file with three channels(both acoustic_grand_piano), when calling MIDI.Player.Loadfile, it only plays one track.
Another problem is that the player dropped the very first note of the midi file, and the note is at time 0:00. After I move the first note 2 seconds later then it sounds good.
And also I checkout to a branch called "abcjs", and I change the code to the newest commit, the two problems are resolved. Since there're too many commits(29commits ahead of HEAD), so I can't figure out which important part has been changed. But that commit has a problem with pause function, which will play one or two unexpected notes after 3 or 4 seconds after resume.

What's more is another project Color Piano (https://galactic.ink/piano/), which is awesome and it doesn't have any of issues described above. But it seems not open source. So @mudcube , would you mind creating a repo of that project? Much thanks!

@KomeijiSatori KomeijiSatori changed the title Midi files with multi tracks doesn't work using MIDI.Player.Loadfile Midi files with multi channels doesn't work using MIDI.Player.Loadfile Jun 20, 2020
@MX-Qulin
Copy link

yes. same problem.

@KomeijiSatori
Copy link
Author

KomeijiSatori commented Aug 15, 2020

yes. same problem.

@MX-Qulin Hi, friend, I solved this problem by replacing the whole js/midi folder to single midi file used by https://galactic.ink/piano/ , and the problem resolved, cheers!

My github:
https://github.com/KomeijiSatori/3d-piano-player

@MX-Qulin
Copy link

well, as i use the brach abcjs if this repo, it can't work...

i'll try your solution. Anyway, thank you:)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants