Skip to content

Add Pixel API v2 #731

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

Open
wants to merge 30 commits into
base: migrate-api-v2
Choose a base branch
from
Open

Add Pixel API v2 #731

wants to merge 30 commits into from

Conversation

brentru
Copy link
Member

@brentru brentru commented Apr 16, 2025

This pull request updates the src/pixels component for compatibility with WipperSnapper_Proto API v2.

Minor:

  • Updates the display-handling-related variables and methods to use the "V2" naming convention
    • Note: add_text_to_terminal() is disabled, for now as it is out of scope of this PR and was required to get FunHouse working for DotStar Testing

Testing

Board: Adafruit Feather ESP32-S2
Component: Built-in Status NeoPixel
✅ Init
✅ Remove/Deinit
✅ Write
✅ Write after deinit
✅ Off after deinit (bug)

Board: Adafruit FunHouse ESP32-S2
Component: 5x DotStars
✅ Init
✅ Remove/Deinit
✅ Write 5x DotStars
✅ Write 4x DotStars
✅ Write after deinit
✅ Off after deinit (bug)

@brentru brentru requested a review from tyeth April 16, 2025 18:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant