WLED icon indicating copy to clipboard operation
WLED copied to clipboard

feat: bus status usermod

Open casesolved-co-uk opened this issue 1 year ago • 10 comments

Works roughly similar to the STATUSLED functionality.

Now implemented as an effect usermod, description in the README

casesolved-co-uk avatar Jan 01 '24 18:01 casesolved-co-uk

The changes to core functionality are unacceptable just to support usermod unless done wisely with the possibility to be used in other usermods or in core. I do not see such usability here.

blazoncek avatar Jan 01 '24 19:01 blazoncek

The changes to core functionality are unacceptable just to support usermod unless done wisely with the possibility to be used in other usermods or in core. I do not see such usability here.

Could you be more specific and make some suggestions please?

The changes to core functionality are fully backward compatible.

casesolved-co-uk avatar Jan 01 '24 19:01 casesolved-co-uk

If you want a status to be displayed on a segment, why bother with all this?

Just make a custom effect function, that can be applied to any segment. You can append your custom effect to regular effects using strip.addEffect() in your setup().

Do not complicate things when they don't need to be complicated. 😉

blazoncek avatar Jan 02 '24 07:01 blazoncek

Similar to #3574

blazoncek avatar Jan 02 '24 10:01 blazoncek

If you want a status to be displayed on a segment, why bother with all this?

Just make a custom effect function, that can be applied to any segment. You can append your custom effect to regular effects using strip.addEffect() in your setup().

Thanks for the tips. How do you suggest I make the segment:

  • Always on, freeze doesn't appear to work when changing another segment
  • Static, for example having the breathe effect on an adjacent segment also makes the status fade even though not selected

casesolved-co-uk avatar Jan 03 '24 05:01 casesolved-co-uk

Just a regular segment with regular (custom) effect applied. But I'm not sure I understand what you want. And BTW never force push your changes. Do normal commits and normal pushes.

blazoncek avatar Jan 03 '24 07:01 blazoncek

Just a regular segment with regular (custom) effect applied.

This is not true.

  • If you try putting the Breathe and Solid effects on separate segments (and busses), the Solid effect is still faded in and out
  • Freeze of the Solid segment causes it to fade to black

version v0.14.1-b2 Should I start a new bug issue?

casesolved-co-uk avatar Jan 03 '24 18:01 casesolved-co-uk

This is not true.

How so? Perhaps I am the only one who knows how WLED works as I wrote the whole damn thing regarding segments, transitions and effect blending. 😁

* If you try putting the Breathe and Solid effects on separate segments (and busses), the Solid effect is still faded in and out

Faded in what way? Are we talking about fading aka transitions or displaying a WLED status (whatever that means to you) on a part of the strip (aka segment)? If you are talking about transitions (when powering WLED on or off) then yes all segments are transitioned (faded) in and out. When WLED is in off state nothing is displayed on strips except if using STATUSLED (aka setStatusPixel()). If you are talking about displaying separate content of segments then each and every segment can display separate effect.

* Freeze of the Solid segment causes it to fade to black

Freeze does not do that. It only stops updating segment with effect data.

version v0.14.1-b2 Should I start a new bug issue?

IDK but it looks to me that maybe we are not talking about the same thing. It will be very difficult to convince me to approve current implementation as I think the "status usermod" could be done without changing WLED core functions.

blazoncek avatar Jan 03 '24 18:01 blazoncek

Faded in what way? Are we talking about fading aka transitions or displaying a WLED status (whatever that means to you) on a part of the strip (aka segment)?

I've posted a video showing the problems. The implementation of this usermod has now been changed to just be an effect as you requested. The issues I'm seeing are part of the core functionality because I used the official release build for the video; it did not include this usermod.

casesolved-co-uk avatar Jan 03 '24 22:01 casesolved-co-uk

Hardware (power) issue, not WLED functionality.

blazoncek avatar Jan 04 '24 06:01 blazoncek