WLED icon indicating copy to clipboard operation
WLED copied to clipboard

Issue when trying to add more then 10 digital buses

Open dosipod opened this issue 1 year ago • 14 comments

What happened?

The 10 buses limit is not clear , It was increased recently as I see in const.h but if you try to add more then 10 and you reboot then it will go back to 10 buses only .

The behavior before was that you can not add more then 10 digital buses as after 10 the led output will be grayed same as below 389810626-55255b56-157a-4e74-9a83-97bc9dd331fa

To Reproduce Bug

Add more then 10 digital buses

Expected Behavior

Exceeding the limit for digital buses should not be allowed in UI

Install Method

Self-Compiled

What version of WLED?

WLED 0.15.0-b7 (build 2410270)

Which microcontroller/board are you seeing the problem on?

ESP32

Relevant log/trace output

No response

Anything else?

No response

Code of Conduct

  • [X] I agree to follow this project's Code of Conduct

dosipod avatar Nov 26 '24 17:11 dosipod

🤔 as this (more than 10 busses on esp32) is a new feature, I think its an option to use as-is for 0.15.RC1, and fix it later.

We should then add a warning to the release notes, saying that >10 outputs are available technically, however the LED settings page has known problems with it, and it may not be possible to configure more outputs than in older versions.

Or we are lucky and someone finds a fix.

softhack007 avatar Nov 26 '24 18:11 softhack007

You could change the label as "needs investigation" and remove bug as I could not do that .

dosipod avatar Nov 26 '24 18:11 dosipod

One thing more noticed is that this limit issue is also there when I tried to test with DDP output , in this case if you save the page with more then 10 then settings/leds will bug and all output will be gone ( strange thing is the master still control the slave even if the setting page is empty ) , rebooting does not help and factory reset might the only option as editing cfg.json might be too much effort

image

image

Debug build did not show any useful error , still the cfg.json is below when saving with more then 10 DDP output

morethen_10_ddp_output_cfg.json

dosipod avatar Nov 26 '24 20:11 dosipod

Just FYI , I do not need 10 outputs with leds nor with DDP . I am happy with 2 and the max I have is 4 output for a unit I made only for testing . But we have received some questions from users with regard to using more then 10 outputs with DDP as they wanted that for a long time so this is not unrealistic setups if you care to check discord .

dosipod avatar Nov 27 '24 15:11 dosipod

Hi, I would like to try DDP more than 10 Athom GU10 WLED bulbs, currently I have 18 bulbs split into 2 ESP32 ethernet controllers and another master controller, if I can try everything with just one controller it would be great, I would like to be able to eliminate the delay to have smooth effects like wipe and another effects; will it be possible to add more than 10 DDP from the next version of WLED? I am very interested in adding many GU10 bulbs and I am available for all tests; thank you!!!

tech1977 avatar Nov 27 '24 15:11 tech1977

@tech1977 ,As the output of more then 10 could be edge case so might be not solved ,for DDP if you need more then 10 output then we have found a simple workaround , in one of the slaves just also use DDP to other slaves , you can see details in discord here https://discord.com/channels/473448917040758787/473448917543944193/1317398474626433024 , if you do that then you can still from one master control the end slaves ( nested DDP ) with segments on the master so that is 32 segments on esp32 .

dosipod avatar Dec 14 '24 08:12 dosipod

@dosipod I did this with 18 test bulbs: bulbs from 1-10 on DDP RGBW output of the first ESP32 ethernet bulbs from 11-18 on DDP RGBW output of the second ESP32 ethernet third ESP32 ethernet as master with DDP RGBW output the 2 previous ESP32s for a total of 18 pixels

tech1977 avatar Dec 16 '24 09:12 tech1977

It would be nice to have this bug fixed for an upcoming QuinLED board that has 15 analog outputs. Is anyone working on this and/or has pointers on how to go about fixing this? It sounds like the issue is with the settings page. I have yet to test actual output, but configuring 11 outputs at compile seems to at least work correctly through debug.

PaoloTK avatar Jan 13 '25 00:01 PaoloTK

@PaoloTK Sure it is a work in progress , look at those two https://github.com/Aircoookie/WLED/issues/4389 https://github.com/Aircoookie/WLED/issues/4380

Or better get in touch with the guys on QuinLED discord for updates and workarounds . We may close this issue or keep it for DDP related as we do not wish to have so many issues related to the same thing .

dosipod avatar Jan 13 '25 02:01 dosipod

@PaoloTK Sure it is a work in progress , look at those two #4389 #4380

I don't think those are affected by the problem reported in this issue? Or is it a more general bug that breaks both certain outputs when used with specific GPIOs due to hardware and all outputs if more than 10 are configured?

Or better get in touch with the guys on QuinLED discord for updates and workarounds . We may close this issue or keep it for DDP related as we do not wish to have so many issues related to the same thing .

Sorry, my message wasn't clear. I wasn't speaking as a customer, but as a dev. I would like to get the bug fixed myself and have started looking into it, but was wondering if anyone had already made some progress to save time.

For example I have noticed this line loops up to i=10:

https://github.com/Aircoookie/WLED/blob/3c19692312df2eda362db394faeae5f92bb5f747/wled00/data/settings_leds.htm#L609

However increasing that alone hasn't fixed the bug, so need to look deeper.

PaoloTK avatar Jan 13 '25 14:01 PaoloTK

At the moment I do not have the HW to test such number of outputs so we just kept this issue for DDP workarounds , the original case was here https://github.com/Aircoookie/WLED/issues/4301 . I think when the guys are now reporting issues with less then 10 outputs it may make sense to focus on that before saying why more then 10 do not work .

You could also open a PR if you think you have something to add that may fix it but I think you would need to dig a bit deeper for that as what you just listed about the settings page is just a ui thing .

dosipod avatar Jan 13 '25 16:01 dosipod

does https://github.com/wled/WLED/pull/4657 fix this?

DedeHai avatar May 19 '25 05:05 DedeHai

does #4657 fix this?

I think it does.

blazoncek avatar May 19 '25 05:05 blazoncek

Can we code this now as fixed @dosipod ?

netmindz avatar May 27 '25 09:05 netmindz