-
-
Notifications
You must be signed in to change notification settings - Fork 3.9k
Fix operation of length 1 strips, such as PWM LEDs #4878
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
Conversation
This reverts commit e5ba97b.
WalkthroughUpdated WS2812FX::blendSegment in wled00/FX_fcn.cpp to remove an early return for single-pixel segments, allowing them to follow the standard blending path. A later guard still enforces BLEND_STYLE_FADE when width*height == 1. No public API changes. Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Suggested reviewers
Tip 🔌 Remote MCP (Model Context Protocol) integration is now available!Pro plan users can now connect to remote MCP servers from the Integrations page. Connect with popular remote MCPs such as Notion and Linear to add more context to your reviews and chats. 📜 Recent review detailsConfiguration used: CodeRabbit UI Review profile: CHILL Plan: Pro 💡 Knowledge Base configuration:
You can enable these sources in your CodeRabbit configuration. 📒 Files selected for processing (1)
💤 Files with no reviewable changes (1)
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)
✨ Finishing Touches
🧪 Generate unit tests
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. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. CodeRabbit Commands (Invoked using PR/Issue comments)Type Other keywords and placeholders
CodeRabbit Configuration File (
|
|
If we just revert, will we not then see a crash back during blend? |
|
yes, but that needs a different fix. |
|
Sorry about this, my bad. I'll look at the proper fix of the panic issue |
Revert e5ba97b because it disables all length 1 segments, commonly used for PWM outputs. Segments of length 1 are valid and expected under many conditions.
This fixes the output issues reported by @naster_mick on Discord and @qistoph in #4782.
Summary by CodeRabbit