Today I completed the installation of the first of my 4 channel 240V LED light dimmers. This is installed in the roof and supports 4 buttons to turn on/off/increase/decrease each channel. It uses an AC zero crossing detector circuit and can be configured for leading or trailing edge dimming.
Hum I am not sure what the deal is here but I have spent the last two days updating and then removing 2.3.0 from Gateways and Repeaters. It completely trashed my 22 node network. I still have some 2.3.0 nodes running but if I installed 2.3.0 and setup #define MY_REPEATER_FEATURE my nodes would crash within about 30 mins of booting. Also on the gateway the through-put of the radio RF24 died to a crawl. I was trying to OTA my nodes and they were going line by line for about 60mins or more each (MYSController). I simply went back to 2.2.0 on Ethernet Gateway made NO other configuration changes and the nodes flashed in less than a minute each.
Since removing 2.3.0 from gateway and repeaters the whole network appears to have stabilised again.
@hek Great thanks - I am setting a pull request right now which fixes another issue with the goGetValveTimes function which causes NACKs due to 3 successive sends to the radio without any pause or delay. This is a simple bug in the code and should not really need a discussion to implement as careful analysis of the DEBUG results will always yield this error. I basically copied the setup code that reads the data - put it in the function and changed the setup to call this updated function.
There is another bug I am trying to stamp out right now which is causing the manual button start to NOT send the status updates to the controller via the radio - again this is a standard bug and may or may not get seen by many if you are not testing the manual button operation. Once this is resolved I will make a pull for it also.
@fredswed thanks for your comments. I have gone with Ethernet gateway as it works well and at this stage I do not need MQTT - maybe if I play with node-red or something down the track but for the moment happy with Ethernet - it just works. I did have MQTT working but the MYSController connection is a possible problem.
I have also gone to the config files directly and have worked out the bridge setup in the things file and have a few test items running.
Yes backups - done that and have my system backing up to a local NAS box.
Cheers David
I had fake interupts on my device - would work great on test bench then with a 5v dc converter that was noisey plus a longer cable to the bucket I was getting a lot of trips. My fix which solved all the issues was a 0.1uF ceramic cap from ground to the signal pin of the bucket (right at the adruino) coupled with an inline resistor 330ohm to the bucket and a 1Kohm resistor from +5v to the signal pin. This is a basic pull up and debounce system and it manages to hold the 5V via the cap with 100% reliability thus far for 3 weeks including a lot of rain over the last few days. I would draw a circuit diagram if someone could tell me the best program to use?
OK so I have added several pull requests to Github for the irrigation controller. I thought I would now show you may take on the irrigation controller firstly a pic of my unit:
The button on the left that is glowing - turns on/off the LCD backlight as most of the time I do not want to see it and have it glowing at night - this box is mounted on a wall outside my house under the eves. The right button is the flashing led coupled with the manual operation function - I also slowed down the on time for this led again to not glow and flash as much.
As I live in Australia I regionalised the displays for none US dates and here is my final code which is setup for 6 zones. I also made a modification to always switch on Zone 8 as a master valve when any other zone is activated. My original system has a master valve which enables the water supply to the other valves and hence needed to switch this on. If you want to copy my code and remove this it is at line 500 - 501 - just remove the ' - 128' from both these lines. The rest of this code has all the fixes added to the github repo for the Irrigation Controller - enjoy.
/*
MySprinkler for MySensors
Arduino Multi-Zone Sprinkler Control
May 31, 2015
*** Version 2.0
*** Upgraded to http://MySensors.org version 1.4.1
*** Expanded for up to 16 Valves
*** Setup for active low relay board or comment out #define ACTIVE_LOW to switch to active high
*** Switch to bitshift method vs byte arrays
*** Changed RUN_ALL_ZONES Vera device to 0 (was highest valve)
*** Added optional LCD display featuring remaining time, date last ran & current time
*** Features 'raindrop' and 'clock' icons which indicate sensor is updating valve data and clock respectively
*** Added single pushbutton menu to manually select which program to run (All Zones or a Single Zone)
*** Added option of naming your Zones programmatically or with Vera (V_VAR3 used to store names)
Utilizing your Vera home automation controller and the MySensors.org gateway you can
control up to a sixteen zone irrigation system with only three digital pins. This sketch
will create NUMBER_OF_VALVES + 1 devices on your Vera controller
This sketch features the following:
Allows you to cycle through All zones (RUN_ALL_ZONES) or individual zone (RUN_SINGLE_ZONE) control.
Use the 0th controller to activate RUN_ALL_ZONES (each zone in numeric sequence 1 to n)
using Variable1 as the "ON" time in minutes in each of the vera devices created.
Use the individual zone controller to activate a single zone. This feature uses
Variable2 as the "ON" time for each individual device/zone.
Connect according to pinout below and uses Shift Registers as to allow the MySensors
standard radio configuration and still leave available digital pins
Turning on any zone will stop the current process and begin that particular process.
Turning off any zone will stop the current process and turn off all zones.
To push your new time intervals for your zones, simply change the variable on your Vera and
your arduino will call to Vera once a minute and update accordingly. Variables will also be
requested when the device is first powered on.
Pushbutton activation to RUN_ALL_ZONES, RUN_SINGLE_ZONE or halt the current program
LED status indicator
PARTS LIST:
Available from the MySensors store - http://www.mysensors.org/store/
Relays (8 channel)
Female Pin Header Connector Strip
Prototype Universal Printed Circuit Boards (PCB)
NRF24L01 Radio
Arduino (I used a Pro Mini)
FTDI USB to TTL Serial Adapter
Capacitors (10uf and .1uf)
3.3v voltage regulator
Resistors (270, 1K & 10K)
Female Dupont Cables
1602 LCD (with I2C Interface)
LED
Push button
Shift Register (SN74HC595)
2 Pole 5mm Pitch PCB Mount Screw Terminal Block
3 Pole 5mm Pitch PCB Mount Screw Terminal Block
22-24 gauge wire or similar (I used Cat5/Cat6 cable)
18 gauge wire (for relay)
Irrigation Power Supply (24-Volt/750 mA Transformer)
INSTRUCTIONS:
A step-by-step setup video is available here: http://youtu.be/l4GPRTsuHkI
After assembling your arduino, radio, decoupling capacitors, shift register(s), status LED, pushbutton LCD (I2C connected to
A4 and A5) and relays, and load the sketch.
Following the instructions at https://MySensors.org include the device to your MySensors Gateway.
Verify that each new device has a Variable1, Variable2 and Variable3. Populate data accordingly with whole minutes for
the RUN_ALL_ZONES routine (Variable1) and the RUN_SINGLE_ZONE routines (Variable 2). The values entered for times may be zero and
you may use the defaulet zone names by leaving Variable3 blank.
Once you have entered values for each zone and each variable, save the settings by pressing the red save button on your Vera.
Restart your arduino; verify the settings are loaded into your arduino with the serial monitor; the array will be printed
on the serial monitor.
Your arduino should slow-flash, indicating that it is in ready mode.
There are multiple debug serial prints that can be monitored to assure that it is operating properly.
* ***THIS SHOULD NO LONGER BE NEEDED*** The standard MySensors library now works. https://bitbucket.org/fmalpartida/new-liquidcrystal/downloads for the I2C library, or use yours
Contributed by Jim (BulldogLowell@gmail.com) with much contribution from Pete (pete.will@mysensors.org) and is released to the public domain
Modified David Martin.
Changed date format to Australia standard dd/mm/yy
Added valve 8 as master valve during watering sequence - so 8 will be on whenever a valve/zone is active
Fixed initial valve read system to ensure all data read - via update of getValveTimes routine
Changed display details for VAR_3 to be consistent with other data reads
Updated various displays and debug outputs
*/
//
// Enable debug prints
//#define MY_DEBUG
// Enable node as Repeater
//#define MY_REPEATER_FEATURE
// Enable and select radio type attached
#define MY_RADIO_NRF24
#define MY_RF24_PA_LEVEL RF24_PA_LOW
#define MY_NODE_ID 2 // Set this to fix your Radio ID or use Auto
#include <Wire.h>
#include <TimeLib.h>
#include <SPI.h>
#include <MySensors.h>
#include <LiquidCrystal.h>
#include <LiquidCrystal_I2C.h>
#define NUMBER_OF_VALVES 6 // Change this to set your valve count up to 16.
#define VALVE_RESET_TIME 2500UL // Change this (in milliseconds) for the time you need your valves to hydraulically reset and change state
#define VALVE_TIMES_RELOAD 720000UL // Change this (in milliseconds) for how often to update all valves data from the controller (Loops at value/number valves)
// ie: 300000 for 5 valves produces requests every minute with all valves updated every 5mins
#define SKETCH_NAME "MySprinkler"
#define SKETCH_VERSION "2.2"
//
#define CHILD_ID_SPRINKLER 0
//
#define ACTIVE_LOW // comment out this line if your relays are active high
//
#define DEBUG_ON // comment out to supress serial monitor output
//
#ifdef ACTIVE_LOW
#define BITSHIFT_VALVE_NUMBER ~(1U << (valveNumber-1))
#define ALL_VALVES_OFF 0xFFFF
#else
#define BITSHIFT_VALVE_NUMBER (1U << (valveNumber-1))
#define ALL_VALVES_OFF 0U
#endif
//
#ifdef DEBUG_ON
#define DEBUG_PRINT(x) Serial.print(x)
#define DEBUG_PRINTLN(x) Serial.println(x)
#else
#define DEBUG_PRINT(x)
#define DEBUG_PRINTLN(x)
#define SERIAL_START(x)
#endif
//
typedef enum {
STAND_BY_ALL_OFF, RUN_SINGLE_ZONE, RUN_ALL_ZONES, CYCLE_COMPLETE, ZONE_SELECT_MENU
}
SprinklerStates;
//
SprinklerStates state = STAND_BY_ALL_OFF;
SprinklerStates lastState;
byte menuState = 0;
unsigned long menuTimer;
byte countDownTime = 10;
//
int allZoneTime [NUMBER_OF_VALVES + 1];
int valveSoloTime [NUMBER_OF_VALVES + 1];
int valveNumber;
int lastValve;
unsigned long startMillis;
const int ledPin = 5;
const int waterButtonPin = 3;
bool buttonPushed = false;
bool showTime = true;
bool clockUpdating = false;
bool recentUpdate = true;
int allVars[] = {V_VAR1, V_VAR2, V_VAR3};
const char *dayOfWeek[] = {
"Null", "Sunday ", "Monday ", "Tuesday ", "Wednesday ", "Thursday ", "Friday ", "Saturday "
};
// Name your Zones here or use Vera to edit them by adding a name in Variable3...
String valveNickName[17] = {
"All Zones", "Zone 1", "Zone 2", "Zone 3", "Zone 4", "Zone 5", "Zone 6", "Zone 7", "Zone 8", "Zone 9", "Zone 10", "Zone 11", "Zone 12", "Zone 13", "Zone 14", "Zone 15", "Zone 16"
};
//
time_t lastTimeRun = 0;
//Setup Shift Register...
const int latchPin = 8;
const int clockPin = 4;
const int dataPin = 7;
const int outputEnablePin = 6;
//
byte clock[8] = {0x0, 0xe, 0x15, 0x17, 0x11, 0xe, 0x0}; // fetching time indicator
byte raindrop[8] = {0x4, 0x4, 0xA, 0xA, 0x11, 0xE, 0x0,}; // fetching Valve Data indicator
// Set the pins on the I2C chip used for LCD connections:
// addr, en,rw,rs,d4,d5,d6,d7,bl,blpol
LiquidCrystal_I2C lcd(0x27, 2, 1, 0, 4, 5, 6, 7, 3, POSITIVE); // Set the LCD I2C address to 0x27
//
MyMessage msg1valve(CHILD_ID_SPRINKLER, V_LIGHT);
MyMessage var1valve(CHILD_ID_SPRINKLER, V_VAR1);
MyMessage var2valve(CHILD_ID_SPRINKLER, V_VAR2);
MyMessage var3valve(CHILD_ID_SPRINKLER, V_VAR3);
bool receivedInitialValue = false;
bool inSetup = true;
//
void setup()
{
DEBUG_PRINTLN(F("Initialising..."));
pinMode(latchPin, OUTPUT);
pinMode(clockPin, OUTPUT);
pinMode(dataPin, OUTPUT);
pinMode(ledPin, OUTPUT);
pinMode(outputEnablePin, OUTPUT);
digitalWrite (outputEnablePin, LOW);
pinMode(waterButtonPin, INPUT_PULLUP);
//pinMode(waterButtonPin, INPUT);
attachInterrupt(digitalPinToInterrupt(waterButtonPin), PushButton, RISING); //May need to change for your Arduino model
digitalWrite (ledPin, HIGH);
DEBUG_PRINTLN(F("Turning All Valves Off..."));
updateRelays(ALL_VALVES_OFF);
//delay(5000);
lcd.begin(16, 2); //(16 characters and 2 line display)
lcd.clear();
lcd.backlight();
lcd.createChar(0, clock);
lcd.createChar(1, raindrop);
//
//check for saved date in EEPROM
DEBUG_PRINTLN(F("Checking EEPROM for stored date:"));
delay(500);
if (loadState(0) == 0xFF) // EEPROM flag
{
DEBUG_PRINTLN(F("Retreiving last run time from EEPROM..."));
for (int i = 0; i < 4 ; i++)
{
lastTimeRun = lastTimeRun << 8;
lastTimeRun = lastTimeRun | loadState(i + 1); // assemble 4 bytes into an ussigned long epoch timestamp
}
}
DEBUG_PRINTLN(F("Sensor Presentation Complete"));
//
digitalWrite (ledPin, LOW);
DEBUG_PRINTLN(F("Ready..."));
//
lcd.setCursor(0, 0);
lcd.print(F(" Syncing Time "));
lcd.setCursor(15, 0);
lcd.write(byte(0));
lcd.setCursor(0, 1);
int clockCounter = 0;
while (timeStatus() == timeNotSet && clockCounter < 21)
{
requestTime();
DEBUG_PRINTLN(F("Requesting time from Gateway:"));
wait(1000);
lcd.print(".");
clockCounter++;
if (clockCounter > 16)
{
DEBUG_PRINTLN(F("Failed initial clock synchronization!"));
lcd.clear();
lcd.print(F(" Failed Clock "));
lcd.setCursor(0, 1);
lcd.print(F(" Syncronization "));
wait(2000);
break;
}
}
for (byte i = 1; i <= NUMBER_OF_VALVES; i++)
{
lcd.clear();
goGetValveTimes();
}
lcd.clear();
}
void presentation()
{
sendSketchInfo(SKETCH_NAME, SKETCH_VERSION);
for (byte i = 0; i <= NUMBER_OF_VALVES; i++)
{
present(i, S_LIGHT);
}
}
//
void loop()
{
inSetup = false;
updateClock();
updateDisplay();
goGetValveTimes();
//
if (buttonPushed)
{
menuTimer = millis();
DEBUG_PRINTLN(F("Button Pressed"));
if (state == STAND_BY_ALL_OFF)
{
state = ZONE_SELECT_MENU;
menuState = 0;
}
else if (state == ZONE_SELECT_MENU)
{
menuState++;
if (menuState > NUMBER_OF_VALVES)
{
menuState = 0;
}
}
else
{
state = STAND_BY_ALL_OFF;
}
buttonPushed = false;
}
if (state == STAND_BY_ALL_OFF)
{
slowToggleLED ();
if (state != lastState)
{
updateRelays(ALL_VALVES_OFF);
DEBUG_PRINTLN(F("State Changed... all Zones off"));
for (byte i = 0; i <= NUMBER_OF_VALVES; i++)
{
wait(50);
send(msg1valve.setSensor(i).set(false), false);
}
lcd.clear();
lcd.setCursor(0, 0);
lcd.print(F("** Irrigation **"));
lcd.setCursor(0, 1);
lcd.print(F("** Halted **"));
wait(2000);
lastValve = -1;
}
}
//
else if (state == RUN_ALL_ZONES)
{
if (lastValve != valveNumber)
{
for (byte i = 0; i <= NUMBER_OF_VALVES; i++)
{
if (i == 0 || i == valveNumber)
{
send(msg1valve.setSensor(i).set(true), false);
}
else
{
send(msg1valve.setSensor(i).set(false), false);
}
wait(50);
}
}
lastValve = valveNumber;
fastToggleLed();
if (state != lastState)
{
valveNumber = 1;
updateRelays(ALL_VALVES_OFF);
DEBUG_PRINTLN(F("State Changed, Running All Zones..."));
}
unsigned long nowMillis = millis();
if (nowMillis - startMillis < VALVE_RESET_TIME)
{
updateRelays(ALL_VALVES_OFF);
}
else if (nowMillis - startMillis < (allZoneTime[valveNumber] * 60000UL))
{
updateRelays(BITSHIFT_VALVE_NUMBER);
}
else
{
DEBUG_PRINTLN(F("Changing Valves..."));
updateRelays(ALL_VALVES_OFF);
startMillis = millis();
valveNumber++;
if (valveNumber > NUMBER_OF_VALVES)
{
state = CYCLE_COMPLETE;
startMillis = millis();
lastValve = -1;
lastTimeRun = now();
saveDateToEEPROM(lastTimeRun);
for (byte i = 0; i <= NUMBER_OF_VALVES; i++)
{
send(msg1valve.setSensor(i).set(false), false);
}
DEBUG_PRINT(F("State = "));
DEBUG_PRINTLN(state);
}
}
}
//
else if (state == RUN_SINGLE_ZONE)
{
fastToggleLed();
if (state != lastState)
{
for (byte i = 0; i <= NUMBER_OF_VALVES; i++)
{
if (i == 0 || i == valveNumber)
{
send(msg1valve.setSensor(i).set(true), false);
}
else
{
send(msg1valve.setSensor(i).set(false), false);
}
wait(50);
}
DEBUG_PRINTLN(F("State Changed, Single Zone Running..."));
DEBUG_PRINT(F("Zone: "));
DEBUG_PRINTLN(valveNumber);
}
unsigned long nowMillis = millis();
if (nowMillis - startMillis < VALVE_RESET_TIME)
{
updateRelays(ALL_VALVES_OFF);
}
else if (nowMillis - startMillis < (valveSoloTime [valveNumber] * 60000UL))
{
updateRelays(BITSHIFT_VALVE_NUMBER);
}
else
{
updateRelays(ALL_VALVES_OFF);
for (byte i = 0; i <= NUMBER_OF_VALVES; i++)
{
send(msg1valve.setSensor(i).set(false), false);
}
state = CYCLE_COMPLETE;
startMillis = millis();
DEBUG_PRINT(F("State = "));
DEBUG_PRINTLN(state);
}
lastTimeRun = now();
}
else if (state == CYCLE_COMPLETE)
{
if (millis() - startMillis < 30000UL)
{
fastToggleLed();
}
else
{
state = STAND_BY_ALL_OFF;
}
}
if (state == ZONE_SELECT_MENU)
{
displayMenu();
}
else
{
lastState = state;
}
}
//
void displayMenu(void)
{
static byte lastMenuState = -1;
static int lastSecond;
if (menuState != lastMenuState)
{
lcd.clear();
lcd.setCursor(0, 0);
lcd.print(valveNickName[menuState]);
lcd.setCursor(0, 1);
lcd.print(F("Starting"));
DEBUG_PRINT(valveNickName[menuState]);
Serial.print(F(" Starting Shortly"));
}
int thisSecond = (millis() - menuTimer) / 1000UL;
if (thisSecond != lastSecond && thisSecond < 8)
{
lcd.print(F("."));
Serial.print(".");
}
lastSecond = thisSecond;
if (millis() - menuTimer > 10000UL)
{
startMillis = millis();
if (menuState == 0)
{
valveNumber = 1;
state = RUN_ALL_ZONES;
}
else
{
valveNumber = menuState;
state = RUN_SINGLE_ZONE;
}
}
else
{
}
lastMenuState = menuState;
}
//
void updateRelays(int value)
{
digitalWrite(latchPin, LOW);
if (value == -1) {
shiftOut(dataPin, clockPin, MSBFIRST, highByte(value));
shiftOut(dataPin, clockPin, MSBFIRST, lowByte(value));
} else {
// Valve 8 is master and must be on to turn on water supply -128 for 8
shiftOut(dataPin, clockPin, MSBFIRST, highByte(value - 128));
shiftOut(dataPin, clockPin, MSBFIRST, lowByte(value - 128));
}
digitalWrite(latchPin, HIGH);
}
//
void PushButton() //interrupt with debounce
{
static unsigned long last_interrupt_time = 0;
unsigned long interrupt_time = millis();
if (interrupt_time - last_interrupt_time > 200)
{
buttonPushed = true;
}
last_interrupt_time = interrupt_time;
}
//
void fastToggleLed()
{
static unsigned long fastLedTimer;
if (millis() - fastLedTimer >= 100UL)
{
digitalWrite(ledPin, !digitalRead(ledPin));
fastLedTimer = millis ();
}
}
//
void slowToggleLED ()
{
static unsigned long slowLedTimer;
if (millis() - slowLedTimer >= 20000UL)
{
digitalWrite(ledPin, !digitalRead(ledPin));
if (!digitalRead(ledPin)) {
slowLedTimer = millis();
} else {
slowLedTimer = millis() - 19000UL;
}
}
}
//
void receive(const MyMessage &message)
{
bool zoneTimeUpdate = false;
if (message.isAck())
{
DEBUG_PRINTLN(F("This is an ack from gateway"));
}
for (byte i = 0; i <= NUMBER_OF_VALVES; i++)
{
if (message.sensor == i)
{
if (message.type == V_LIGHT)
{
int switchState = atoi(message.data);
if (switchState == 0)
{
state = STAND_BY_ALL_OFF;
DEBUG_PRINTLN(F("Recieved Instruction to Cancel..."));
}
else
{
if (i == 0)
{
state = RUN_ALL_ZONES;
valveNumber = 1;
DEBUG_PRINTLN(F("Recieved Instruction to Run All Zones..."));
}
else
{
state = RUN_SINGLE_ZONE;
valveNumber = i;
DEBUG_PRINT(F("Recieved Instruction to Activate Zone: "));
DEBUG_PRINTLN(i);
}
}
startMillis = millis();
}
else if (message.type == V_VAR1)
{
int variable1 = atoi(message.data);// RUN_ALL_ZONES time
DEBUG_PRINT(F("Recieved variable1 valve:"));
DEBUG_PRINT(i);
DEBUG_PRINT(F(" = "));
DEBUG_PRINTLN(variable1);
if (variable1 != allZoneTime[i])
{
allZoneTime[i] = variable1;
zoneTimeUpdate = true;
}
receivedInitialValue = true;
}
else if (message.type == V_VAR2)
{
int variable2 = atoi(message.data);// RUN_SINGLE_ZONE time
DEBUG_PRINT(F("Recieved variable2 valve:"));
DEBUG_PRINT(i);
DEBUG_PRINT(F(" = "));
DEBUG_PRINTLN(variable2);
if (variable2 != valveSoloTime[i])
{
valveSoloTime[i] = variable2;
zoneTimeUpdate = true;
}
receivedInitialValue = true;
}
else if (message.type == V_VAR3)
{
String newMessage = String(message.data);
if (newMessage.length() == 0)
{
DEBUG_PRINT(F("No Name Recieved for zone "));
DEBUG_PRINTLN(i);
break;
}
if (newMessage.length() > 16)
{
newMessage.substring(0, 16);
}
valveNickName[i] = "";
valveNickName[i] += newMessage;
DEBUG_PRINT(F("Recieved variable3 valve: "));
DEBUG_PRINT(i);
DEBUG_PRINT(F(" = "));
DEBUG_PRINTLN(valveNickName[i]);
}
receivedInitialValue = true;
}
}
if (zoneTimeUpdate)
{
//
DEBUG_PRINTLN(F("New Zone Times Recieved..."));
for (byte i = 0; i <= NUMBER_OF_VALVES; i++)
{
if (i != 0)
{
DEBUG_PRINT(F("Zone "));
DEBUG_PRINT(i);
DEBUG_PRINT(F(" individual time: "));
DEBUG_PRINT(valveSoloTime[i]);
DEBUG_PRINT(F(" group time: "));
DEBUG_PRINT(allZoneTime[i]);
DEBUG_PRINT(F(" name: "));
DEBUG_PRINTLN(valveNickName[i]);
recentUpdate = true;
}
}
}
else
{
recentUpdate = false;
}
}
//
void updateDisplay()
{
static unsigned long lastUpdateTime;
static bool displayToggle = false;
//static byte toggleCounter = 0;
static SprinklerStates lastDisplayState;
if (state != lastDisplayState || millis() - lastUpdateTime >= 3000UL)
{
displayToggle = !displayToggle;
switch (state) {
case STAND_BY_ALL_OFF:
//
fastClear();
lcd.setCursor(0, 0);
if (displayToggle)
{
lcd.print(F(" System Ready "));
if (clockUpdating)
{
lcd.setCursor(15, 0);
lcd.write(byte(0));
}
lcd.setCursor(0, 1);
lcd.print(hourFormat12() < 10 ? F(" ") : F(""));
lcd.print(hourFormat12());
lcd.print(minute() < 10 ? F(":0") : F(":"));
lcd.print(minute());
lcd.print(isAM() ? F("am") : F("pm"));
lcd.print(day() < 10 ? F(" 0") : F(" "));
lcd.print(day());
lcd.print(month() < 10 ? F("/0") : F("/"));
lcd.print(month());
lcd.print(F("/"));
lcd.print(year() % 100);
}
else
{
lcd.print(F(" Last Watered "));
if (clockUpdating)
{
lcd.setCursor(15, 0);
lcd.write(byte(0));
}
lcd.setCursor(0, 1);
lcd.print(dayOfWeek[weekday(lastTimeRun)]);
lcd.setCursor(11, 1);
lcd.print(day(lastTimeRun) < 10 ? F("0") : F(""));
lcd.print(day(lastTimeRun));
lcd.print(month(lastTimeRun) < 10 ? F("/0") : F("/"));
lcd.print(month(lastTimeRun));
}
break;
case RUN_SINGLE_ZONE:
//
fastClear();
lcd.setCursor(0, 0);
if (displayToggle)
{
lcd.print(F("Single Zone Mode"));
lcd.setCursor(0, 1);
lcd.print(F(" Zone:"));
if (valveNumber < 10) lcd.print(F("0"));
lcd.print(valveNumber);
lcd.print(F(" Active"));
}
else
{
lcd.print(F(" Time Remaining "));
lcd.setCursor(0, 1);
if (valveSoloTime[valveNumber] == 0)
{
lcd.print(F(" No Valve Time "));
}
else
{
unsigned long timeRemaining = (valveSoloTime[valveNumber] * 60) - ((millis() - startMillis) / 1000);
lcd.print(timeRemaining / 60 < 10 ? " 0" : " ");
lcd.print(timeRemaining / 60);
lcd.print("min");
lcd.print(timeRemaining % 60 < 10 ? " 0" : " ");
lcd.print(timeRemaining % 60);
lcd.print("sec ");
}
}
break;
case RUN_ALL_ZONES:
//
fastClear();
lcd.setCursor(0, 0);
if (displayToggle)
{
lcd.print(F(" All-Zone Mode "));
lcd.setCursor(0, 1);
lcd.print(F(" Zone:"));
if (valveNumber < 10) lcd.print(F("0"));
lcd.print(valveNumber);
lcd.print(F(" Active "));
}
else
{
lcd.print(F(" Time Remaining "));
lcd.setCursor(0, 1);
int timeRemaining = (allZoneTime[valveNumber] * 60) - ((millis() - startMillis) / 1000);
lcd.print((timeRemaining / 60) < 10 ? " 0" : " ");
lcd.print(timeRemaining / 60);
lcd.print("min");
lcd.print(timeRemaining % 60 < 10 ? " 0" : " ");
lcd.print(timeRemaining % 60);
lcd.print("sec ");
}
break;
case CYCLE_COMPLETE:
//
if (displayToggle)
{
lcd.setCursor(0, 0);
lcd.print(F(" Watering Cycle "));
lcd.setCursor(0, 1);
lcd.print(F(" Complete "));
}
else
{
int totalTimeRan = 0;
for (int i = 1; i < NUMBER_OF_VALVES + 1; i++)
{
totalTimeRan += allZoneTime[i];
}
lcd.setCursor(0, 0);
lcd.print(F(" Total Time Run "));
lcd.setCursor(0, 1);
lcd.print(totalTimeRan < 10 ? " 0" : " ");
lcd.print(totalTimeRan);
lcd.print(" Minutes ");
}
default:
// what of ZONE_SELECT_MENU?
break;
}
lastUpdateTime = millis();
}
lastDisplayState = state;
}
void receiveTime(time_t newTime)
{
DEBUG_PRINTLN(F("Time value received and updated..."));
int lastSecond = second();
int lastMinute = minute();
int lastHour = hour();
setTime(newTime);
if (((second() != lastSecond) || (minute() != lastMinute) || (hour() != lastHour)) || showTime)
{
DEBUG_PRINTLN(F("Clock updated...."));
DEBUG_PRINT(F("Sensor's time currently set to:"));
DEBUG_PRINT(hourFormat12() < 10 ? F(" 0") : F(" "));
DEBUG_PRINT(hourFormat12());
DEBUG_PRINT(minute() < 10 ? F(":0") : F(":"));
DEBUG_PRINT(minute());
DEBUG_PRINTLN(isAM() ? F("am") : F("pm"));
DEBUG_PRINT(month());
DEBUG_PRINT(F("/"));
DEBUG_PRINT(day());
DEBUG_PRINT(F("/"));
DEBUG_PRINTLN(year());
DEBUG_PRINTLN(dayOfWeek[weekday()]);
showTime = false;
}
else
{
DEBUG_PRINTLN(F("Sensor's time did NOT need adjustment greater than 1 second."));
}
clockUpdating = false;
}
void fastClear()
{
lcd.setCursor(0, 0);
lcd.print(F(" "));
lcd.setCursor(0, 1);
lcd.print(F(" "));
}
//
void updateClock()
{
static unsigned long lastVeraGetTime;
if (millis() - lastVeraGetTime >= 3600000UL) // updates clock time and gets zone times from vera once every hour
{
DEBUG_PRINTLN(F("Requesting time and valve data from Gateway..."));
lcd.setCursor(15, 0);
lcd.write(byte(0));
clockUpdating = true;
requestTime();
lastVeraGetTime = millis();
}
}
//
void saveDateToEEPROM(unsigned long theDate)
{
DEBUG_PRINTLN(F("Saving Last Run date"));
if (loadState(0) != 0xFF)
{
saveState(0, 0xFF); // EEPROM flag for last date saved stored in EEPROM (location zero)
}
//
for (int i = 1; i < 5; i++)
{
saveState(5 - i, byte(theDate >> 8 * (i - 1))); // store epoch datestamp in 4 bytes of EEPROM starting in location one
}
}
//
void goGetValveTimes()
{
static unsigned long valveUpdateTime;
static byte valveIndex = 1;
if (inSetup || millis() - valveUpdateTime >= VALVE_TIMES_RELOAD / NUMBER_OF_VALVES) // update each valve once every 5 mins (distributes the traffic)
{
if (inSetup) {
lcd.print(F(" Updating "));
lcd.setCursor(0, 1);
lcd.print(F(" Valve Data: "));
lcd.print(valveIndex);
}
bool flashIcon = false;
DEBUG_PRINT(F("Calling for Valve "));
DEBUG_PRINT(valveIndex);
DEBUG_PRINTLN(F(" Data..."));
for (int a = 0; a < (sizeof(allVars)/sizeof(int)); a++) {
receivedInitialValue = false;
byte timeout = 10;
while (!receivedInitialValue && timeout > 0)
{
lcd.setCursor(15, 0);
flashIcon = !flashIcon;
flashIcon ? lcd.write(byte(1)) : lcd.print(F(" "));
request(valveIndex, allVars[a]);
wait(50);
timeout--;
}
}
valveUpdateTime = millis();
valveIndex++;
if (valveIndex > NUMBER_OF_VALVES)
{
valveIndex = 1;
}
}
}
@tekka Loaded onto my MEGA based Ethernet gateway connected to openHab using a E01-ML01DP5 with 9db antenna - this previously lasted less than 6 hours. Will advise as testing goes ahead. I do not send much with this unit more receive but have some test nodes I will code to toggle back and forth - I am unable to add my signed nodes to this as yet!
I will also grab my MEGA based repeater on a different network and see what happens this is a nrf24l01+ with pa + lna.
I give up - at least if someone is thinking of building this gateway and wanting to use security they can hopefully read this thread before they start and will look at a higher level controller before they purchase the hardware. I am only trying to help others with what to me is still a simple and easy fix alas I must have upset the apple cart.
There is no big problem at all, just that it is really simple to know right now, with technology available today, if you need an Ethernet gateway and signing then use a Mega 2560 or better for the Ethernet Gateway controller to ensure you have the ram. @mfalkvidd - You state variations increase exponentially and most not under library control however using the MySensors Library and building the hardware shown and documented on this page using the sketch provided (completely under control of MySensors) you will need a Mega 2560 or better if you intend to enable signing (also under control of the Library). Do not get me wrong I really love the whole MySensors thing and have over 30 devices up and running and have enjoyed the whole learning thing however simple requirements (IMHO) like this should be detailed.
Hopefully this post will help the next person, it certainly would have help me and I am sure I am not the only one.
@mfalkvidd My thoughts are change the Buying Guide and related text to the BME it is a much better sensor with Humidity included and is only slightly more $. I have ordered some BME's and will be testing the provided sketch in due course.
@Woodside the code in this post is the latest with one exception. I am not using the internal pullup on the waterbuttonPin any more as I have installed a cap/resistor mode to facilitate a hardware debouce. The details of this have been published in the Rain Gauge project (see troubleshooting). Apart from that the above code is fully made for AU dates and is running my system now.
The GitHub version is US date based and not AU converted but has my mods for the gateway comms and some updates - https://github.com/mysensors/MySensorsArduinoExamples/blob/master/examples/IrrigationController/IrrigationController.ino
If you run into any problems or issues let me know.
@TimO thanks for the pointers. All my nodes were working ok but via a lot of playing around I found a repeater node that also runs a light dimmer (which the light dimmer worked fine and so did all of the repeating functions) was constantly sending an ID request. I disabled the repeater function and flashed it back and it is now not sending these requests. I then deployed another (better) repeater node into the same area and did not catch anymore ID requests so it would appear the repeater was causing the issue. I did flash it again with repeater on but it went bad again so gave that away and now have a different repeater serving that part of the property and no more weird ID requests. Thanks again for your help and continuing work on the binding.
PS: Is there somewhere that we can always get the latest JAR files from, I find I have to troll posts to find links?
@TimO There is no given_ids.cached file - there is a given_ids_mysensors_bridge-eth_gateway.cached file and it contains:
[0,1,2,3,4,5,6,7,8,9,10,11,12,13,14,15,16,17,18,19,20,21,22,23,24,25,26,27,28,29,30,31,32,33,34,35,36,37,38,39,40,41,42,43,44,45,46,47,48,49,50,51,52,53,54,55,56,57,58,59,60,61,62,63,64,65,66,67,68,69,70,71,72,73,74,75,76,77,78,79,80,81,82,83,84,85,86,87,88,89,90,91,92,93,94,95,96,97,98,99,100,101,102,103,104,105,106,107,108,109,110,111,112,113,114,115,116,117,118,119,120,121,122,123,124,125,126,127,128,129,130,131,132,133,134,135,136,137,138,139,140,141,142,143,144,145,146,147,148,149,150,151,152,153,154,155,156,157,158,159,160,161,162,163,164,165,166,167,168,169,170,171,172,173,174,175,176,177,178,179,180,181,182,183,184,185,186,187,188,189,190,191,192,193,194,195,196,197,198,199,200,201,202,203,204,205,206,207,208,209,210,211,212,213,214,215,216,217,218,219,220,221,222,223,224,225,226,227,228,229,230,231,232,233,234,235,236,237,238,239,240,241,242,243,244,245,246,247,248,249,250,251,252,253,254]
I have fixed ID on all my nodes and have not changed anything for a long time.
I have just upgraded my Docker based RPI Openhab to 2.5.6 and updated the jar to org.openhab.binding.mysensors-2.5.0-SNAPSHOT.jar
Since I did this and tail the openhab log file I get this:
2020-07-28 16:24:38.576 [DEBUG] [rsAbstractConnection$MySensorsReader] - Message from gateway received: 255;255;3;0;3;0
2020-07-28 16:24:38.580 [INFO ] [rs.internal.gateway.MySensorsGateway] - ID Request received
2020-07-28 16:24:38.588 [ERROR] [rs.internal.gateway.MySensorsGateway] - No more IDs available for this node, you could try cleaning cache file
My Node:3 is a multi function, Humidity, Temp, Light, Rain and Barometric sensor. I believe 255 is used for some other comms as per the docs it is not recommended to be used for nodes. So why this being constantly logged, do I need to add a dummy node 255 or something?
Oh and I have tried doing what it says and cleaning the cache file, I deleted both files in the userdata/logs/mysensors/cache and restarted my docker container.
@mfalkvidd My thoughts are change the Buying Guide and related text to the BME it is a much better sensor with Humidity included and is only slightly more $. I have ordered some BME's and will be testing the provided sketch in due course.
@mfalkvidd Yes seems right but the most confusing part and the one I stumbled on is that on the page https://www.mysensors.org/build/pressure for the sensor the Shopping Guide points you at the BMP280 which as we are identifying is not the relevant item for the project - I followed the provided link and purchased the sensor (after waiting for weeks for delivery) only to find it does not work at all (as I suspect my be the case for @Andrew-Maynard as the sensors does not init and the loop does not run) and hence my code was needed to run the node using the item shown in the shopping guide.
I could not make the example BME280 sketch work either and upon investigation the code is not for the standard BME280 it is for the BME280 library by Embedded Adventures - https://github.com/embeddedadventures/BME280. This device is not the same as the cheaper standard BME280 as linked on the forum page. The Embedded Adventure one has humidity, the standard BME280 does not.
Anyway I researched and converted some other BME280 code I found from the SFE_BMP180 library, coupled the Forecast code and created the following sketch for the BME280. You will need to install the SFE_BMP180.h library using library manager
/**
* 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
*
*/
// Enable debug prints
#define MY_DEBUG
// Enable and select radio type attached
#define MY_RADIO_NRF24
// #define MY_RF24_CHANNEL 82
// #define MY_NODE_ID 22 //uncomment this line to assign a static ID
#include <MySensors.h>
#define CHILD_ID_BARO 0
#define CHILD_ID_TEMP 1
static bool metric = true;
// Sleep time between sensor updates (in milliseconds)
unsigned long UPDATE_INTERVAL = 60000;
#define ALTITUDE 28.0 // Altitude in meters
#define GENERATE_FORECAST
#define CONVERSION_FACTOR (1.0/10.0) // used by forecast algorithm to convert from Pa to kPa, by dividing hPa by 10.
#ifdef GENERATE_FORECAST // Below you will find a lot of variables used by the forecast algorithm.
const char *weather[] = { "stable", "sunny", "cloudy", "unstable", "thunderstorm", "unknown" };
enum FORECAST
{
STABLE = 0, // "Stable Weather Pattern"
SUNNY = 1, // "Slowly rising Good Weather", "Clear/Sunny "
CLOUDY = 2, // "Slowly falling L-Pressure ", "Cloudy/Rain "
UNSTABLE = 3, // "Quickly rising H-Press", "Not Stable"
THUNDERSTORM = 4, // "Quickly falling L-Press", "Thunderstorm"
UNKNOWN = 5 // "Unknown (More Time needed)
};
int lastForecast = -1; // Stores the previous forecast, so it can be compared with a new forecast.
const int LAST_SAMPLES_COUNT = 5;
float lastPressureSamples[LAST_SAMPLES_COUNT];
int minuteCount = 0; // Helps the forecst algorithm keep time.
bool firstRound = true; // Helps the forecast algorithm recognise if the sensor has just been powered up.
float pressureAvg; // Average value is used in forecast algorithm.
float pressureAvg2; // Average after 2 hours is used as reference value for the next iteration.
float dP_dt; // Pressure delta over time
#endif
#include <SFE_BMP180.h>
static SFE_BMP180 pressure;
static MyMessage msgBaro(CHILD_ID_BARO, V_PRESSURE );
static MyMessage msgTemp(CHILD_ID_TEMP, V_TEMP);
#ifdef GENERATE_FORECAST
MyMessage msgForecast(CHILD_ID_BARO, V_FORECAST);
#endif
void presentation()
{
// Send the sketch info to the gateway
sendSketchInfo("TemperatureAndPressure", "1.0");
// Present sensors as children to gateway
present(CHILD_ID_BARO, S_BARO, "Pressure");
wait(50);
present(CHILD_ID_TEMP, S_TEMP, "Temperature");
wait(50);
metric = getControllerConfig().isMetric;
}
void setup()
{
pressure.begin();
}
void loop()
{
char status;
double T,P,p0,a;
unsigned long currentMillis = millis();
status = pressure.startTemperature();
if (status != 0) { wait(status); }
status = pressure.getTemperature(T);
if (status != 0)
{
// Print out the measurement:
Serial.print("temperature: ");
Serial.print(T,2);
Serial.print(" deg C, ");
Serial.print((9.0/5.0)*T+32.0,2);
Serial.println(" deg F");
send(msgTemp.set(T, 2));
}
// Start a pressure measurement:
// The parameter is the oversampling setting, from 0 to 3 (highest res, longest wait).
// If request is successful, the number of ms to wait is returned.
// If request is unsuccessful, 0 is returned.
status = pressure.startPressure(3);
if (status != 0) { wait(status); }
status = pressure.getPressure(P,T);
if (status != 0)
{
// Print out the measurement:
Serial.print("absolute pressure: ");
Serial.print(P,2);
Serial.print(" mb, ");
Serial.print(P*0.0295333727,2);
Serial.println(" inHg");
send(msgBaro.set(P, 2));
}
#ifdef GENERATE_FORECAST
int forecast = sample(P); // Run the forecast function with a new pressure update.
// Send forecast
if (forecast != lastForecast) {
Serial.println("BME280 - Sending the latest forecast to the gateway.");
send(msgForecast.set(weather[forecast]));
lastForecast = forecast;
}
#endif
// Sleep until next update to save energy but check for how long to sleep based on delays in getting data
unsigned long sleeptime = UPDATE_INTERVAL - (millis() - currentMillis); // How much time has passed already during the calculating? Subtract that from the intended interval time.
wait(sleeptime);
}
#ifdef GENERATE_FORECAST
// These functions are only included if the forecast function is enables. The are used to generate a weater prediction by checking if the barometric pressure is rising or falling over time.
float getLastPressureSamplesAverage()
{
float lastPressureSamplesAverage = 0;
for (int i = 0; i < LAST_SAMPLES_COUNT; i++) {
lastPressureSamplesAverage += lastPressureSamples[i];
}
lastPressureSamplesAverage /= LAST_SAMPLES_COUNT;
return lastPressureSamplesAverage;
}
// Forecast algorithm found here
// http://www.freescale.com/files/sensors/doc/app_note/AN3914.pdf
// Pressure in hPa --> forecast done by calculating kPa/h
int sample(float pressure) {
// Calculate the average of the last n minutes.
int index = minuteCount % LAST_SAMPLES_COUNT;
lastPressureSamples[index] = pressure;
minuteCount++;
if (minuteCount > 185) {
minuteCount = 6;
}
if (minuteCount == 5) {
pressureAvg = getLastPressureSamplesAverage();
}
else if (minuteCount == 35) {
float lastPressureAvg = getLastPressureSamplesAverage();
float change = (lastPressureAvg - pressureAvg) * CONVERSION_FACTOR;
if (firstRound) { // first time initial 3 hour
dP_dt = change * 2; // note this is for t = 0.5hour
}
else {
dP_dt = change / 1.5; // divide by 1.5 as this is the difference in time from 0 value.
}
}
else if (minuteCount == 65) {
float lastPressureAvg = getLastPressureSamplesAverage();
float change = (lastPressureAvg - pressureAvg) * CONVERSION_FACTOR;
if (firstRound) { //first time initial 3 hour
dP_dt = change; //note this is for t = 1 hour
}
else {
dP_dt = change / 2; //divide by 2 as this is the difference in time from 0 value
}
}
else if (minuteCount == 95) {
float lastPressureAvg = getLastPressureSamplesAverage();
float change = (lastPressureAvg - pressureAvg) * CONVERSION_FACTOR;
if (firstRound) { // first time initial 3 hour
dP_dt = change / 1.5; // note this is for t = 1.5 hour
}
else {
dP_dt = change / 2.5; // divide by 2.5 as this is the difference in time from 0 value
}
}
else if (minuteCount == 125) {
float lastPressureAvg = getLastPressureSamplesAverage();
pressureAvg2 = lastPressureAvg; // store for later use.
float change = (lastPressureAvg - pressureAvg) * CONVERSION_FACTOR;
if (firstRound) { // first time initial 3 hour
dP_dt = change / 2; // note this is for t = 2 hour
}
else {
dP_dt = change / 3; // divide by 3 as this is the difference in time from 0 value
}
}
else if (minuteCount == 155) {
float lastPressureAvg = getLastPressureSamplesAverage();
float change = (lastPressureAvg - pressureAvg) * CONVERSION_FACTOR;
if (firstRound) { // first time initial 3 hour
dP_dt = change / 2.5; // note this is for t = 2.5 hour
}
else {
dP_dt = change / 3.5; // divide by 3.5 as this is the difference in time from 0 value
}
}
else if (minuteCount == 185) {
float lastPressureAvg = getLastPressureSamplesAverage();
float change = (lastPressureAvg - pressureAvg) * CONVERSION_FACTOR;
if (firstRound) { // first time initial 3 hour
dP_dt = change / 3; // note this is for t = 3 hour
}
else {
dP_dt = change / 4; // divide by 4 as this is the difference in time from 0 value
}
pressureAvg = pressureAvg2; // Equating the pressure at 0 to the pressure at 2 hour after 3 hours have past.
firstRound = false; // flag to let you know that this is on the past 3 hour mark. Initialized to 0 outside main loop.
}
int forecast = UNKNOWN;
if (minuteCount < 35 && firstRound) { //if time is less than 35 min on the first 3 hour interval.
forecast = UNKNOWN;
}
else if (dP_dt < (-0.25)) {
forecast = THUNDERSTORM;
}
else if (dP_dt > 0.25) {
forecast = UNSTABLE;
}
else if ((dP_dt > (-0.25)) && (dP_dt < (-0.05))) {
forecast = CLOUDY;
}
else if ((dP_dt > 0.05) && (dP_dt < 0.25))
{
forecast = SUNNY;
}
else if ((dP_dt >(-0.05)) && (dP_dt < 0.05)) {
forecast = STABLE;
}
else {
forecast = UNKNOWN;
}
// uncomment when debugging
//Serial.print(F("BME280 - Forecast at minute "));
//Serial.print(minuteCount);
//Serial.print(F(" dP/dt = "));
//Serial.print(dP_dt);
//Serial.print(F("kPa/h --> "));
//Serial.println(weather[forecast]);
return forecast;
}
#endif
I am investigating a problem I have found with I think openHAB but I am not sure if this issue is not related to the Gateway code of 2.3.0.
When a node performs a requestTime() I am seeing this reported:
2018-10-09 10:47:13.337 [INFO ] [rs.internal.gateway.MySensorsGateway] - I_TIME request received from 2, answering...
Looks perfect and the node receives the time however I was curious as to why sometimes the node would receive GMT and at other times my local timezone GMT+8. I decided to run the gateway via MYSContoller so I could monitor and log the traffic and I have found the problem. When the Gateway receives the request it responds twice within a few ms of each other, for what reason I do not know but the first response send the local time zone response (GMT+8) and the second is a GMT response. The logs from MYSController for the above report are as follows:
318857 9/10/2018 10:47:19 RX 2 - Irrigation Ctrl (2) INTERNAL C_INTERNAL NO I_TIME
318858 9/10/2018 10:47:19 TX 2 - Irrigation Ctrl (2) INTERNAL C_INTERNAL NO I_TIME 1539082039
318859 9/10/2018 10:47:19 TX 2 - Irrigation Ctrl (2) INTERNAL C_INTERNAL NO I_TIME "1539053233
"
Note the quotes where picked up in the copy and paste (I did this several times) so it looks like the body was changed to a string type but I also caught this in the MYSController Debug:
9/10/2018 10:47:19 RX 2;255;3;0;1;
9/10/2018 10:47:19 INFO Reply time request
9/10/2018 10:47:19 TX 2;255;3;0;1;1539082039
9/10/2018 10:47:19 FWD 2;255;3;0;1;1539053233
9/10/2018 10:47:19 TX 2;255;3;0;1;1539053233
So I am curious as to where the FWD comes from and why the response body has changed between the TX, FWD & TX?
I also use the lookup in PaperUI to be sure but it looks like you have added the thing in PaperUI from the inbox. What I would do is delete from PaperUI both nodes. If it is as you say setup in the .things files correctly you will not be able to delete it from PaperUI it will report a problem with any things that are setup in the manual things files.