LadderButton with 2 analog inputs

[geniusled]

Hello, i am trying to modify This code which is essentially based on the provided example of this library.

in my context:

• 2x analog ladder button inputs (same scale/levels) of ADC readings
• of those 2 inputs, one has 5 buttons and the other 4 buttons (+ open/high)
• only ever a single button will be pressed at once
• i wish to use the virtual buttons and have 0-8 as result/serial print

i have viewed the multiple buttons examples provided here but they are for non-ladder. are there any suggestions you can make to have this work on ladder?

ive tried essentially doubling the lines with _B after each variable/ definitions of the initial code but it seems to always detect the second input as the first one (buttons 1-4) regardless of the Analog input used.

are there any known limitations regarding ladderButton and more then one input?

as a side mention, it seems trying to use INPUT_PULLUP with esp32 results in no pullup applied, but from some searches it seems this is an arduino limitation with analogRead() which changes the pin mode every time. i can work around this with a physical pullup

[geniusled]

TIL about Github Discussions.
Sorry and Thank you for the library!

(also maybe +1 for example sketch with multiple ladderbutton)

[bxparks]

#108 is probably a good place to start. I cannot say much more without seeing a circuit diagram... But I also don't have much spare time right now to debug other people's projects.

In general, it's easier to build your system from simple to complex. Verify that your circuit and code works for a single set of resistor ladder switches. Then add the second one, make sure both work. Good luck.

[geniusled]

thank you for the reply, i have used the example provided in #108 and got something working. its probably not optimal but it does work.
i have posted my code with my changes,context and schematic if it can be usefull for others.
i would have prefered using only 1 event handler but was not able to detect which virtual button it came from (analog input 1 or 2) and i just wanted to get something working in the end.
again thanks for the library!

[bxparks]

I looked at your code and your schematic. It looks about right.

If I understand your schematic properly (I assume your switches are pulling to GND), you are using Circuit 1 referenced in my Resistor Ladder doc. I found Circuit 1 to be tricky to get the resistor values configured correctly. I much prefer Circuit 3 because each ladder button is roughly independent of the other ladder buttons. But if your solution work, that's fine.

Regarding the multiple event handlers, you are correct. Once you are in the event handler, you cannot tell whether you received the event from ButtonConfig1 or ButtonConfig2. So using 2 event handlers is the easy solution.

The trick to combining the 2 event handlers into a single handler is to use the id field of the AceButton object. Those are free parameters available to the end-user. Instead of using the AceButton::getPin(), you would use AceButton::getId(). You would assign the ids in a contiguous sequence when creating the array of AceButton objects. For example, you would assign ids 0-5 for the buttons on swcA, then ids 6-10 for the buttons on swcB, for a total of 11 ids. When the event handler is triggered, you would use getId() to do your look up. Hope that makes sense.

[geniusled]

You are correct! Im using gpios to pull low or opendrain as its a 5v system and esp can only do 3.3 logic high (but pins are 5v tolerant). In my case the factory steering buttons are configured like in circuit 1 and my wish was to not intrude or modify them, i made a temporary build outputting the adc values to serial and found the average/middle value and used those adc values. So far its been reliable.

Thank you for suggesting a method to use a single event handler! If i poke back at the thing ill probably make the change as to simplify things. At this time the thing lives behind the radio so its not accessible without more work.