Repository navigation
Conversation
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
LED count, max brightness, color order and chipset are now Home Assistant entities applied at runtime, so end users never need to recompile. Music Mode runs a chosen effect while the player is playing. Bump version to 26.10.13.1. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
All LED strip entities are now in Core.yaml and disabled by default in Home Assistant. Both strips defaulted to all 192 RMT TX symbols, so whichever set up second failed: the onboard LED stopped working with the strip in use, or the strip hit RMT TX timeouts. Give the onboard LED 48 and the strip 144. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Add LED strip support on the IO1 header (disabled by default)
|
Warning Review limit reachedYou've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository. Next included review available in 55 minutes. View limit details
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Music Mode watched the main player, which only goes idle ~10s after stop while its buffers drain, then waited 10s more. Watch Sendspin's player and the HTTP source instead, which report stop at once, and bridge gaps for 3s. Keep the strip's RMT buffers in internal RAM and re-send the settled frame, so a frame cut short under audio load can't leave LEDs lit after turn-off. Bump version to 26.10.13.2. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
LEDs past the LED Strip Count are never written, so on a strip longer than the count they kept colors latched from IO1 noise during reboots. Blank every pixel the buffers cover at boot and once the strip is off. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Fix Music Mode stop delay and stray lit LEDs on long strips
Bunton33
changed the title
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Add CAST-1 product photo to the README
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Center the README product photo
bharvey88 is no longer on the team, so every auto-assign run failed. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Auto-assign new PRs and issues to WestwardWinds
- Replace the ignored PlatformIO flash_mode with esp32 flash_mode: dio and the manual PSRAM sdkconfig options with execute_from_psram (identical sdkconfig and memory use; see #92) - Log the WizMote sequence as uint32 and build it from widened bytes - Pin release builds to ESPHome 2026.9.1 until a 2026.10+ build is confirmed to connect to Music Assistant over Sendspin - Bump esphome/workflows to 2026.9.0 for the beta build (from #79) and point Dependabot at beta - Bump version to 26.10.14.1 Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Use native ESP32 options, fix WizMote log format, pin release ESPHome
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Unpin release ESPHome and keep Dependabot on main
Bunton33
changed the title
Keeps the config matching the Home Assistant Voice PE template, as Trevor asked. Bump version to 26.10.14.2. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Restore the Voice PE flash mode and PSRAM options
Bunton33
changed the title
Not ready to change auto-assign yet. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Revert auto-assign change from #96
Inlines the LED strip values instead of substitutions. Bump version to 26.10.14.3. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Move LED strip code into led_strip.yaml
Bunton33
changed the title
Matches main (#102), so new PRs go to WestwardWinds instead of a hard-coded TrevorSchirmer. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
2 of 10 tasks
Use the AUTO_ASSIGNEE org variable for auto-assign
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Version: 26.10.14.3
What does this implement/fix?
led_strip.yaml(Add LED strip support on the IO1 header (disabled by default) #90, Fix Music Mode stop delay and stray lit LEDs on long strips #93, Move LED strip code into led_strip.yaml #101)Types of changes
Checklist / Checklijst:
If user-visible functionality or configuration variables are added/modified:
🤖 Generated with Claude Code