Skip to content

Estimated Current Isn't Reported When Off Since 0.15.0 #4383

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

Closed
1 task done
AlexKusnezov opened this issue Dec 12, 2024 · 11 comments · Fixed by #4402
Closed
1 task done

Estimated Current Isn't Reported When Off Since 0.15.0 #4383

AlexKusnezov opened this issue Dec 12, 2024 · 11 comments · Fixed by #4402
Labels
discussion documentation documentation should be updated to explain behaviour

Comments

@AlexKusnezov
Copy link

AlexKusnezov commented Dec 12, 2024

What happened?

Estimated Current Isn't Reported When off Since 0.15.0:
Screenshot 2024-12-12 at 16 26 40

0.14.0:
Screenshot 2024-12-12 at 16 28 17

Brighness limiter was on in both cases
After reverting back to 0.14.0 brightness limiter got disbaled so I had to re-enable it manually

Also on Athom bulbs 0.15.0 doesn't report estimated current at all!

To Reproduce Bug

  • update to 0.15.0
  • look at "estimated current"

Expected Behavior

Estimated Current should be prperly reported

Install Method

Binary from WLED.me

What version of WLED?

0.15.0

Which microcontroller/board are you seeing the problem on?

ESP32

Relevant log/trace output

No response

Anything else?

No response

Code of Conduct

  • I agree to follow this project's Code of Conduct
@AlexKusnezov AlexKusnezov changed the title Estimated Current Isn't Reported When Ooff Since 0.15.0 Estimated Current Isn't Reported When Off Since 0.15.0 Dec 12, 2024
@blazoncek
Copy link
Collaborator

0.15.0 uses new ABL, please update settings and verify again.

@AlexKusnezov
Copy link
Author

AlexKusnezov commented Dec 12, 2024

0.15.0 uses new ABL, please update settings and verify again.

Same
unchecked ABL, reboot, check ABL set mA, reboot, result:
"Not Caclulated" when off

@blazoncek
Copy link
Collaborator

That is expected. ABL calculation has been moved into LED output driver to support per output limitations.

@blazoncek blazoncek added discussion documentation documentation should be updated to explain behaviour and removed bug labels Dec 12, 2024
@AlexKusnezov
Copy link
Author

@blazoncek got it, sadly that way the consuption of the esp itself is lost.

@blazoncek
Copy link
Collaborator

ESP current is 120mA constant and irrelevant when LEDs are off as far as calculation goes.

@Riscue
Copy link

Riscue commented Dec 16, 2024

it's not reported to Home Assistant Integration also, lost my power consumption values after 0.15.0

@AlexKusnezov
Copy link
Author

it's not reported to Home Assistant Integration also, lost my power consumption values after 0.15.0

that's right, same here.

@DedeHai
Copy link
Collaborator

DedeHai commented Dec 16, 2024

@blazoncek would it be possible to just report the 0.12A fixed instead of 'not calculated'? Or is there any good reason to break backward compatibility with HA? If this is something that is not possible with the new ABL, then this needs to be reported to HA to implement a fix for the new behaviour IMHO.

@blazoncek
Copy link
Collaborator

@DedeHai please cherry-pick blazoncek/WLED@eb748f5 from my repo.

@DedeHai
Copy link
Collaborator

DedeHai commented Dec 17, 2024

@AlexKusnezov can you check if its ok now? latest bin with a fix is here: https://github.com/Aircoookie/WLED/actions/runs/12378696924

@AlexKusnezov
Copy link
Author

@AlexKusnezov can you check if its ok now? latest bin with a fix is here: https://github.com/Aircoookie/WLED/actions/runs/12378696924

thanks, looks good.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
discussion documentation documentation should be updated to explain behaviour
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants