Skip to content
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

POW energy_delta blank #258

Closed
xoseperez opened this issue Oct 28, 2017 · 8 comments
Closed

POW energy_delta blank #258

xoseperez opened this issue Oct 28, 2017 · 8 comments

Comments

@xoseperez
Copy link
Owner

Originally reported by: Chad Faurie (Bitbucket: ChadFaurie, GitHub: Unknown)


Hi

After a while the POW reports nothing in the energy_delta topic on MQTT

@xoseperez
Copy link
Owner Author

The energy_delta topic reports energy consumption over the last minute in Joules. So, previous energy_total value plus current energy_delta should be current energy_total.

@xoseperez
Copy link
Owner Author

Original comment by Chad Faurie (Bitbucket: ChadFaurie, GitHub: Unknown):


I did see that, and it works really well
What I observed however is that the actual topic message returns blank, it does not reflect any value at all where as energy_total works as it should

This occurs after about an hour of up time
I end up with a blank measurement in Home Assistant as below

Screenshot_20171029-210950.png

@xoseperez
Copy link
Owner Author

Original comment by jcoertel (Bitbucket: jcoertel, GitHub: jcoertel):


Hello,
Facing the same problem and here is how it happened : :e

  • report of xxx_energy_delta fine until I turn on the equipment connected to Sonofff POW

  • first xxx_energy_delta report after turning on is OK

  • after that xxx_energy_delta report is (null)

Here is MQTT report (subscribe on .../energy_delta) :

Capture d'écran 2017-11-02 à 14.54.26.png

You can see that turning on rad01 sets the right level for the first report but after topic value is (null).

Did the same with another equipment (rad02) with same result. After a while the values are still null.

Not sure if the this is the same issue.

Cheers

@xoseperez
Copy link
Owner Author

Original comment by jcoertel (Bitbucket: jcoertel, GitHub: jcoertel):


Hello again,

Had a look to the code (file power.ino) and I did a test after increasing the buffer size of conversion buffers.

Previous code was:

char buf_energy_delta[10];
char buf_energy_total[10];

Changed code :

char buf_energy_delta[20];
char buf_energy_total[20];

After that I did not experience the problem anymore.

Maybe it can help to fix the issue.

Cheers

@xoseperez
Copy link
Owner Author

Original comment by Chad Faurie (Bitbucket: ChadFaurie, GitHub: Unknown):


Awesome, I'll be honest I have not had the chance to look at it, I have been traveling the last 2 weeks.

I'll test the fixes when I get back unless they get pulled into the latest release

@xoseperez
Copy link
Owner Author

Changed in dev branch, will be released with 1.9.8

@xoseperez
Copy link
Owner Author

Released with 1.9.8

@xoseperez
Copy link
Owner Author

Removing milestone: 1.9.8 (automated comment)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant