Reliability again

[Yveaux]

@hek I didn't fix that (yet) as we didn't agree on the solution yet ;-)
Let's finish that discussion then and fix this bug :facepunch:

[JeJ]

I had a similar problem.
When the sensor node doesn't receive kWh from the gateway it "spams" the radio network with the kWh request. (Value 24)
In my case all of the other senors also lost communication.
If i set "boolean pcReceived = true;" it works.

[hek]

@JeJ
Yes, thats it! A bug in the PulseSensor code.
I will check in an updated version during the evening (CET). lastSend must be updated in loop().

 .....
 } else if (sendTime && !pcReceived) {
    // No count received. Try requesting it again
   gw.request(CHILD_ID, V_VAR1);
   lastSend=now;
 }
.....

[hek]

Fix for this has been checked into master now.

[tbully]

Thanks guys. Nice catch. I also had this problem with the water version.

[tbully]

Can someone explain how to call gw.begin(); in a sensor to adjust the power level?

I read the troubleshooting page but it wasn't clear how the parameters would look.

When calling the function, do you have to call all the parameters or just the power level?

gw.begin(RF24_PA_LOW); ?

Is it the same for a gateway?

[blacey]

The best place to set the power level is in MyConfig.h. You can set the channel, gateway power level and sensor power level. See https://github.com/mysensors/Arduino/blob/master/libraries/MySensors/MyConfig.h

[tbully]

Thanks @blacey . I was trying to configure it per sensor without having to remember to go in to the library each time.

This works fine, though.

[jocke4u]

@hek
Thanks for the fix.
I have implemented it but still the gateway hangs when sending the requested KWH value to the sensor. Doesn't it have any timeout etc?

I have also implemented the fix + lowered PA. Still not working.

[jocke4u]

@blacey said:

The best place to set the power level is in MyConfig.h.

I am not sure if I agree if you have different sensors with different hardware (as I have) since changing MyConfig.h will change it globally.

@tbully
I have used this code:

gw.begin(incomingMessage, AUTO, false, AUTO, RF24_PA_LOW);

Since the PA parameter is the fourth one, you need to set the ones before (but go with defaults for the ones after)

[hek]

@jocke4u said:

I have implemented it but still the gateway hangs when sending the requested KWH value to the sensor.

Strange. Is the sensor still "spamming" the gateway?

[tbully]

The fix certainly fixed the random "spamming" issue for me. I was tearing my hair out on that one.

I have not had the hang yet but have only been running my water sensors (pulse, just like the power) for 4-5 days.

[jocke4u]

@hek said:

Strange. Is the sensor still "spamming" the gateway?

Damn, I made the fix but because of a stupid reason I uploaded the old version :(
I have now loaded a limited software with only EnergyPulseMeter, it partly works but I get too many "fail" when sending (even though many reaches GW).
As next step I think I need to connect an external 5V power supply and AMS1117 to feed the radio and hopefully get better reliability in transmits.
I guess no extra capacitor is needed due to 800mA max current?

[blacey]

@jocke4u said:

I have used this code:

gw.begin(incomingMessage, AUTO, false, AUTO, RF24_PA_LOW);

Since the PA parameter is the fourth one, you need to set the ones before (but go with defaults for the ones after)

Yes, the MySensors configuration design is to set the unified configuration parameters in MyConfig.h to ensure common sensor configs (.e.g. channel) across sensors but then override on a per-sensor basis as needed.