nRf24L01+ connection quality meter

Oitzu

@AWI there also 2 versions of the PA/LNA Modules out there. Which one do you tested?

Yveaux

@AWI said:

The standard radio's I have tested with a silk screen (white lines) and pin designation print on the back seem to be fine

In the picture, these radios have the notorious 1242AF datecode on the chip (if you didn't take the pictures yourself, do you have the same date code?). I also have a batch of these and they perform very well!

drock1985

Thanks @awi for this, might have to build one myself to test a potential bad batch of radios i have.

Do you need the LCD for this or is the data also print in serial?

Thanks,

AWI

@drock1985 It will print the data on serial but not very well formatted. You don't need the LCD but it is easy to carry around the house. (a laptop too :smiley: but be aware of disturbances coming from the laptop i.e. Wifi ) I thought off blinking led's also. You can easily change the sketch, there is not much to it..

AWI

@Oitzu which versions do you relate to? I also have some with ceramic antenna (smd connection)

boozz

@AWI

Kijken je buren niet gek op wanneer je met je meter rondloopt? :wink:

gr.

Boozz

AWI

@boozz my neighbors ask me every two weeks what I am doing and what all those strange looking things in my garden and on my roof are... :flushed:

boozz

@AWI

And when you start explaining they look somewhat odd don't they?

mrc-core

I'm having a problem in this part off the code:
"present(counterChild, S_DIMMER, "Quality counter " NODE_TXT) ; // counter uses percentage from dimmer value"

I'm getting this error:
"exit status 1
'present' was not declared in this scope"

How can i fix this ?

mfalkvidd

@mrc-core sounds like you are using the stable (1.5.x) version of MySensors. This sketch requires the development version which is available on github

mrc-core

Thanks for the quick reply.
i'm going to change the versions :)

bjacobse

Can this be extended to also measure NRF24l01 standby current? Maybe some "fake" modules have a high current consumption in stadnby mode, this could drain batteries quickly when used in battery operated devices

cjrpriest

Hi @AWI, could you kindly post your updated version of the sketch that has the parent and destination node numbers on the display?

AWI

@cjrpriest Here you go..

/*
 PROJECT: MySensors / Quality of radio transmission 
 PROGRAMMER: AWI (MySensors libraries)
 DATE: 20160529/ last update: 20160530
 FILE: AWI_Send.ino
 LICENSE: Public domain

 Hardware: ATMega328p board w/ NRF24l01
	and MySensors 2.0
	
Special:
	
	
Summary:
	Sends a radio message with counter each  x time to determine fault ratio with receiver
Remarks:
	Fixed node-id & communication channel to other fixed node
	
Change log:
20160530 - added moving average on fail/ miss count, update to 2.0
*/


//****  MySensors *****
// Enable debug prints to serial monitor
#define MY_DEBUG 
#define MY_RADIO_NRF24									// Enable and select radio type attached
//#define MY_RF24_CHANNEL 80								// radio channel, default = 76

#define MY_NODE_ID 250
#define NODE_TXT "Q 250"								// Text to add to sensor name

//#define MY_PARENT_NODE_ID 42							// fixed parent to controller when 0 (else comment out = AUTO)


// #define MY_RF24_CE_PIN 7								// Ceech board, 3.3v (7,8)  (pin default 9,10)
// #define MY_RF24_CS_PIN 8
#define DESTINATION_NODE 0								// receiving fixed node id (default 0 = gateway)

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

// display
#include <Wire.h>											// I2C
#include <LiquidCrystal_I2C.h>								// LCD display with I2C interface


// helpers
#define LOCAL_DEBUG

#ifdef LOCAL_DEBUG
#define Sprint(a) (Serial.print(a))						// macro as substitute for print, enable if no print wanted
#define Sprintln(a) (Serial.println(a))					// macro as substitute for println
#else
#define Sprint(a)										// enable if no print wanted -or- 
#define Sprintln(a)										// enable if no print wanted
#endif


// MySensors sensor
#define counterChild 0

// send constants and variables
int messageCounter = 0 ; 
const int messageCounterMax = 100 ; 					// maximum message counter value 
const unsigned counterUpdateDelay = 500 ;				// send every x ms and sleep in between

// receive constants and variables
boolean failStore[messageCounterMax] ;					// moving average stores & pointers
int failStorePointer = 0 ;
boolean missedStore[messageCounterMax] ;
int missedStorePointer = 0 ;
int newMessage = 0 ;
int lastMessage = -1 ;
int missedMessageCounter = 0 ; 							// total number of messages in range (messageCounterMax)
int failMessageCounter = 0 ; 							// total number of messages in range (messageCounterMax)
uint8_t parent = 0 ;									// parent node-id 

// Loop delays
const unsigned long displayInterval = 1000UL ;			// display update in ms
unsigned long lastDisplayUpdate = 0 ;					// last update for loop timers

// standard messages
MyMessage counterMsg(counterChild, V_PERCENTAGE);		// Send value

// ***** LCD
//LiquidCrystal_I2C lcd(0x27, 2, 1, 0, 4, 5, 6, 7, 3, POSITIVE);  // Set the LCD I2C address
LiquidCrystal_I2C lcd(0x27, 16, 2);  // Set the LCD I2C address

void setup() {
	Wire.begin();  // I2C
    // ** LCD display **
    // LCD 2 lines * 16 char.
    lcd.begin(16, 2);
    lcd.setBacklight(HIGH);
    lcd.setCursor(0, 0);
	lcd.print("AWI Quality nRF24");

	for(int i= 0 ; i <  messageCounterMax ; i++){		// init stores for moving averages
		failStore[i] = true ;
		missedStore[i] = true ;
 	}
	missedStorePointer = failStorePointer = 0 ;
	delay(1000);
}

void presentation(){
// MySensors
	present(counterChild, S_DIMMER, "Quality counter " NODE_TXT) ;  // counter uses percentage from dimmer value
}


void loop() {
	// Sprint("count:") ; Sprintln(messageCounter) ;
	LCD_local_display();
	missedStore[failStorePointer] = false  ; 			// set slot to false (ack message needs to set) ; 
	boolean succes = failStore[failStorePointer] = send(counterMsg.setDestination(DESTINATION_NODE).set(failStorePointer), true);  // send to destination with ack
	if (!succes){
		failMessageCounter++ ; 
		Sprint("Fail on message: ") ; Sprint(failStorePointer) ;
		Sprint(" # ") ; Sprintln(failMessageCounter);
	}
	failStorePointer++ ;
	if(failStorePointer >= messageCounterMax){
		failStorePointer =  0	;						// wrap counter
	}
	parent = getParentNodeId();							// get the parent node (0 = gateway)
	wait(counterUpdateDelay) ;							// wait for things to settle and ack's to arrive
}

void receive(const MyMessage &message) {  				// Expect few types of messages from controller
	newMessage = message.getInt();						// get received value
	switch (message.type){
		case V_PERCENTAGE:
			missedStore[newMessage] = true ;			// set corresponding flag to received.
			if (newMessage > lastMessage){				// number of messages missed from lastMessage (kind of, faulty at wrap)
				Sprint("Missed messages: ") ; Sprintln( newMessage - lastMessage - 1) ;
				missedMessageCounter += newMessage - lastMessage - 1 ;
			}
			lastMessage = newMessage ;
			break ;
		default: break ;
	}
}


// calculate number of false values in array 
// takes a lot of time, but who cares...
int getCount(boolean countArray[], int size){
	int falseCount = 0 ;
	for (int i = 0 ; i < size ; i++){
		falseCount += countArray[i]?0:1 ;
	}
	return falseCount ;
}

void LCD_local_display(void){
/* prints all available variables on LCD display with units
*/
	
    char buf[17]; 											// buffer for max 16 char display
    lcd.setCursor(0, 0);
    snprintf(buf, sizeof buf, "p%-3dFail%4d%3d%%", parent, failMessageCounter, getCount(failStore, messageCounterMax));
    lcd.print(buf);
    lcd.setCursor(0, 1);
    snprintf(buf, sizeof buf, "d%-3dMiss%4d%3d%%", DESTINATION_NODE , missedMessageCounter, getCount(missedStore, messageCounterMax));
	lcd.print(buf);
}

cjrpriest

Thanks very much @AWI 👍

karl261

@AWI Strange, with the latest sketch I get

nrf-quality-meter.ino:88:34: error: invalid conversion from 'int' to 't_backlighPol' [-fpermissive]
In file included from nrf-quality-meter.ino:45:0:
C:\Dokumente und Einstellungen\Philipp\Eigene Dateien\Arduino\libraries\LiquidCrystal/LiquidCrystal_I2C.h:53:4: error: initializing argument 3 of 'LiquidCrystal_I2C::LiquidCrystal_I2C(uint8_t, uint8_t, t_backlighPol)' [-fpermissive]
LiquidCrystal_I2C (uint8_t lcd_Addr, uint8_t backlighPin, t_backlighPol pol);
^
invalid conversion from 'int' to 't_backlighPol' [-fpermissive]

compiling...

It works using the other line...

// ***** LCD
//LiquidCrystal_I2C lcd(0x27, 2, 1, 0, 4, 5, 6, 7, 3, POSITIVE);  // Set the LCD I2C address
LiquidCrystal_I2C lcd(0x27, 16, 2);  // Set the LCD I2C address

AWI

@karl261 probably I am using another library for the display. I got it from the Arduino library manager.

Nicklas Starkel

@AWI what is the board between the nano and the radio?
Something available via aliexpress?

I really do not need this as I live in a small flat.. but I NEED THIS! :laughing:

AWI

@Nicklas-Starkel you can find it in the MySensors shop It is a nrf24l01+ socket adapter board. It guarantees a stable supply and includes the famous "radio capacitor"

siod

Hi @AWI,

thanks for the code and the project! ~~Unfortunately my biggest problem is to get my LCD working. It does not do anything with your code, other testcode to test the display is working fine. I am using this line and it should work fine:

LiquidCrystal_I2C lcd(0x27, 2, 1, 0, 4, 5, 6, 7, 3, POSITIVE);  // Set the LCD I2C address

Any idea what else I could check? I am using arduino IDE 1.6.10

edit: getting this in the serial monitor of the device:

Starting sensor (RNNNA-, 2.0.0)
TSM:INIT
TSM:RADIO:OK
TSP:ASSIGNID:OK (ID=250)
TSM:FPAR
TSP:MSG:SEND 250-250-255-255 s=255,c=3,t=7,pt=0,l=0,sg=0,ft=0,st=bc:
TSM:FPAR
TSP:MSG:SEND 250-250-255-255 s=255,c=3,t=7,pt=0,l=0,sg=0,ft=0,st=bc:
TSM:FPAR
TSP:MSG:SEND 250-250-255-255 s=255,c=3,t=7,pt=0,l=0,sg=0,ft=0,st=bc:
TSM:FPAR
TSP:MSG:SEND 250-250-255-255 s=255,c=3,t=7,pt=0,l=0,sg=0,ft=0,st=bc:
TSP:MSG:READ 3-3-255 s=255,c=3,t=7,pt=0,l=0,sg=0:
TSP:MSG:BC
!TSM:FPAR:FAIL
!TSM:FAILURE
TSM:PDT

then it starts over again...
~~
edit2:
Think I got it now, seems like my gateway wasn´t responding or whatever, now it seems to work...

Edit3:

I got 1 fail and 5000 miss 60%...
Now what does this mean??

Edit4:

I think I need some help: Most of the time I get this on my gw´s Serial Monitor:

0;255;3;0;9;TSP:MSG:SEND 0-0-250-250 s=255,c=3,t=8,pt=1,l=1,sg=0,ft=0,st=ok:0
0;255;3;0;9;TSP:MSG:READ 250-250-255 s=255,c=3,t=7,pt=0,l=0,sg=0:
0;255;3;0;9;TSP:MSG:BC
0;255;3;0;9;TSP:MSG:FPAR REQ (sender=250)
0;255;3;0;9;TSP:CHKUPL:OK (FLDCTRL)
0;255;3;0;9;TSP:MSG:GWL OK
0;255;3;0;9;TSP:MSG:SEND 0-0-250-250 s=255,c=3,t=8,pt=1,l=1,sg=0,ft=0,st=ok:0
0;255;3;0;9;TSP:MSG:READ 250-250-255 s=255,c=3,t=7,pt=0,l=0,sg=0:
0;255;3;0;9;TSP:MSG:BC
0;255;3;0;9;TSP:MSG:FPAR REQ (sender=250)
0;255;3;0;9;TSP:CHKUPL:OK (FLDCTRL)
0;255;3;0;9;TSP:MSG:GWL OK
0;255;3;0;9;TSP:MSG:SEND 0-0-250-250 s=255,c=3,t=8,pt=1,l=1,sg=0,ft=0,st=ok:0
0;255;3;0;9;TSP:MSG:READ 250-250-255 s=255,c=3,t=7,pt=0,l=0,sg=0:
0;255;3;0;9;TSP:MSG:BC
0;255;3;0;9;TSP:MSG:FPAR REQ (sender=250)
0;255;3;0;9;TSP:CHKUPL:OK
0;255;3;0;9;TSP:MSG:GWL OK

After some reboots of the quality meter and I guess with a lot of luck it starts working occasionally

This is from the serial monitor of the quality meter:

TSP:MSG:READ 0-0-0 s=0,c=0,t=0,pt=0,l=0,sg=0:
!TSP:MSG:PVER mismatch
!TSM:FPAR:FAIL
!TSM:FAILURE
TSM:PDT

I already added a 47u Cap and added those lines to the code:

#define MY_PARENT_NODE_ID 0                         // fixed parent to controller when 0 (else comment out = AUTO)
#define MY_PARENT_NODE_IS_STATIC

no luck at all...

edit5:
Now I opened the serial output of my GW again, which seems to started the communication:

0;255;3;0;9;TSP:MSG:READ 250-250-255 s=255,c=3,t=7,pt=0,l=0,sg=0:
0;255;3;0;9;TSP:MSG:BC
0;255;3;0;9;TSP:MSG:FPAR REQ (sender=250)
0;255;3;0;9;TSP:CHKUPL:OK (FLDCTRL)
0;255;3;0;9;TSP:MSG:GWL OK
0;255;3;0;9;TSP:MSG:SEND 0-0-250-250 s=255,c=3,t=8,pt=1,l=1,sg=0,ft=0,st=ok:0
0;255;3;0;9;TSP:MSG:READ 250-250-0 s=255,c=3,t=24,pt=1,l=1,sg=0:1
0;255;3;0;9;TSP:MSG:PINGED (ID=250, hops=1)
0;255;3;0;9;TSP:MSG:SEND 0-0-250-250 s=255,c=3,t=25,pt=1,l=1,sg=0,ft=0,st=ok:1
0;255;3;0;9;TSP:MSG:READ 250-250-0 s=255,c=3,t=15,pt=6,l=2,sg=0:0100
0;255;3;0;9;!TSP:MSG:SEND 0-0-250-250 s=255,c=3,t=15,pt=6,l=2,sg=0,ft=0,st=fail:0100
0;255;3;0;9;TSP:MSG:READ 250-250-0 s=255,c=0,t=17,pt=0,l=5,sg=0:2.0.0
0;255;3;0;9;Sending message on topic: mygateway1-out/250/255/0/0/17
0;255;3;0;9;TSP:MSG:READ 250-250-0 s=255,c=3,t=6,pt=1,l=1,sg=0:0
0;255;3;0;9;Sending message on topic: mygateway1-out/250/255/3/0/6
0;255;3;0;9;TSP:MSG:READ 250-250-0 s=0,c=0,t=4,pt=0,l=21,sg=0:Quality counter Q 250
0;255;3;0;9;Sending message on topic: mygateway1-out/250/0/0/0/4
0;255;3;0;9;TSP:MSG:READ 250-250-0 s=255,c=3,t=26,pt=1,l=1,sg=0:2
0;255;3;0;9;TSP:MSG:SEND 0-0-250-250 s=255,c=3,t=27,pt=1,l=1,sg=0,ft=0,st=ok:1
0;255;3;0;9;TSP:MSG:READ 250-250-0 s=255,c=3,t=26,pt=1,l=1,sg=0:2
0;255;3;0;9;TSP:MSG:SEND 0-0-250-250 s=255,c=3,t=27,pt=1,l=1,sg=0,ft=0,st=ok:1
0;255;3;0;9;TSP:MSG:READ 250-250-0 s=255,c=3,t=26,pt=1,l=1,sg=0:2
0;255;3;0;9;TSP:MSG:SEND 0-0-250-250 s=255,c=3,t=27,pt=1,l=1,sg=0,ft=0,st=ok:1
0;255;3;0;9;TSP:MSG:READ 250-250-0 s=0,c=1,t=3,pt=2,l=2,sg=0:0
0;255;3;0;9;TSP:MSG:ACK msg
0;255;3;0;9;TSP:MSG:SEND 0-0-250-250 s=0,c=1,t=3,pt=2,l=2,sg=0,ft=0,st=ok:0
0;255;3;0;9;Sending message on topic: mygateway1-out/250/0/1/0/3
0;255;3;0;9;TSP:MSG:READ 250-250-0 s=0,c=1,t=3,pt=2,l=2,sg=0:1
0;255;3;0;9;TSP:MSG:ACK msg
0;255;3;0;9;TSP:MSG:SEND 0-0-250-250 s=0,c=1,t=3,pt=2,l=2,sg=0,ft=0,st=ok:1
0;255;3;0;9;Sending message on topic: mygateway1-out/250/0/1/0/3
0;255;3;0;9;TSP:MSG:READ 250-250-0 s=0,c=1,t=3,pt=2,l=2,sg=0:2
0;255;3;0;9;TSP:MSG:ACK msg

I don´t get it what is going on here. Restarted quality meter: blank LCD... :(

nRf24L01+ connection quality meter

16

11.7k

11.2k

113.1k