EmonLibCM

EmonLibCM copied to clipboard

Long promised cleaned up commit to support DS2438 sensors as found here: https://www.ibuttonlink.com/collections/ibuttonlink-sensors-1 (e.g. MS-T sensor)

alandpearson

This affects the avrdb branch. This is more a question than an issue. I tried to calculate the time to pass into EmonLibCM_setADC() as ADC_duration from the data sheet. While...

kt-work

This always evaluates to `true`: https://github.com/openenergymonitor/EmonLibCM/blob/23ca81e0597e87a5e9555d7577a351068b8de708/emonLibCM.cpp#L371-L375 You probably mean `_datalog_period_in_seconds` here: https://github.com/openenergymonitor/EmonLibCM/blob/23ca81e0597e87a5e9555d7577a351068b8de708/emonLibCM.cpp#L325-L326

ramiropolla

There appears to be a typo in the #include directive in the 'max' example. Removing the trailing 'x' allows the Sketch to compile (via the Arduino IDE 2.0).

PaulSchulz

I am looking at the code, and it doesn't seem to follow any particular style guide. For example, indentations are sometimes 2 spaces, and other time 4 spaces. Also, comment...

llpamies

Double check that latest version works ok with the ESP8266

5

See: https://community.openenergymonitor.org/t/vrms-drops-to-zero-no-more-data/13813/7

TrystanLea

There may be another way to do this.. In my emonTX setup, I have solar PV, but I don't have access to clamp the solar production, only at the grid....

magnayn

This has no effect ADCSRA |= (0

JensMunkHansen