Skip to content

feat(neopixel): Add simple neopixel class wrapping RMT led encoding#374

Merged
finger563 merged 3 commits intomainfrom
feat/neopixel
Feb 18, 2025
Merged

feat(neopixel): Add simple neopixel class wrapping RMT led encoding#374
finger563 merged 3 commits intomainfrom
feat/neopixel

Conversation

@finger563
Copy link
Contributor

@finger563 finger563 commented Feb 18, 2025

Description

  • Add neopixel component wrapping led-strip like functionality specifically tailored for rmt-encoder hardware transmission to LED / strips.
  • Update led_strip to mark parameters as const refs.

Motivation and Context

  • Simplifies the use / interfacing with LEDs which can leverage RMT-encoder for transmission.

How has this been tested?

  • Building and running the new example on a QtPy ESP32s3.

Screenshots (if appropriate, e.g. schematic, board, console logs, lab pictures):

CleanShot 2025-02-18 at 14 45 15

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation Update
  • Hardware (schematic, board, system design) change
  • Software change

Checklist:

  • My change requires a change to the documentation.
  • I have added / updated the documentation related to this change via either README or WIKI

Software

  • I have added tests to cover my changes.
  • I have updated the .github/workflows/build.yml file to add my new test to the automated cloud build github action.
  • All new and existing tests passed.
  • My code follows the code style of this project.

@finger563 finger563 self-assigned this Feb 18, 2025
@finger563 finger563 added enhancement New feature or request led neopixel labels Feb 18, 2025
@github-actions
Copy link

✅Static analysis result - no issues found! ✅

@finger563 finger563 merged commit c31688c into main Feb 18, 2025
70 of 71 checks passed
@finger563 finger563 deleted the feat/neopixel branch February 18, 2025 21:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request led neopixel

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant