search

jarzebski / Arduino-INA226

INA226 Bi-directional Current/Power Monitor Arduino Librar
MIT License
48 stars 46 forks source link

setShuntVoltageLimit() has been modified #1

Closed neutronstriker closed 1 year ago

neutronstriker commented 9 years ago

Hi,

I just changed uint16_t value = voltage * 25000; to this uint16_t value = voltage / 0.0000025; Because it was setting alert incorrectly for shuntOverlimit() After this change it works fine.

And thank you for your time and effort for making this library.

Regards Neutron

jarzebski commented 9 years ago

Thanks, i will fix it in next release

W dniu 06/14/2015 o 07:56 PM, N Srinivas pisze:

Hi,

I just changed uint16_t value = voltage * 25000; to this uint16_t value = voltage / 0.0000025; Because it was setting alert incorrectly for shuntOverlimit() After this change it works fine.

And thank you for your time and effort for making this library.

Regards Neutron

    You can view, comment on, or merge this pull request online at:

https://github.com/jarzebski/Arduino-INA226/pull/1

    Commit Summary

— Reply to this email directly or view it on GitHub https://github.com/jarzebski/Arduino-INA226/pull/1.