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

OSP overflow is not 1000 Wh #37

Closed
connorimes opened this issue Apr 5, 2017 · 1 comment
Closed

OSP overflow is not 1000 Wh #37

connorimes opened this issue Apr 5, 2017 · 1 comment
Labels

Comments

@connorimes
Copy link
Member

The existing value of 1000 was assumed by the number of available digits on the device display, but in fact it will go higher and simply not display the full "Wh" on the display once 1000 Wh is reached. If the ODROID Smart Power is plugged into an ODROID device and is performing computation, it can still take days of continuous execution to reach the 1000 Wh threshold. Perhaps the overflow is 10k Wh? 100k Wh? Higher? Leaving this issue open until we can identify the actual overflow point.

Fortunately the implementation doesn't incorrectly detect an overflow when 1000 Wh is reached. A problem will only appear when the actual overflow point is reached, at which point the read function will return an value that is too small.

@connorimes connorimes added the bug label Apr 5, 2017
connorimes added a commit to connorimes/energymon that referenced this issue Apr 11, 2017
Force overflow of Watt-hour counter.
Fixes energymon#37.
connorimes added a commit to connorimes/energymon that referenced this issue Apr 11, 2017
Force overflow of Watt-hour counter.
Document udev permissions for Linux.
Fixes energymon#37.
@connorimes
Copy link
Member Author

Reopening - it seems that the device actually maxes out at 8192.0 Wh. It doesn't overflow there, it just stops increasing in value. Perhaps a floating point overflow interrupt prevents it from rolling over.

So the 10k value we're now forcing a restart at is too high.

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

No branches or pull requests

1 participant