jrnelson90 / tinderboxpedal

Bluetooth "Universal Remote" Footpedal for Digital Guitar Amps, on ESP32 or Pi Zero W
Apache License 2.0
71 stars 11 forks source link

Minor possible issue #2

Open Baza-1969 opened 4 years ago

Baza-1969 commented 4 years ago

Just a quick one Justin - This is great work!. Looks like your electrical schematic has the buttons going sequentially from bottom to top 1-4? Button 1 and 2 look ok, but buttons 3 & 4 seem to be going to the wrong pins according to you GPIO assigment on Tinderbox.py (assuming the schematic buttons are drawn from 1 to 4 sequentially upwards) Button 3 goes to 19 / button 4 goes to 26 This could cause confusion, with button 3 operating preset 4 and visa versa - easily fixed with labelling the schematic buttons?

jrnelson90 commented 4 years ago

I agree, I did that schematic very quickly last night so I could go to bed. I'll be working on a much better one. I just need to reorient myself with Kicad or Eagle, it's been a while.