-
Notifications
You must be signed in to change notification settings - Fork 159
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
would it be posible to change the char to int to allow more keys? #25
Comments
Did you try to redefine LIST_MAX? It controls the scanning |
Hello @garyocampo @brorgustav @Chris--A @PaulStoffregen I do drive a 132 keys keyboard (but using only 121) with this library. AFAIK, I had to modify MAPSIZE 10 directive to MAPSIZE 12 in keypad.h to make possible to do the scanning properly. If I can recall, number of columns is limited to 16, so the map size is by default 16x10 keys. There is a limit, however, in how many keys you can drive, and it is limited to Arduino digital i/o pins. I use arduino MEGA so if I call recall a maximum of a 15x15 matrix could be used. Again, it depend on how many REAL digital I/O pins you have in your design, because for instance using ethernet shield some digital pins (51?) can't be used. And about your suggestion, it would be sure and I beg some real Arduino programmer tackle it!!! But because the library asks for CHAR, arduino accepts the ascii code for the chars, so in fact you have 255 possible keys!!! Take a look to my keyboard matrix:
What I do next is "recover" the measure string with the index of the pressed key. Letting the keyboard to return the string directly instead of a char or the integer would be absolutely cool. But that's another story!!! By the way, what's your keyboard for? I'm so curious!! All the best Jorge |
i'm trying to get a 166key matrix working. and chars aren't good enough. changing to int or string however would allow for a bigger matrix.
i've tried modifying the library myself with no success.
The text was updated successfully, but these errors were encountered: