RyoKosaka / HelloDrum-arduino-Library

This is a library for making E-Drum with arduino.
https://open-e-drums.com/
MIT License
244 stars 61 forks source link

7.7 version Trouble with MUX #59

Open ojungleboy opened 7 months ago

ojungleboy commented 7 months ago

I am trying to use MUX with ESP32, The baud rate is matching to the Hairless serial but it seems that the MIDI messages are coming with garbage image

Both 4051 and 4067 MUXs are presenting the same issue.

Does anyone know how to solve it?

Thank you

Originally posted by @ojungleboy in https://github.com/RyoKosaka/HelloDrum-arduino-Library/issues/51#issuecomment-1962800891

ojungleboy commented 7 months ago

Looks like its a version issue. I removed the 7.7 version and rolled back to the 7.3 version.

The problem is gone on the 7.3 version. Once I reinstalled the 7.7 version. The problem is back

ojungleboy commented 7 months ago

If someone could help troubleshoot this would be nice. Thank you so much

steeltape commented 3 weeks ago

I have the same problem.. did you figured out how to fix?

ojungleboy commented 3 weeks ago

I kinda did solve it. There is something bugged in the examples related to MUX code. I guess it it's regarding the velocity curve array.

The way I did was to get a simple piezo code and gradually adding the second piezo code, and finally addind the other piezos. Finally I added the mux state and included one by one at the MUX pins and it worked. I am not on my pc right now, but have a shot on this approach and let us know

On Fri, Sep 6, 2024, 14:43 steeltape @.***> wrote:

I have the same problem.. did you figured out how to fix?

— Reply to this email directly, view it on GitHub https://github.com/RyoKosaka/HelloDrum-arduino-Library/issues/59#issuecomment-2334541099, or unsubscribe https://github.com/notifications/unsubscribe-auth/AXUPICSV2REWDYLCTMPWNWTZVHSUFAVCNFSM6AAAAABD2MYUG2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGMZUGU2DCMBZHE . You are receiving this because you authored the thread.Message ID: @.***>

steeltape commented 3 weeks ago

I kinda did solve it. There is something bugged in the examples related to MUX code. I guess it it's regarding the velocity curve array. The way I did was to get a simple piezo code and gradually adding the second piezo code, and finally addind the other piezos. Finally I added the mux state and included one by one at the MUX pins and it worked. I am not on my pc right now, but have a shot on this approach and let us know - Pedro Zinn Hensel Nunes On Fri, Sep 6, 2024, 14:43 steeltape @.> wrote: I have the same problem.. did you figured out how to fix? — Reply to this email directly, view it on GitHub <#59 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AXUPICSV2REWDYLCTMPWNWTZVHSUFAVCNFSM6AAAAABD2MYUG2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGMZUGU2DCMBZHE . You are receiving this because you authored the thread.Message ID: @.>

if you could share your code is much appreciated

ojungleboy commented 3 weeks ago

Dear friend, I have no access to the code right now. On the pad declaration try to check the order of parameters of the pad

On Fri, Sep 6, 2024, 16:17 steeltape @.***> wrote:

I kinda did solve it. There is something bugged in the examples related to MUX code. I guess it it's regarding the velocity curve array. The way I did was to get a simple piezo code and gradually adding the second piezo code, and finally addind the other piezos. Finally I added the mux state and included one by one at the MUX pins and it worked. I am not on my pc right now, but have a shot on this approach and let us know - Pedro Zinn Hensel Nunes … <#m-3632869251951993278> On Fri, Sep 6, 2024, 14:43 steeltape @.> wrote: I have the same problem.. did you figured out how to fix? — Reply to this email directly, view it on GitHub <#59 (comment) https://github.com/RyoKosaka/HelloDrum-arduino-Library/issues/59#issuecomment-2334541099>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AXUPICSV2REWDYLCTMPWNWTZVHSUFAVCNFSM6AAAAABD2MYUG2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGMZUGU2DCMBZHE https://github.com/notifications/unsubscribe-auth/AXUPICSV2REWDYLCTMPWNWTZVHSUFAVCNFSM6AAAAABD2MYUG2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGMZUGU2DCMBZHE . You are receiving this because you authored the thread.Message ID: @.>

if you could share your code is much appreciated

— Reply to this email directly, view it on GitHub https://github.com/RyoKosaka/HelloDrum-arduino-Library/issues/59#issuecomment-2334672686, or unsubscribe https://github.com/notifications/unsubscribe-auth/AXUPICVMFA6LJV2MIDQZ2NDZVH5WBAVCNFSM6AAAAABD2MYUG2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGMZUGY3TENRYGY . You are receiving this because you authored the thread.Message ID: @.***>