Skip to content

fix for https://github.com/wled/WLED/issues/4488 #4692

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 1 commit into
base: main
Choose a base branch
from

Conversation

DedeHai
Copy link
Collaborator

@DedeHai DedeHai commented May 19, 2025

virtual strip index is added even if strip is 1D. this change fixes FX using virtual strips not working when WLED_DISABLE_2D is used.

I recommend waiting to merge this until segment layering PR is merged.

Summary by CodeRabbit

  • Bug Fixes
    • Improved handling of pixel indices to ensure correct color assignment even when 2D support is disabled. This change prevents unexpected behavior when setting pixel colors with out-of-range indices.

virtual strip index is added even if strip is 1D. this change fixes FX using virtual strips not working when WLED_DISABLE_2D is used.

This comment was marked as resolved.

@DedeHai
Copy link
Collaborator Author

DedeHai commented May 19, 2025

this fix does NOT need porting to 0.15 as the bug this PR fixes was introduced in 0.16

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