lab11 / powerblade

1 in² AC power meter

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Num Received Number Confusing?

bradjc opened this issue · comments

With multiple powerblades, sometimes it prints 8, sometimes 0, which I believe are both incorrect (assuming it is a count of the repeated advertisements that powerblade sends). After a brief look, it seems like iden_count is a global value that will get incremented by multiple powerblades, leading to what I'm seeing.

Also, "Packets in last 30s" doesn't seem quite right, and looking at the code it seems the maximum number of packets considered is capped at 30, even if there may have been more.

Yea both of those were hacky, put in place in a time of need. I think this and other analytics don’t belong in the shipping version of this code. All the shipping version should do is unpack the packet, calculate and apply the scaling values, and calculate PF.

On Dec 1, 2015, at 11:21 PM, Brad Campbell notifications@github.com wrote:

Also, "Packets in last 30s" doesn't seem quite right, and looking at the code it seems the maximum number of packets considered is capped at 30, even if there may have been more.


Reply to this email directly or view it on GitHub #2 (comment).

Data access example script has been simplified as of edfb0dd