Energy meter pulse sensor (ambiguous) won't compile

Opus40

Hi All

Im trying to get the Energy meter pulse sensor sketch working but when i test the code i get the following message.
(exit status 1
call of overloaded 'set(volatile long unsigned int&)' is ambiguous)

any ideas would be great

thanks in advance

I'm running arduino 1.6.12
and using the development branch

/**
 * 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
 * Contribution by a-lurker and Anticimex,
 * Contribution by Norbert Truchsess <norbert.truchsess@t-online.de>
 * Contribution by Ivo Pullens (ESP8266 support)
 *
 * DESCRIPTION
 * The EthernetGateway sends data received from sensors to the WiFi link.
 * The gateway also accepts input on ethernet interface, which is then sent out to the radio network.
 *
 * VERA CONFIGURATION:
 * Enter "ip-number:port" in the ip-field of the Arduino GW device. This will temporarily override any serial configuration for the Vera plugin.
 * E.g. If you want to use the defualt values in this sketch enter: 192.168.178.66:5003
 *
 * LED purposes:
 * - To use the feature, uncomment any of the MY_DEFAULT_xx_LED_PINs in your sketch, only the LEDs that is defined is used.
 * - RX (green) - blink fast on radio message recieved. In inclusion mode will blink fast only on presentation recieved
 * - TX (yellow) - blink fast on radio message transmitted. In inclusion mode will blink slowly
 * - ERR (red) - fast blink on error during transmission error or recieve crc error
 *
 * See http://www.mysensors.org/build/esp8266_gateway for wiring instructions.
 * nRF24L01+  ESP8266
 * VCC        VCC
 * CE         GPIO4
 * CSN/CS     GPIO15
 * SCK        GPIO14
 * MISO       GPIO12
 * MOSI       GPIO13
 * GND        GND
 *
 * Not all ESP8266 modules have all pins available on their external interface.
 * This code has been tested on an ESP-12 module.
 * The ESP8266 requires a certain pin configuration to download code, and another one to run code:
 * - Connect REST (reset) via 10K pullup resistor to VCC, and via switch to GND ('reset switch')
 * - Connect GPIO15 via 10K pulldown resistor to GND
 * - Connect CH_PD via 10K resistor to VCC
 * - Connect GPIO2 via 10K resistor to VCC
 * - Connect GPIO0 via 10K resistor to VCC, and via switch to GND ('bootload switch')
 *
  * Inclusion mode button:
 * - Connect GPIO5 via switch to GND ('inclusion switch')
 *
 * Hardware SHA204 signing is currently not supported!
 *
 * Make sure to fill in your ssid and WiFi password below for ssid & pass.
 */


// Enable debug prints to serial monitor
#define MY_DEBUG

// Use a bit lower baudrate for serial prints on ESP8266 than default in MyConfig.h
#define MY_BAUD_RATE 9600

// Enables and select radio type (if attached)
//#define MY_RADIO_NRF24
//#define MY_RADIO_RFM69

#define MY_GATEWAY_ESP8266

#define MY_ESP8266_SSID "home"
#define MY_ESP8266_PASSWORD "Password"

// Enable UDP communication
//#define MY_USE_UDP

// Set the hostname for the WiFi Client. This is the hostname
// it will pass to the DHCP server if not static.
// #define MY_ESP8266_HOSTNAME "sensor-gateway"

// Enable MY_IP_ADDRESS here if you want a static ip address (no DHCP)
#define MY_IP_ADDRESS 192,168,1,198

// If using static ip you need to define Gateway and Subnet address as well
#define MY_IP_GATEWAY_ADDRESS 192,168,1,254
#define MY_IP_SUBNET_ADDRESS 255,255,255,0

// The port to keep open on node server mode
#define MY_PORT 5003

// How many clients should be able to connect to this gateway (default 1)
#define MY_GATEWAY_MAX_CLIENTS 2

// Controller ip address. Enables client mode (default is "server" mode).
// Also enable this if MY_USE_UDP is used and you want sensor data sent somewhere.
//#define MY_CONTROLLER_IP_ADDRESS 192, 168, 178, 68

// Enable inclusion mode
//#define MY_INCLUSION_MODE_FEATURE

// Enable Inclusion mode button on gateway
// #define MY_INCLUSION_BUTTON_FEATURE
// Set inclusion mode duration (in seconds)
//#define MY_INCLUSION_MODE_DURATION 60
// Digital pin used for inclusion mode button
//#define MY_INCLUSION_MODE_BUTTON_PIN  3


// Set blinking period
// #define MY_DEFAULT_LED_BLINK_PERIOD 300

// Flash leds on rx/tx/err
// Led pins used if blinking feature is enabled above
//#define MY_DEFAULT_ERR_LED_PIN 16  // Error led pin
//#define MY_DEFAULT_RX_LED_PIN  16  // Receive led pin
//#define MY_DEFAULT_TX_LED_PIN  16  // the PCB, on board LED

#if defined(MY_USE_UDP)
  #include <WiFiUdp.h>
#endif

#include <ESP8266WiFi.h>

#include <MySensors.h>


#define DIGITAL_INPUT_SENSOR 3  // The digital input you attached your light sensor.  (Only 2 and 3 generates interrupt!)
#define PULSE_FACTOR 1000       // Nummber of blinks per KWH of your meeter
#define SLEEP_MODE false        // Watt-value can only be reported when sleep mode is false.
#define MAX_WATT 10000          // Max watt value to report. This filetrs outliers.
#define CHILD_ID 1              // Id of the sensor child

unsigned long SEND_FREQUENCY = 20000; // Minimum time between send (in milliseconds). We don't wnat to spam the gateway.
double ppwh = ((double)PULSE_FACTOR)/1000; // Pulses per watt hour
bool pcReceived = false;
volatile unsigned long pulseCount = 0;   
volatile unsigned long lastBlink = 0;
volatile unsigned long watt = 0;
unsigned long oldPulseCount = 0;   
unsigned long oldWatt = 0;
double oldKwh;
unsigned long lastSend;
MyMessage wattMsg(CHILD_ID,V_WATT);
MyMessage kwhMsg(CHILD_ID,V_KWH);
MyMessage pcMsg(CHILD_ID,V_VAR1);


void setup()  
{  
  // Fetch last known pulse count value from gw
  request(CHILD_ID, V_VAR1);

  // Use the internal pullup to be able to hook up this sketch directly to an energy meter with S0 output
  // If no pullup is used, the reported usage will be too high because of the floating pin
  pinMode(DIGITAL_INPUT_SENSOR,INPUT_PULLUP);

  attachInterrupt(digitalPinToInterrupt(DIGITAL_INPUT_SENSOR), onPulse, RISING);
  lastSend=millis();
}

void presentation() {
  // Send the sketch version information to the gateway and Controller
  sendSketchInfo("Energy Meter", "1.0");

  // Register this device as power sensor
  present(CHILD_ID, S_POWER);
}

void loop()     
{ 
  unsigned long now = millis();
  // Only send values at a maximum frequency or woken up from sleep
  bool sendTime = now - lastSend > SEND_FREQUENCY;
  if (pcReceived && (SLEEP_MODE || sendTime)) {
    // New watt value has been calculated  
    if (!SLEEP_MODE && watt != oldWatt) {
      // Check that we dont get unresonable large watt value. 
      // could hapen when long wraps or false interrupt triggered
      if (watt<((unsigned long)MAX_WATT)) {
        send(wattMsg.set(watt));  // Send watt value to gw 
      }  
      Serial.print("Watt:");
      Serial.println(watt);
      oldWatt = watt;
    }
  
    // Pulse cout has changed
    if (pulseCount != oldPulseCount) {
      send(pcMsg.set(pulseCount));  // Send pulse count value to gw 
      double kwh = ((double)pulseCount/((double)PULSE_FACTOR));     
      oldPulseCount = pulseCount;
      if (kwh != oldKwh) {
        send(kwhMsg.set(kwh, 4));  // Send kwh value to gw 
        oldKwh = kwh;
      }
    }    
    lastSend = now;
  } else if (sendTime && !pcReceived) {
    // No count received. Try requesting it again
    request(CHILD_ID, V_VAR1);
    lastSend=now;
  }
  
  if (SLEEP_MODE) {
    sleep(SEND_FREQUENCY);
  }
}

void receive(const MyMessage &message) {
  if (message.type==V_VAR1) {  
    pulseCount = oldPulseCount = message.getLong();
    Serial.print("Received last pulse count from gw:");
    Serial.println(pulseCount);
    pcReceived = true;
  }
}

void onPulse()     
{ 
  if (!SLEEP_MODE) {
    unsigned long newBlink = micros();  
    unsigned long interval = newBlink-lastBlink;
    if (interval<10000L) { // Sometimes we get interrupt on RISING
      return;
    }
    watt = (3600000000.0 /interval) / ppwh;
    lastBlink = newBlink;
  } 
  pulseCount++;
}

TheoL · I'm running arduino 1.6.12 and using the development branch

@Opus40 I can't compile it for ESP because I don't have it installed. But when I change it to an RFF sensor it compiles without errors in Arduino 1.6.8 IDE.

Could you post the compile errors?

Opus40

hi so this is what i get from the error messages

"

C:\Users\opus\Desktop\GatewayESP8266_-power_meter\GatewayESP8266-_power_meter.ino: In function 'void loop()':

GatewayESP8266_-_power_meter:189: error: call of overloaded 'set(volatile long unsigned int&)' is ambiguous

     send(wattMsg.set(watt));  // Send watt value to gw 

                          ^

C:\Users\opus\Desktop\GatewayESP8266_-power_meter\GatewayESP8266-_power_meter.ino:189:30: note: candidates are:

In file included from C:\Program Files (x86)\Arduino\libraries\MySensors-development/MySensors.h:322:0,

             from C:\Users\opus\Desktop\GatewayESP8266_-_power_meter\GatewayESP8266_-_power_meter.ino:134:

C:\Program Files (x86)\Arduino\libraries\MySensors-development/core/MyMessage.cpp:222:12: note: MyMessage& MyMessage::set(const char*) <near match>

MyMessage& MyMessage::set(const char* value) {

C:\Program Files (x86)\Arduino\libraries\MySensors-development/core/MyMessage.cpp:222:12: note: no known conversion for argument 1 from 'volatile long unsigned int' to 'const char*'

C:\Program Files (x86)\Arduino\libraries\MySensors-development/core/MyMessage.cpp:234:12: note: MyMessage& MyMessage::set(bool)

MyMessage& MyMessage::set(bool value) {

C:\Program Files (x86)\Arduino\libraries\MySensors-development/core/MyMessage.cpp:241:12: note: MyMessage& MyMessage::set(uint8_t)

MyMessage& MyMessage::set(uint8_t value) {

C:\Program Files (x86)\Arduino\libraries\MySensors-development/core/MyMessage.cpp:256:12: note: MyMessage& MyMessage::set(uint32_t)

MyMessage& MyMessage::set(uint32_t value) {

C:\Program Files (x86)\Arduino\libraries\MySensors-development/core/MyMessage.cpp:263:12: note: MyMessage& MyMessage::set(int32_t)

MyMessage& MyMessage::set(int32_t value) {

C:\Program Files (x86)\Arduino\libraries\MySensors-development/core/MyMessage.cpp:270:12: note: MyMessage& MyMessage::set(uint16_t)

MyMessage& MyMessage::set(uint16_t value) {

C:\Program Files (x86)\Arduino\libraries\MySensors-development/core/MyMessage.cpp:277:12: note: MyMessage& MyMessage::set(int16_t)

MyMessage& MyMessage::set(int16_t value) {

GatewayESP8266_-_power_meter:197: error: call of overloaded 'set(volatile long unsigned int&)' is ambiguous

   send(pcMsg.set(pulseCount));  // Send pulse count value to gw 

                            ^

C:\Users\opus\Desktop\GatewayESP8266_-power_meter\GatewayESP8266-_power_meter.ino:197:32: note: candidates are:

In file included from C:\Program Files (x86)\Arduino\libraries\MySensors-development/MySensors.h:322:0,

             from C:\Users\opus\Desktop\GatewayESP8266_-_power_meter\GatewayESP8266_-_power_meter.ino:134:

C:\Program Files (x86)\Arduino\libraries\MySensors-development/core/MyMessage.cpp:222:12: note: MyMessage& MyMessage::set(const char*) <near match>

MyMessage& MyMessage::set(const char* value) {

C:\Program Files (x86)\Arduino\libraries\MySensors-development/core/MyMessage.cpp:222:12: note: no known conversion for argument 1 from 'volatile long unsigned int' to 'const char*'

C:\Program Files (x86)\Arduino\libraries\MySensors-development/core/MyMessage.cpp:234:12: note: MyMessage& MyMessage::set(bool)

MyMessage& MyMessage::set(bool value) {

C:\Program Files (x86)\Arduino\libraries\MySensors-development/core/MyMessage.cpp:241:12: note: MyMessage& MyMessage::set(uint8_t)

MyMessage& MyMessage::set(uint8_t value) {

C:\Program Files (x86)\Arduino\libraries\MySensors-development/core/MyMessage.cpp:256:12: note: MyMessage& MyMessage::set(uint32_t)

MyMessage& MyMessage::set(uint32_t value) {

C:\Program Files (x86)\Arduino\libraries\MySensors-development/core/MyMessage.cpp:263:12: note: MyMessage& MyMessage::set(int32_t)

MyMessage& MyMessage::set(int32_t value) {

C:\Program Files (x86)\Arduino\libraries\MySensors-development/core/MyMessage.cpp:270:12: note: MyMessage& MyMessage::set(uint16_t)

MyMessage& MyMessage::set(uint16_t value) {

C:\Program Files (x86)\Arduino\libraries\MySensors-development/core/MyMessage.cpp:277:12: note: MyMessage& MyMessage::set(int16_t)

MyMessage& MyMessage::set(int16_t value) {

exit status 1
call of overloaded 'set(volatile long unsigned int&)' is ambiguous

"

TheoL

@Opus40 Might be a bug in the development branch. I use the 2.0 release.

What happens if you compile it for a normal arduino and not as a gateway? (Just comment all the gateway and esp stuff and uncomment NRF) That's what I did.

hek

https://forum.mysensors.org/topic/4820/water-meter-pulse-sensor/7

Opus40

Thanks Hek that's done it, should have looked for threads on the water sensor as well. and thanks to you alsoTheoL

Energy meter pulse sensor (ambiguous) won't compile

18

12.0k

11.2k

113.4k