I am running umqttsimple.py with Micropython 1.17 on an ESP32 client. Everything was running perfectly and then, by accident, I used the same MQTT client ID on another device. MQTT responded by disconnecting the original device - it is designed to disconnect the "older" client. I did have a dummy callback in place but was not expecting any topic messages as I had not subscribed to any.

I assume that when MQTT disconnected it created a socket error. The code at line 175 says: "if res == b"": raise OSERROR(-1)". This is the error I got and the main program stopped right there. This may be a socket handling error elsewhere but, in order to make the code more robust I changed the logic to "if res is None or res == b"": return None"

This patch works. Even if the code is not going to be patched, I hope that, by raising the issue, others will see what is happening when it happens to them.

Maybe the real problem is with the socket code which should return "none" when disconnected?

code:
client = umqttsimple.MQTTClient(ubinascii.hexlify(machine.unique_id()), '192.168.2.254',keepalive=100, **{'user':'a','password':'ni'})
client.set_last_will('a/availability','offline',retain=True)
client.connect()
client.publish('a/availability','online',retain=True)
while True:
client.ping();print('ping');time.sleep(1)
ping
ping
ping
ping
Disconnected

when I try ping my esp from other computer I see that esp desapear

I work on esp8266-20200911-v1.13.bin

Hi,

I have added support for circles and display rotation to your OLED lib.

https://github.com/perryflynn/joshua

Are you interested in a Merge Request?

Regards
Christian

It seems that there is a typing error in BME280.py. In method read_pressure (see code lines below) the right-shift operator >>12 in line 9 should be replaced by a left-shift operator <<12. This can be easily seen when comparing the coding with the coding in the original Bosch data sheet on page 25 (Document number: BST-BME280-DS002-15, Revision_1.6_092018).

In my test case the error mentioned above and as shown below caused a pressure decay of about 1500 Pa when I increased the sensor temperature by about 8 K via touching the sensor with my finger. From a physical point of view the pressure decay observed seems to be unreasonable and fortunately it vanishes, when the left-shift operator is used.

def read_pressure(self):
"""Gets the compensated pressure in Pascals."""
adc = self.read_raw_pressure()
var1 = self.t_fine - 128000
var2 = var1 * var1 * self.dig_P6
var2 = var2 + ((var1 * self.dig_P5) << 17)
var2 = var2 + (self.dig_P4 << 35)
var1 = (((var1 * var1 * self.dig_P3) >> 8) +
((var1 * self.dig_P2) >> 12))
!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
var1 = (((1 << 47) + var1) * self.dig_P1) >> 33
if var1 == 0:
return 0
p = 1048576 - adc
p = (((p << 31) - var2) * 3125) // var1
var1 = (self.dig_P9 * (p >> 13) * (p >> 13)) >> 25
var2 = (self.dig_P8 * p) >> 19
return ((p + var1 + var2) >> 8) + (self.dig_P7 << 4)

Best regards
Ebi

PS:

Here is the Bosch coding:

// Returns pressure in Pa as unsigned 32 bit integer in Q24.8 format (24 integer bits and 8
fractional bits).
// Output value of “24674867” represents 24674867/256 = 96386.2 Pa = 963.862 hPa
BME280_U32_t BME280_compensate_P_int64(BME280_S32_t adc_P)
{
BME280_S64_t var1, var2, p;
var1 = ((BME280_S64_t)t_fine) – 128000;
var2 = var1 * var1 * (BME280_S64_t)dig_P6;
var2 = var2 + ((var1*(BME280_S64_t)dig_P5)<<17);
var2 = var2 + (((BME280_S64_t)dig_P4)<<35);
var1 = ((var1 * var1 * (BME280_S64_t)dig_P3)>>8) + ((var1 * (BME280_S64_t)dig_P2)<<12);
var1 = (((((BME280_S64_t)1)<<47)+var1))*((BME280_S64_t)dig_P1)>>33;
if (var1 == 0)
{
return 0; // avoid exception caused by division by zero
}
p = 1048576-adc_P;
p = (((p<<31)-var2)*3125)/var1;
var1 = (((BME280_S64_t)dig_P9) * (p>>13) * (p>>13)) >> 25;
var2 = (((BME280_S64_t)dig_P8) * p) >> 19;
p = ((p + var1 + var2) >> 8) + (((BME280_S64_t)dig_P7)<<4);
return (BME280_U32_t)p;