Error compiling for Arduino / Energy Monitor Shield

Christian Wilhelm

Hello,

I am trying to get the Energy Monitor Shield from Seeed to work (http://wiki.seeedstudio.com/wiki/Energy_Monitor_Shield_V0.9b) but struggling to use the provided sketch. As far as I know I need to transform the code to make it work with MySensors 2.0. I am rubbish with code (only know a little python) and have no clue where to start or decipher the error messages.

The original code looks like this:

/*
 This example code is in the public domain. 
 */
 
#include <SPI.h>
#include <MySensor.h>  // Include MySensors.org Library V1.5
#include "EmonLib.h" // Include Emon Library
#include <LCD5110_Graph_SPI.h> // Include NOKIA5110 Library
 
#define CHILD_ID_POWER 0
 
EnergyMonitor emon;
 
LCD5110 myGLCD(5,6,3);
extern unsigned char SmallFont[];
 
MyTransportNRF24 transport(7, 8); //for EMv1
MySensor gw(transport);
 
unsigned long lastSend;
unsigned long SEND_FREQUENCY = 20000; // Minimum time between send (in milliseconds). We don't wnat to spam the gateway.
 
float Irms;
float lastIrms = -99;
 
char tbuf[8];
char sbuf[12];
 
MyMessage IrmsMsg(CHILD_ID_POWER, V_KWH);
 
void setup()  
{ 
  myGLCD.InitLCD();
  myGLCD.setFont(SmallFont);
  myGLCD.update();
 
  // The third argument enables repeater mode.
  gw.begin(NULL, AUTO, true),
  gw.sendSketchInfo("Energy Monitor v1", "1.0");
 
//  emon.current(0, 111.1);             // Current: input pin, calibration.
   emon.current(0, 66.5); 
 
  // Register all sensors to gw (they will be created as child devices)
 
  gw.present(CHILD_ID_POWER, S_POWER); 
}
 
void loop()      
{      
  gw.process(); 
  unsigned long now = millis();
 
  double Irms = emon.calcIrms(1480);  // Calculate Irms only
  float realIrms  = emon.Irms*220;        //extract Real Power into variable
 
 if (realIrms != lastIrms) {
      gw.send(IrmsMsg.set(realIrms, 1)); //send to gateway
 
  lastIrms=realIrms;
  }
 
   dtostrf(realIrms,5,2,tbuf);
    sprintf(sbuf, "  %s kWt", tbuf);
    myGLCD.print(sbuf, 20, 0);   
    myGLCD.print("Powr:", 0, 0);  
 
    dtostrf(Irms,5,2,tbuf);
    sprintf(sbuf, "  %s Amp", tbuf);
    myGLCD.print(sbuf, 20, 10);   
    myGLCD.print("Irms:", 0, 10);  
 
    myGLCD.update(); 
 
      Serial.print("Power: ");
      Serial.println(realIrms);
 
  gw.sleep(SEND_FREQUENCY);
  }```


Then I transformed it to:

/*
This example code is in the public domain.
*/

#include <nRF24L01.h>
#include <RF24.h>
#include <RF24_config.h>
#include <SPI.h>
#include <MySensors.h> // Include MySensors.org Library V1.5
#include "EmonLib.h" // Include Emon Library
#include <LCD5110_Graph_SPI.h> // Include NOKIA5110 Library

#define CHILD_ID_POWER 0
#define MY_RADIO_NRF24

EnergyMonitor emon;

LCD5110 myGLCD(5,6,3);
extern unsigned char SmallFont[];

MyTransportNRF24 transport(7, 8); //for EMv1

unsigned long lastSend;
unsigned long SEND_FREQUENCY = 20000; // Minimum time between send (in milliseconds). We don't wnat to spam the gateway.

float Irms;
float lastIrms = -99;

char tbuf[8];
char sbuf[12];

MyMessage IrmsMsg(CHILD_ID_POWER, V_KWH);

void setup()
{
myGLCD.InitLCD();
myGLCD.setFont(SmallFont);
myGLCD.update();

// The third argument enables repeater mode.
begin(NULL, AUTO, true),
sendSketchInfo("Energy Monitor v1", "1.0");

// emon.current(0, 111.1); // Current: input pin, calibration.
emon.current(0, 66.5);

// Register all sensors to gw (they will be created as child devices)

present(CHILD_ID_POWER, S_POWER);
}

void loop()
{

unsigned long now = millis();

double Irms = emon.calcIrms(1480); // Calculate Irms only
float realIrms = emon.Irms*220; //extract Real Power into variable

if (realIrms != lastIrms) {
send(IrmsMsg.set(realIrms, 1)); //send to gateway

lastIrms=realIrms;
}

dtostrf(realIrms,5,2,tbuf);
sprintf(sbuf, " %s kWt", tbuf);
myGLCD.print(sbuf, 20, 0);
myGLCD.print("Powr:", 0, 0);

dtostrf(Irms,5,2,tbuf);
sprintf(sbuf, "  %s Amp", tbuf);
myGLCD.print(sbuf, 20, 10);   
myGLCD.print("Irms:", 0, 10);  

myGLCD.update(); 

  Serial.print("Power: ");
  Serial.println(realIrms);

sleep(SEND_FREQUENCY);
} ```

But still get the same error which is:

MySensors.h:287:4: error: #error No forward link or gateway feature activated. This means nowhere to send messages! Pretty pointless.
#error No forward link or gateway feature activated. This means nowhere to send messages! Pretty pointless.
^
exit status 1
Error compiling for board Arduino/Genuino Mega or Mega 2560.

I think the sketch was written some time ago, using MySensors 1.5, not sure how to change it to the updated version.

Appreciate any help

Regards

Christian

hek

Move the MySensors include to the bottom of your sketch

Christian Wilhelm

Hi,

Thanks for your help, still struggling to make it work. I have amended the sketch to that:

/*
 This example code is in the public domain. 
 */
 
#include <SPI.h>

#include "EmonLib.h" // Include Emon Library
#include <LCD5110_Graph_SPI.h> // Include NOKIA5110 Library
 
#define CHILD_ID_POWER 0
 
EnergyMonitor emon;
 
LCD5110 myGLCD(5,6,3);
extern unsigned char SmallFont[];
 
MyTransportNRF24 transport(7, 8); //for EMv1
MySensors gw(transport);
 
unsigned long lastSend;
unsigned long SEND_FREQUENCY = 20000; // Minimum time between send (in milliseconds). We don't wnat to spam the gateway.
 
float Irms;
float lastIrms = -99;
 
char tbuf[8];
char sbuf[12];
 
MyMessage IrmsMsg(CHILD_ID_POWER, V_KWH);
 
void setup()  
{ 
  myGLCD.InitLCD();
  myGLCD.setFont(SmallFont);
  myGLCD.update();
 
  // The third argument enables repeater mode.
  begin(NULL, AUTO, true),
  sendSketchInfo("Energy Monitor v1", "1.0");
 
//  emon.current(0, 111.1);             // Current: input pin, calibration.
   emon.current(0, 66.5); 
 

 
void loop()      
{      
  process(); 
  unsigned long now = millis();
 
  double Irms = emon.calcIrms(1480);  // Calculate Irms only
  float realIrms  = emon.Irms*220;        //extract Real Power into variable
 
 if (realIrms != lastIrms) {
      gw.send(IrmsMsg.set(realIrms, 1)); //send to gateway
 
  lastIrms=realIrms;
  }
 
   dtostrf(realIrms,5,2,tbuf);
    sprintf(sbuf, "  %s kWt", tbuf);
    myGLCD.print(sbuf, 20, 0);   
    myGLCD.print("Powr:", 0, 0);  
 
    dtostrf(Irms,5,2,tbuf);
    sprintf(sbuf, "  %s Amp", tbuf);
    myGLCD.print(sbuf, 20, 10);   
    myGLCD.print("Irms:", 0, 10);  
 
    myGLCD.update(); 
 
      Serial.print("Power: ");
      Serial.println(realIrms);
 
  sleep(SEND_FREQUENCY);
  }
  #include <MySensors.h>  // Include MySensors.org Library V2.0

Still get this error message:

Documents/Arduino/libraries/MySensors-master/MySensors.h:287:4: error: #error No forward link or gateway feature activated. This means nowhere to send messages! Pretty pointless.
#error No forward link or gateway feature activated. This means nowhere to send messages! Pretty pointless.
^
exit status 1
Error compiling for board Arduino/Genuino Mega or Mega 2560.

Thanks

Christian

hek

You seem to have picked some old 1.5 sketch and compile it using MySensors 2.0 .

Christian Wilhelm

Yes the sketch was written using MySensors 1.5, but these don't seem to work anymore with the new IDE. I assume to code needs to look quite a bit differently.

Oleg Greengo

@Christian-Wilhelm
try this sketch


/*
This sketch is for a Energy Monitor v 1.0 
http://www.elecrow.com/cooperated-designers/devicter/energy-monitor-shield-p-835.html

and MySensors 2.0
https://www.mysensors.org/

 modified
 17 Oct 2016
 by greengo
 */
// Enable debug prints to serial monitor
#define MY_DEBUG

// Enable and select radio type attached
#define MY_RADIO_NRF24

#define MY_NODE_ID       AUTO

#define SKETCH_NAME "Energy Monitor v1.0"
#define SKETCH_VERSION "2.0.0"

// Enable repeater functionality for this node
#define MY_REPEATER_FEATURE

#define MY_RF24_CHANNEL 76

// Connected we have to move CE/CSN pins for NRF radio
#define MY_RF24_CE_PIN   7
#define MY_RF24_CS_PIN   8

#include <SPI.h>
#include <MySensors.h>
#include "EmonLib.h"
#include <Time.h>
#include <LCD5110_Graph_SPI.h>

LCD5110 myGLCD(5,6,3);
//extern unsigned char SmallFont[];
extern uint8_t SmallFont[];

#define WINDOW 15

// How many milli seconds between each measurement
#define MEASURE_INTERVAL    1000 //for 1 sec

// FORCE_TRANSMIT_INTERVAL, this number of times of wakeup, the sensor is forced to report all values to the controller
#define FORCE_TRANSMIT_INTERVAL 600 //every 10 min

unsigned long SLEEP_TIME = 1000; // Sleep time between reads (in milliseconds)

int TEMP_TRANSMIT_THRESHOLD = 0; //~3-5 Watt

#define CHILD_ID_POWER 0
#define CHILD_ID_FLOATING_CUR_SENSOR 1 //floating sensor

EnergyMonitor emon;

unsigned long CHECK_TIME = millis();

MyMessage IrmsMsg(CHILD_ID_POWER, V_WATT);
MyMessage kWhMsg(CHILD_ID_POWER, V_KWH);
MyMessage FloatingMsg(CHILD_ID_FLOATING_CUR_SENSOR, V_VAR1);

// Global settings
int measureCount = 0;
boolean transmission_occured = false;

boolean timeReceived = false;
unsigned long lastUpdate = 0, lastRequest = 0;

// Storage of old measurements
double realWatt = 0;
float realkWt = 0;
double Irms;
float lastIrms = 0;
int Floating;
float PMin = 99;
float PMax = 0;

int stateFloating = 0;

long wattsum  = 0;
long wattsumS = 0;
//int seconds   = 0;
double wh     = 0;
double kwh    = 0;
double whS    = 0;
double kwhS   = 0;

void setup()  
{ 
 double cIrms = 0;  
   emon.current(0, 11.0);        // Current: input pin, calibration.

 double Irms[WINDOW];   
         Irms[0] = emon.calcIrms(1480); // первое значение при измерении явно "кривое"
  //    Serial.println("calculate delta");
      for (int i=0; i<WINDOW; i++) {
        Irms[i] = emon.calcIrms(1480);
     cIrms = cIrms + Irms[i];
        delay(100);      
      } 
      
sendPowerMeasurements(true);
  wait(500);
  
  //request Time
  requestTime();
  
  myGLCD.InitLCD();
  myGLCD.setFont(SmallFont);
}

void presentation()  
{   
// Send the sketch version information to the gateway and Controller
  
  sendSketchInfo(SKETCH_NAME, SKETCH_VERSION);

  // Register all sensors to gw (they will be created as child devices)
  present(CHILD_ID_POWER, S_POWER, "POWER");
//present(CHILD_ID_POWER_COUNTER, S_CUSTOM, "COUNTER");
  present(CHILD_ID_FLOATING_CUR_SENSOR, S_CUSTOM, "Float");
}

// This is called when a new time value was received
void receiveTime(unsigned long time) {
  Serial.print("Time value received: ");
  Serial.println(time);
  // Ok, set incoming time
  setTime(time);
  timeReceived = true;
}

void loop()      
{       
   unsigned long now = millis();

  // If no time has been received yet, request it every 10 second from controller
  // When time has been received, request update every hour
  if ((!timeReceived && now - lastRequest > (unsigned long)10 * 1000)
      || (timeReceived && now - lastRequest > (unsigned long)60 * 1000 * 60)) {
    // Request time from controller.
    Serial.println("Requesting time");
    Serial.println("Time now: ");
    Serial.println(timeReceived);
    wait(300);
    requestTime();
    lastRequest = now;
  }
  //*********************************
   
   displayUpdate();
   
  int HH = hour();
  int MM = minute();
  int SS = second();
   
  unsigned long NOW_TIME = millis();
  
  if(NOW_TIME - CHECK_TIME >= SLEEP_TIME)
  { 
  measureCount ++;
  
    double Irms = emon.calcIrms(1480);  // Calculate Irms only
    realWatt  = (emon.Irms * 220);  // Extract Real Power into variable
    long watt = Irms * 220.0;
    realkWt   = (Irms * 0.220);

    wattsum = wattsum + watt;
    wattsumS = wattsumS + watt;
    //   seconds++;

/*
    Serial.print("    Real TIME   =    ");
    Serial.print(hour());
    Serial.print(" : ");
    Serial.print(minute());
    Serial.println();
*/
    // hours KW reading
    if ( (MM >= 59) & (SS >= 59)) {
      wh = wh + wattsum / 3600;
      kwh = wh / 1000;
      send(kWhMsg.set(kwh, 4)); // Send kwh value to gw

      wattsum = 0;
      wh = 0;
      //   seconds = 0;
      // end of hourly KW reading
    }
  
    // fixing minimum and maximum values for Nokia
  if ( realkWt > PMax) PMax = realkWt;
  if ( realkWt < PMin &&  realkWt > 0.05) PMin = realkWt;   
  
  
  bool forceTransmit = false;
  transmission_occured = false;
 
  
  if (measureCount > FORCE_TRANSMIT_INTERVAL) { // force a transmission
    forceTransmit = true; 
    measureCount = 0;
   
      send(kWhMsg.set(kwh, 4)); // Send kwh value to gw 
    
  }
     sendPowerMeasurements(forceTransmit);
   
 // Serial.print(" measureCount: ");
 // Serial.println(measureCount); 
 // Serial.print(" ");
  
       CHECK_TIME = NOW_TIME;
     } 
  }

void sendPowerMeasurements(bool force)
{
  bool tx = force;       
  
 // Set relay to last known state (using eeprom storage)
  TEMP_TRANSMIT_THRESHOLD = loadState(CHILD_ID_FLOATING_CUR_SENSOR);
  Floating = TEMP_TRANSMIT_THRESHOLD;

  //  Serial.print("TEMP_TRANSMIT_THRESHOLD: ");Serial.println(TEMP_TRANSMIT_THRESHOLD);

  float diffIrms = abs(lastIrms - realWatt);

  //  Serial.print(F("IrmsDiff :"));Serial.println(diffIrms);

  if (diffIrms > TEMP_TRANSMIT_THRESHOLD) tx = true;

  if (tx) {
    measureCount = 0;

    //   Serial.print("Watt: ");Serial.println(realWatt);
  
    send(IrmsMsg.set(realWatt, 1));

    lastIrms = realWatt;
    transmission_occured = true;
  }
/*
  //  Serial.print("seconds : "); Serial.println(seconds);
  Serial.print(" ***************** ");
  Serial.print("wh : "); Serial.println(wh);
  Serial.print(" ");
  Serial.print("kwhS : "); Serial.println(kwhS, 4);
  Serial.print(" ***************** ");
  Serial.print("wattsum : "); Serial.println(wattsum);
  Serial.print(" ");
  Serial.print("wattsumS : "); Serial.println(wattsumS);
  Serial.print(" ");
  Serial.print("Irms : "); Serial.println(Irms, 4);
  Serial.print(" ");
  Serial.print("kwh : "); Serial.println(kwh, 4);
  Serial.println(" ");
  Serial.print("Watt: "); Serial.println(realWatt);
  Serial.println(" ");
  Serial.print("realkWt : ");Serial.println(realkWt);
*/  
}  
  // LCD Nokia 
 void displayUpdate()
 {
// myGLCD.clrScr();
  
 char tbuf[8];
 char sbuf[12];
    
    dtostrf(realWatt,5,2,tbuf);
    sprintf(sbuf, " %s Watt", tbuf);
    myGLCD.print(sbuf, 20, 0);   //от края, высота 
    myGLCD.print("PWR:", 0, 0);   //от края, высота  
      
    dtostrf(Irms,5,2,tbuf);
    sprintf(sbuf, " %s Amp", tbuf);
    myGLCD.print(sbuf, 20, 10);   //от края, высота 
    myGLCD.print("IRM:", 0, 10);   //от края, высота 
    
    dtostrf(PMin,5,2,tbuf);
    sprintf(sbuf, " %s kWt", tbuf);
    myGLCD.print(sbuf, 20, 20);   //от края, высота 
    myGLCD.print("MiN:", 0, 20);   //от края, высота 
    
    dtostrf(PMax,5,2,tbuf);
    sprintf(sbuf, " %s kWt", tbuf);
    myGLCD.print(sbuf, 20, 30);   //от края, высота 
    myGLCD.print("MaX:", 0, 30);   //от края, высота 
    
    dtostrf(Floating,5,2,tbuf);
    sprintf(sbuf, " %s Watt", tbuf);
    myGLCD.print(sbuf, 20, 40);   //от края, высота 
    myGLCD.print("flt:", 0, 40);   //от края, высота 
    
    myGLCD.update();     
}
//*********************************************
 void receive(const MyMessage &message) {
  // We only expect one type of message from controller. But we better check anyway.
  if (message.isAck()) {
     Serial.println("This is an ack from gateway");
  }

  if (message.type == V_VAR1) {
     // Change state
     stateFloating = message.getInt();
 
     // Store state in eeprom
     saveState(CHILD_ID_FLOATING_CUR_SENSOR, stateFloating);
    
     // Write some debug info
     Serial.print("Incoming change for sensor:");
     Serial.print(message.sensor);
     Serial.print(", Delta =: ");
     Serial.println(message.getInt());
   //  Serial.print(", New status: ");
   //  Serial.println(message.getBool());
   } 
 }   ```

Christian Wilhelm

Hi,

Thanks for the sketch :)

I managed to find the time library but seem to have a problem with settime I don't understand:

It says 'setTime' was not declared in this scope

: In function 'void receiveTime(long unsigned int)':
New_powermonitor:132: error: 'setTime' was not declared in this scope
setTime(time);
^
: In function 'void loop()':
New_powermonitor:156: error: 'hour' was not declared in this scope
int HH = hour();
^
New_powermonitor:157: error: 'minute' was not declared in this scope
int MM = minute();
^
New_powermonitor:158: error: 'second' was not declared in this scope
int SS = second();
^
: In function 'void displayUpdate()':
277:30: warning: deprecated conversion from string constant to 'char*' [-Wwrite-strings]
myGLCD.print("PWR:", 0, 0); //от края, высота
^
:282:31: warning: deprecated conversion from string constant to 'char*' [-Wwrite-strings]
myGLCD.print("IRM:", 0, 10); //от края, высота
^
:287:31: warning: deprecated conversion from string constant to 'char*' [-Wwrite-strings]
myGLCD.print("MiN:", 0, 20); //от края, высота
^
:292:31: warning: deprecated conversion from string constant to 'char*' [-Wwrite-strings]
myGLCD.print("MaX:", 0, 30); //от края, высота
^
:297:31: warning: deprecated conversion from string constant to 'char*' [-Wwrite-strings]
myGLCD.print("flt:", 0, 40); //от края, высота
^
exit status 1
'setTime' was not declared in this scope

Vincent Pandoy

Does anyone here done with this code?

Oleg Greengo

@Christian-Wilhelm

#include <Time.h>
#include <LCD5110_Graph_SPI.h>

add them to your library
C:\Program Files\Arduino\libraries

Error compiling for Arduino / Energy Monitor Shield

22

11.7k

11.2k

113.1k