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

Support e-drum #108

Open
leocaseiro opened this issue Sep 11, 2023 · 12 comments
Open

Support e-drum #108

leocaseiro opened this issue Sep 11, 2023 · 12 comments

Comments

@leocaseiro
Copy link
Contributor

This app is very cool, and it would be nice to also support e-drum. What are you thoughts on it?

@samouri
Copy link
Contributor

samouri commented Sep 16, 2023

What do you mean e-drum support? Playing the associated midi sounds or a new mode for learning to play e-drum songs? I think that would be great to add in

@leocaseiro
Copy link
Contributor Author

I mean, as to connect a electronic drum via MIDI, and learn how to play songs, but instead of using a keyboard, to use a drumkit.

I was trying to work on that, and the sightread is almost ready for .mid that have drum tracks in a separated channel, but I wasn't able to see the tracks if the kid has all instruments in the same track.

2 similar comments
@leocaseiro
Copy link
Contributor Author

I mean, as to connect a electronic drum via MIDI, and learn how to play songs, but instead of using a keyboard, to use a drumkit.

I was trying to work on that, and the sightread is almost ready for .mid that have drum tracks in a separated channel, but I wasn't able to see the tracks if the kid has all instruments in the same track.

@leocaseiro
Copy link
Contributor Author

I mean, as to connect a electronic drum via MIDI, and learn how to play songs, but instead of using a keyboard, to use a drumkit.

I was trying to work on that, and the sightread is almost ready for .mid that have drum tracks in a separated channel, but I wasn't able to see the tracks if the kid has all instruments in the same track.

@samouri
Copy link
Contributor

samouri commented Sep 18, 2023

@leocaseiro, let me know if you'd be willing to help add e-drum support! Do you have any examples of MIDI files you can send me?

@leocaseiro
Copy link
Contributor Author

Hi @samouri, I would be very happy to contribute on it.

Sure, I attached 2 different types of MID.
SameTrackDrums.mid.zip

The first one, uses the same track, and that's the one I wasn't able to separate the drums.

The other track, I download a guitar pro file (gp5) from songsterr API (https://www.songsterr.com/a/ra/player/song/10586.plist?apiVersion=2), and converted into MIDI using GuitarPro.
That one is almost read to go on sightread. If you test this mid, you can see the track number 3 for the Drums.
(allthough, we would need to replace the drums soundfount).

But, if we just map the midis to the correct drum pad/cymbal, it would be ready to go to play with a e-drum. We can either also make minor changes to the UI to show as drums.

songsterr--michael-jackson-billie-jean.mid.zip

@samouri
Copy link
Contributor

samouri commented Sep 19, 2023

@leocaseiro, thank you for the contribution and example MIDIs. The first one is a format 0 MIDI which I never added support for, but would be great to do. I've been meaning to bolster the MIDI parsing and add unit tests for a long time, but haven't had the time. I should be able to get to it within the next month in case you'd prefer to focus on the drum visualization.

Do you have a discord username?

@leocaseiro
Copy link
Contributor Author

Oh, I see. So the midi format is different? Awesome. I thought it was the tracks that were kind of different.
Good to know. (for what I quick researched, we just need to separate the tracks?)

I would prefer to work on midi mapping, and drum recognition.
My discord username is the same leocaseiro

@samouri
Copy link
Contributor

samouri commented Sep 24, 2023

@leocaseiro: parsing the different formats should work now. still need to teach the player to automatically infer percussion, but it works when manually mapped.

@leocaseiro
Copy link
Contributor Author

leocaseiro commented Jul 29, 2024

I know I have been away for a while, but I am keen to try support e-drums still. I was able to auto infer the soundfont here #148

@leocaseiro
Copy link
Contributor Author

leocaseiro commented Jul 29, 2024

Next step is to have a single staff for drums. Do you have a suggestion how to go with that?

Update: I found that we talked on discord about that an year ago hehe. Will look into that.

@samouri
Copy link
Contributor

samouri commented Jul 30, 2024

Good to hear from you @leocaseiro! I hope you've been well. I've also taken a break as life seems perennially busy.

I'll review your change in the next day or two 😀

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