reporting battery to domoticZ

fhenryco

Hello, i'm using the sketch given here https://www.mysensors.org/build/battery without modification to report battery level to domoticZ. obviously i can't use the Vcc library because the voltage i'm monitoring is the battery voltage, while the arduino is fed through the step up regulator at a stable 3.3V.
I 'm not sure to understand then why domoticz shows 60% battery when my battery level is actually at about 3V so i would expect 3/3.44 = 87% .

So i'm suspecting that sendBatteryLevel(batteryPcnt); does not actually send batteryPcnt as computed in the sketch but a rescaled percentage in order to match 0% when the battery will be at about 2.7V .... can someone confirm ? where can i read the actual void sendBatteryLevel() to understand what it is doing exactly ?

sundberg84

@fhenryco said in reporting battery to domoticZ:

where can i read the actual void sendBatteryLevel() to understand what it is doing exactly ?

Domoticz: https://github.com/domoticz/domoticz/blob/3741a530c2260cea2f337a095aa9477847fc00cb/hardware/MySensorsBase.cpp#L444
MySensors: https://github.com/mysensors/MySensors/blob/9041c8ba3d896484b6d9dca6e8ff5e81a828ffa3/core/MySensorsCore.cpp#L308

Maybe you can show us your sketch code? I have this working with Domoticz just fine.

fhenryco

@sundberg84 thank you but as i said i did no modification to the sketch in https://www.mysensors.org/build/battery appart merging it with a DHT11 sketch. However i did notice something strange: when i switch on the arduino (sensor side) the first percentage i get in DomoticZ was 102%, then at the next update it fell to 60% and has not changed since then. I'm suspecting there is something wrong measuring the battery level from A0 pin just after sending a temperature value ??

Insert Code Here/**
 * The MySensors Arduino library handles the wireless radio link and protocol
 * between your home built sensors/actuators and HA controller of choice.
 * The sensors forms a self healing radio network with optional repeaters. Each
 * repeater and gateway builds a routing tables in EEPROM which keeps track of the
 * network topology allowing messages to be routed to nodes.
 *
 * Created by Henrik Ekblad <henrik.ekblad@mysensors.org>
 * Copyright (C) 2013-2015 Sensnology AB
 * Full contributor list: https://github.com/mysensors/Arduino/graphs/contributors
 *
 * Documentation: http://www.mysensors.org
 * Support Forum: http://forum.mysensors.org
 *
 * This program is free software; you can redistribute it and/or
 * modify it under the terms of the GNU General Public License
 * version 2 as published by the Free Software Foundation.
 *
 *******************************
 *
 * REVISION HISTORY
 * Version 1.0: Henrik EKblad
 * Version 1.1 - 2016-07-20: Converted to MySensors v2.0 and added various improvements - Torben Woltjen (mozzbozz)
 * 
 * DESCRIPTION
 * This sketch provides an example of how to implement a humidity/temperature
 * sensor using a DHT11/DHT-22.
 *  
 * For more information, please visit:
 * http://www.mysensors.org/build/humidity
 * 
 */

// Enable debug prints
#define MY_DEBUG

// Enable and select radio type attached 
#define MY_RADIO_NRF24
//#define MY_RADIO_RFM69
//#define MY_RS485
 
#include <SPI.h>
#include <MySensors.h>  
#include <DHT.h>

// Set this to the pin you connected the DHT's data pin to
#define DHT_DATA_PIN 3

// Set this offset if the sensor has a permanent small offset to the real temperatures
#define SENSOR_TEMP_OFFSET 0

// Sleep time between sensor updates (in milliseconds)
// Must be >1000ms for DHT22 and >2000ms for DHT11
static const uint64_t UPDATE_INTERVAL = 600000;
int BATTERY_SENSE_PIN = A0;  // select the input pin for the battery sense point

int oldBatteryPcnt = 0;
// Force sending an update of the temperature after n sensor reads, so a controller showing the
// timestamp of the last update doesn't show something like 3 hours in the unlikely case, that
// the value didn't change since;
// i.e. the sensor would force sending an update every UPDATE_INTERVAL*FORCE_UPDATE_N_READS [ms]
static const uint8_t FORCE_UPDATE_N_READS = 10;

#define CHILD_ID_HUM 0
#define CHILD_ID_TEMP 1

float lastTemp;
float lastHum;
uint8_t nNoUpdatesTemp;
uint8_t nNoUpdatesHum;
bool metric = true;

MyMessage msgHum(CHILD_ID_HUM, V_HUM);
MyMessage msgTemp(CHILD_ID_TEMP, V_TEMP);
DHT dht;


void presentation()  
{ 
  // Send the sketch version information to the gateway
  sendSketchInfo("TemperatureAndHumidity", "1.1");
  sendSketchInfo("Battery Meter", "1.0");
  // Register all sensors to gw (they will be created as child devices)
  present(CHILD_ID_HUM, S_HUM);
  present(CHILD_ID_TEMP, S_TEMP);
  
  metric = getConfig().isMetric;
}


void setup()
{
  
   // use the 1.1 V internal reference
#if defined(__AVR_ATmega2560__)
    analogReference(INTERNAL1V1);
#else
    analogReference(INTERNAL);
#endif

  dht.setup(DHT_DATA_PIN); // set data pin of DHT sensor
  if (UPDATE_INTERVAL <= dht.getMinimumSamplingPeriod()) {
    Serial.println("Warning: UPDATE_INTERVAL is smaller than supported by the sensor!");
  }
  // Sleep for the time of the minimum sampling period to give the sensor time to power up
  // (otherwise, timeout errors might occure for the first reading)
  sleep(dht.getMinimumSamplingPeriod());
}


void loop()      
{  
  // get the battery Voltage
    int sensorValue = analogRead(BATTERY_SENSE_PIN);
#ifdef MY_DEBUG
    Serial.println(sensorValue);
#endif

    // 1M, 470K divider across battery and using internal ADC ref of 1.1V
    // Sense point is bypassed with 0.1 uF cap to reduce noise at that point
    // ((1e6+470e3)/470e3)*1.1 = Vmax = 3.44 Volts
    // 3.44/1023 = Volts per bit = 0.003363075

    int batteryPcnt = sensorValue / 10;
#ifdef MY_DEBUG
    float batteryV  = sensorValue * 0.003363075;
    Serial.print("Battery Voltage: ");
    Serial.print(batteryV);
    Serial.println(" V");

    Serial.print("Battery percent: ");
    Serial.print(batteryPcnt);
    Serial.println(" %");
#endif

    if (oldBatteryPcnt != batteryPcnt) {
        // Power up radio after sleep
        sendBatteryLevel(batteryPcnt);
        oldBatteryPcnt = batteryPcnt;
    }


    
  // Force reading sensor, so it works also after sleep()
  dht.readSensor(true);
  
  // Get temperature from DHT library
  float temperature = dht.getTemperature();
  if (isnan(temperature)) {
    Serial.println("Failed reading temperature from DHT!");
  } else if (temperature != lastTemp || nNoUpdatesTemp == FORCE_UPDATE_N_READS) {
    // Only send temperature if it changed since the last measurement or if we didn't send an update for n times
    lastTemp = temperature;
    if (!metric) {
      temperature = dht.toFahrenheit(temperature);
    }
    // Reset no updates counter
    nNoUpdatesTemp = 0;
    temperature += SENSOR_TEMP_OFFSET;
    send(msgTemp.set(temperature, 1));

    #ifdef MY_DEBUG
    Serial.print("T: ");
    Serial.println(temperature);
    #endif
  } else {
    // Increase no update counter if the temperature stayed the same
    nNoUpdatesTemp++;
  }

  // Get humidity from DHT library
  float humidity = dht.getHumidity();
  if (isnan(humidity)) {
    Serial.println("Failed reading humidity from DHT");
  } else if (humidity != lastHum || nNoUpdatesHum == FORCE_UPDATE_N_READS) {
    // Only send humidity if it changed since the last measurement or if we didn't send an update for n times
    lastHum = humidity;
    // Reset no updates counter
    nNoUpdatesHum = 0;
    send(msgHum.set(humidity, 1));
    
    #ifdef MY_DEBUG
    Serial.print("H: ");
    Serial.println(humidity);
    #endif
  } else {
    // Increase no update counter if the humidity stayed the same
    nNoUpdatesHum++;
  }

  // Sleep for a while to save energy
  sleep(UPDATE_INTERVAL); 
}

sundberg84

@fhenryco - if you are experience changes it might help adding a wait(); or delay(); to get a more stable reading. If something is using power just as you read it might be strange values.

This is one of my sketches (DHT + Bat): https://github.com/sundberg84/MySensors2.0.0/blob/master/NorrHumAs/NorrHumAs.ino and im using the voltage divider to A0 to measure the battery.

fhenryco

@sundberg84 thanks,
the delay 500 has a big effect , now i get 116% hence a battery voltage=3.5V, while the multimeter gives it 3V ! may be the internal reference is not 1.1V in my low cost chinese pro mini 3.3v .
BTW i did not understand why you have also a delay 500 after reading the analog pin, i would have thought that a delay before reading the analog pin is sufficient ... (?)

sundberg84

@fhenryco - could be the delay before is sufficient.

fhenryco

this morning , i get 102% , much closer to the actual value ... assuming that the ADC activity did not discharge my battery at an unprecedented level (i had this sketch working for two monthes with very little discharge of the battery before adding this battery monitoring functionality). I will know for sure if the battery level does not drop further in the coming days...i'm concerned that the delay 500 might produce non negligible battery consumption

sundberg84

@fhenryco Its hard to reach a 100% correct value on the chinese clones. I get a 5% off probably all the time. Because of that I have added an average so I collect three values every 20min into an array and send the average every hour.

fhenryco

ok, thanks again, something else i v been wondering about is why domoticz will show the temp + humidity sensor as a single sensor even though there are separate instructions
for the presentation :
present(CHILD_ID_HUM, S_HUM);
present(CHILD_ID_TEMP, S_TEMP);
and for the messages themselves
MyMessage msgHum(CHILD_ID_HUM, V_HUM);
MyMessage msgTemp(CHILD_ID_TEMP, V_TEMP);
send(msgTemp.set(temperature, 1));
send(msgHum.set(humidity, 1));
what should be modified if i want domoticZ to treat the two sensors separately ? (this is for Yana, a speech recognition software that has a plugin for domoticZ, but apparantly is not able to handle several values for a given sensor).

is it the sendSketchInfo("TemperatureAndHumidity", "1.1"); that needs to be replaced by two sendSketchInfo, one for temp and one for humidity ?

fhenryco

@fhenryco apparantly something like send(msgTemp.setSensor(CHILD_ID_TEMP).set(temperature,1));
send(msgHum.setSensor(CHILD_ID_HUM).set(humidity,1));
should do the job ! going to try that
but i'm surprised this is not done by default since msgHum and msgTemp are defined separately for each sensor Id

fhenryco

@fhenryco it did nothing different ... still have a single sensor temp+hum don't understand why ...

sundberg84

@fhenryco - Yes, and there is a trick to seperate them... and i if I remembe right @AWI is the one to ask.

Edit, found his post here: https://forum.mysensors.org/topic/5132/ds18b20-ans-sht31-d-show-up-as-combined-sensors-on-domoticz/13

fhenryco

using V_text and S_info is a workaround to avoid grouping

Rene046

i would like to ask something to ,

Now i have the battery level showing up in Domoticz what can i do to log the voltage in volts, should i add something inside the sketch or something in domoticz..

fhenryco

@Rene046
This is the instruction in your sketch which allows to see the battery level of devices in domoticZ
sendBatteryLevel(batteryPcnt);
But as you noticed , this is not reported in the logs. One possibility to see it in the Log is to as well send this batteryPcnt information with a send(msgbatt.set(batteryPcnt ,1);
provided you have defined
MyMessage msgbatt(CHILD_ID_BATT, V_TEXT);

In other words you send the battery info as if it was the info of an additional sensor sending text type info
which you also need to present in the presentation void with

present(CHILD_ID_BATT, S_INFO);
For instance if you already have two sensors in you sketch let's say temp and hum with ID =0 and 1 respectively you would give the battery level the next unused ID
CHILD_ID_BATT=2
There might be a simpler and more natural way but i don't know it .

AWI

@Rene046 The "royal" way to report battery level is to define it as S_MULTIMETER and use it to report voltage. Domoticz will take the voltage and can graph it. i.e.

definitions: MyMessage voltageMsg(VOLTAGE_CHILD_ID, V_VOLTAGE);
in presentation: present(VOLTAGE_CHILD_ID, S_MULTIMETER, "Battery level" );
sending: send(voltageMsg.set(lastVoltage,2)); //send battery in Volt 2 decimal places

Rene046

@ fhenryco

Cant get it to work, if you have time could help with my sketch.

// Enable debug prints to serial monitor
#define MY_DEBUG 

// Enable and select radio type attached
#define MY_RADIO_NRF24
//#define MY_RADIO_RFM69


#include <SPI.h>
#include <MySensors.h>
#include <DHT.h>   



// Enable debug prints
#define MY_DEBUG

#define CHILD_ID_HUM 0
#define CHILD_ID_TEMP 1
#define HUMIDITY_SENSOR_DIGITAL_PIN 3
#define CHILD_ID_BATTERY 4
unsigned long SLEEP_TIME = 60000; // Sleep time between reads (in milliseconds)

#define SKETCH_NAME "Temp_Hum_Batt"        // Change to a fancy name you like
#define SKETCH_VERSION "2.1"                    // Your version

DHT dht;
float lastTemp;
float lastHum;
boolean metric = true; 
MyMessage msgHum(CHILD_ID_HUM, V_HUM);
MyMessage msgTemp(CHILD_ID_TEMP, V_TEMP);
MyMessage msgbatt(CHILD_ID_BATTERY, V_VOLTAGE);
//MyMessage msgbatt(CHILD_ID_BATTERY, V_TEXT);

//=========================
// BATTERY VOLTAGE DIVIDER SETUP
// 1M, 470K divider across battery and using internal ADC ref of 1.1V
// Sense point is bypassed with 0.1 uF cap to reduce noise at that point
// ((1e6+470e3)/470e3)*1.1 = Vmax = 3.44 Volts
// 3.44/1023 = Volts per bit = 0.003363075
#define VBAT_PER_BITS 0.0041055718475073313782991202346  
#define VMIN 3.6                                  //  Vmin (radio Min Volt)=1.9V (564v)
#define VMAX 4.2                                  //  Vmax = (2xAA bat)=3.0V (892v)
int batteryPcnt = 0;                              // Calc value for battery %
int batLoop = 0;                                  // Loop to help calc average
int batArray[3];                                  // Array to store value for average calc.
int BATTERY_SENSE_PIN = A0;                       // select the input pin for the battery sense point
//=========================

void setup()  
{ 
 analogReference(INTERNAL);             // For battery sensing

  delay(500); // Allow time for radio if power used as reset
  
  dht.setup(HUMIDITY_SENSOR_DIGITAL_PIN); 

  metric = getControllerConfig().isMetric;
}

void presentation()  
{ 
  // Send the Sketch Version Information to the Gateway
 // Send the Sketch Version Information to the Gateway
  sendSketchInfo(SKETCH_NAME, SKETCH_VERSION);

  // Register all sensors to gw (they will be created as child devices)
  
  
  present(CHILD_ID_BATTERY, S_INFO);
  //present(CHILD_ID_BATTERY, S_CUSTOM);
  present(CHILD_ID_HUM, S_HUM);
  present(CHILD_ID_TEMP, S_TEMP);
}

void loop()      
{  
  delay(500); // Allow time for radio if power used as reset
  delay(dht.getMinimumSamplingPeriod());
 
  // Fetch temperatures from DHT sensor
  float temperature = dht.getTemperature();
  if (isnan(temperature)) {
      Serial.println("Failed reading temperature from DHT");
  } else if (temperature != lastTemp) {
    lastTemp = temperature;
    if (!metric) {
      temperature = dht.toFahrenheit(temperature);
    }
    send(msgTemp.set(temperature, 1));
    Serial.print("T: ");
    Serial.println(temperature);
  }
  
  // Fetch humidity from DHT sensor
  float humidity = dht.getHumidity();
  if (isnan(humidity)) {
      Serial.println("Failed reading humidity from DHT");
  } else if (humidity != lastHum) {
      lastHum = humidity;
      send(msgHum.set(humidity, 1));
      Serial.print("H: ");
      Serial.println(humidity);
  }
  batM();
  sleep(SLEEP_TIME); //sleep a bit
}

void batM() //The battery calculations
{
   delay(500);
   // Battery monitoring reading
   int sensorValue = analogRead(BATTERY_SENSE_PIN);    
   delay(500);
   
   // Calculate the battery in %
   float Vbat  = sensorValue * VBAT_PER_BITS;
   int batteryPcnt = static_cast<int>(((Vbat-VMIN)/(VMAX-VMIN))*100.);
   Serial.print("Battery percent: "); Serial.print(batteryPcnt); Serial.println(" %");  
   
   // Add it to array so we get an average of 3 (3x20min)
   batArray[batLoop] = batteryPcnt;
  
   if (batLoop > 2) {  
     batteryPcnt = (batArray[0] + batArray[1] + batArray[2] + batArray[3]);
     batteryPcnt = batteryPcnt / 3;
 
   if (batteryPcnt > 100) {
     batteryPcnt=100;
 }
 
     Serial.print("Battery Average (Send): "); Serial.print(batteryPcnt); Serial.println(" %");
       sendBatteryLevel(batteryPcnt);
       msgbatt.set(batteryPcnt ,1);
       batLoop = 0;
      }
     else 
     {
     batLoop++;
     }
}```

My sketch is not working 100% my voltage is not precise, giving me even 221 in percentage while  lipo is at this moment 3.7 volt (simulated with a variable power supply)

im running it on a nano on 1 lipo cell and a dc dc powerup to 5 volt measuring the lipo voltage with a  divider 1M/470K
.
Hope to add a soil moisture sensor and baro sensor in future.

p.s. any one with some better idea sketch its ver welcome. im just started to experiment with Mysensor

Rene046

O i forget to tell im running the latest gateway from Mysensor.

fhenryco

@Rene046
not
msgbatt.set(batteryPcnt ,1);
but
send(msgbatt.set(batteryPcnt ,1));

also may be better to use S_MULTIMETER and V_VOLTAGE (rather than S_INFO and V_TEXT) as adviced by AWI and also 2 decimals hence
send(msgbatt.set(batteryPcnt ,2));

Rene046

thx i got a voltage log only giving me now -44.000 volt and 212 prct at battery level while measuring 3.7 volt battery and voltage on A0 1.3 volts
.

// Enable debug prints to serial monitor
#define MY_DEBUG 

// Enable and select radio type attached
#define MY_RADIO_NRF24
//#define MY_RADIO_RFM69

#include <SPI.h>
#include <MySensors.h>
#include <DHT.h>   

// Enable debug prints
#define MY_DEBUG

#define CHILD_ID_HUM 0
#define CHILD_ID_TEMP 1
#define HUMIDITY_SENSOR_DIGITAL_PIN 3
#define CHILD_ID_BATTERY 4
unsigned long SLEEP_TIME = 60000; // Sleep time between reads (in milliseconds)

#define SKETCH_NAME "Temp_Hum_Batt"        // Change to a fancy name you like
#define SKETCH_VERSION "2.1"                    // Your version

DHT dht;
float lastTemp;
float lastHum;
boolean metric = true; 
MyMessage msgHum(CHILD_ID_HUM, V_HUM);
MyMessage msgTemp(CHILD_ID_TEMP, V_TEMP);
MyMessage msgbatt(CHILD_ID_BATTERY, V_VOLTAGE);

//=========================
// BATTERY VOLTAGE DIVIDER SETUP
// 1M, 470K divider across battery and using internal ADC ref of 1.1V
// Sense point is bypassed with 0.1 uF cap to reduce noise at that point
// ((1e6+470e3)/470e3)*1.1 = Vmax = 3.44 Volts
// 3.44/1023 = Volts per bit = 0.003363075
#define VBAT_PER_BITS 0.0041055718475073313782991202346  // 4.20/1023 volt
#define VMIN 3.6                                  //  Vmin = 3.6 lipo empty
#define VMAX 4.2                                  //  Vmax = 4.2 volt lipo full
int batteryPcnt = 0;                              // Calc value for battery %
int batLoop = 0;                                  // Loop to help calc average
int batArray[3];                                  // Array to store value for average calc.
int BATTERY_SENSE_PIN = A0;                       // select the input pin for the battery sense point
//=========================

void setup()  
{ 
 analogReference(INTERNAL);             // For battery sensing

  delay(500); // Allow time for radio if power used as reset
  
  dht.setup(HUMIDITY_SENSOR_DIGITAL_PIN); 

  metric = getControllerConfig().isMetric;
}

void presentation()  
{ 
  // Send the Sketch Version Information to the Gateway
 // Send the Sketch Version Information to the Gateway
  sendSketchInfo(SKETCH_NAME, SKETCH_VERSION);

  // Register all sensors to gw (they will be created as child devices)
  
  
  present(CHILD_ID_BATTERY, S_MULTIMETER);
  present(CHILD_ID_HUM, S_HUM);
  present(CHILD_ID_TEMP, S_TEMP);
}

void loop()      
{  
  delay(500); // Allow time for radio if power used as reset
  delay(dht.getMinimumSamplingPeriod());
 
  // Fetch temperatures from DHT sensor
  float temperature = dht.getTemperature();
  if (isnan(temperature)) {
      Serial.println("Failed reading temperature from DHT");
  } else if (temperature != lastTemp) {
    lastTemp = temperature;
    if (!metric) {
      temperature = dht.toFahrenheit(temperature);
    }
    send(msgTemp.set(temperature, 1));
    Serial.print("T: ");
    Serial.println(temperature);
  }
  
  // Fetch humidity from DHT sensor
  float humidity = dht.getHumidity();
  if (isnan(humidity)) {
      Serial.println("Failed reading humidity from DHT");
  } else if (humidity != lastHum) {
      lastHum = humidity;
      send(msgHum.set(humidity, 1));
      Serial.print("H: ");
      Serial.println(humidity);
  }
  batM();
  sleep(SLEEP_TIME); //sleep a bit
}

void batM() //The battery calculations
{
   delay(500);
   // Battery monitoring reading
   int sensorValue = analogRead(BATTERY_SENSE_PIN);    
   delay(500);
   
   // Calculate the battery in %
   float Vbat  = sensorValue * VBAT_PER_BITS;
   int batteryPcnt = static_cast<int>(((Vbat-VMIN)/(VMAX-VMIN))*100.);
   Serial.print("Battery percent: "); Serial.print(batteryPcnt); Serial.println(" %");  
   
   // Add it to array so we get an average of 3 (3x20min)
   batArray[batLoop] = batteryPcnt;
  
   if (batLoop > 2) {  
     batteryPcnt = (batArray[0] + batArray[1] + batArray[2] + batArray[3]);
     batteryPcnt = batteryPcnt / 3;
 
   if (batteryPcnt > 100) {
     batteryPcnt=100;
 }
 
     Serial.print("Battery Average (Send): "); Serial.print(batteryPcnt); Serial.println(" %");
       sendBatteryLevel(batteryPcnt);
       send(msgbatt.set(batteryPcnt ,2));
       batLoop = 0;
      }
     else 
     {
     batLoop++;
     }
}```

reporting battery to domoticZ

14

11.7k

11.2k

113.1k