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

Uninitialized values output 0 rather than “NaN” or something more appropriate #4

Open
scruss opened this issue Oct 9, 2014 · 0 comments
Assignees

Comments

@scruss
Copy link
Owner

scruss commented Oct 9, 2014

On startup, energy and temperature fields are not initialized, as the packets are sent asynchronously. For example, three initial records might look like:

[ 87369] Energy:     0 Wh, Power: 576 W, Temp:  0 F
[119189] Energy:     0 Wh, Power: 604 W, Temp: 47 F
[151017] Energy: 17792 Wh, Power: 607 W, Temp: 47 F

In the first line, neither temperature nor energy is initialized. In the second, energy alone isn't.

These should really be something non-numeric, but easy to parse. NaN may not be ideal.

@scruss scruss self-assigned this Oct 9, 2014
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

No branches or pull requests

1 participant