Skip to content
This repository has been archived by the owner on Feb 9, 2022. It is now read-only.

Support for Pico RGB Keypad Base #9

Merged
merged 5 commits into from
Nov 5, 2021
Merged

Conversation

matemaciek
Copy link
Contributor

@matemaciek matemaciek commented Jul 13, 2021

I have a project for Keybow 2040 and I needed it to run on Pico RGB Keypad Base. So I added support for it to this library.

The only change to interface is constructor parameter - instead of i2c there is a class representing hardware - either keybow_hardware.pim56x.PIM56X for Keybow 2040 or keybow_hardware.pim551.PIM551 for Pico RGB Keypad Base.

TODO:

  • Provide implementation for Pico RGB Keypad Base
  • Update keybow2040.py
  • Update examples
  • Update README.md

@Gadgetoid
Copy link
Member

I just spent far too long wondering why my RGB Keypad wasn't lighting up, only to discover it was an intermittent connection or loose LED on my.. somewhat neglected... prototype. 😆

This seems to work great for me on both my Keybow 2040 and RGB Keypad. Thank you!

The one thing I would change is to use the product name instead of the product code, ie: PIM56X should be Keybow2040 and PIM551 should be RGBKeypadBase.

Of course I realize this would conflict somewhat with the Keybow2040 from keybow2040.py - perhaps we need a more generic name for the project to reflect this. Is PMK a little on the nose?

@Gadgetoid Gadgetoid merged commit 7770d33 into pimoroni:master Nov 5, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants