Thank you!

[danielkucera]

Thanks for this library. Are you using it with some automation?
I plan to use it with esphome.

And what do you mean by "changing memory partitioning"? How should be this done?

[MrKrabat]

The Arduino IDE has a setting for memory partitioning (depending on if the selected board supports it).
As the esp32 bluetooth library needs a lot of space, I needed to change the memory partition to make my program fit onto the esp32.
For more about this see here: https://robotzero.one/arduino-ide-partitions/

I use this library to integrate my thermostat into Google Home. For this I use gBridge as cloud backend and bridge between my esp32 and Google :)

Thanks for commenting, I hope you won't find too many bugs :)

[danielkucera]

I have some initial version which compiles and connects: https://github.com/danielkucera/esphome/blob/cometblue/esphome/components/cometblue/cometblue.cpp
But I'm getting following error. Do you think it is related to memory?

[16:57:29][C][cometblue:030]: Setting up CometBlue client for 'B7:1E:5A:99:BF:21'...
[16:57:32][E][BLERemoteCharacteristic.cpp:274] retrieveDescriptors(): esp_ble_gattc_get_all_descr: Unknown
[16:57:32][E][BLERemoteCharacteristic.cpp:274] retrieveDescriptors(): esp_ble_gattc_get_all_descr: Unknown
[16:57:32][E][BLERemoteCharacteristic.cpp:274] retrieveDescriptors(): esp_ble_gattc_get_all_descr: Unknown
[16:57:32][E][BLERemoteCharacteristic.cpp:274] retrieveDescriptors(): esp_ble_gattc_get_all_descr: Unknown
[16:57:32][E][BLERemoteCharacteristic.cpp:274] retrieveDescriptors(): esp_ble_gattc_get_all_descr: Unknown
[16:57:32][E][BLERemoteCharacteristic.cpp:274] retrieveDescriptors(): esp_ble_gattc_get_all_descr: Unknown
[16:57:32][E][BLERemoteCharacteristic.cpp:274] retrieveDescriptors(): esp_ble_gattc_get_all_descr: Unknown
[16:57:32][E][BLERemoteCharacteristic.cpp:274] retrieveDescriptors(): esp_ble_gattc_get_all_descr: Unknown
[16:57:32][E][BLERemoteCharacteristic.cpp:274] retrieveDescriptors(): esp_ble_gattc_get_all_descr: Unknown
[16:57:32][E][BLERemoteCharacteristic.cpp:274] retrieveDescriptors(): esp_ble_gattc_get_all_descr: Unknown
[16:57:32][E][BLERemoteCharacteristic.cpp:274] retrieveDescriptors(): esp_ble_gattc_get_all_descr: Unknown
[16:57:32][E][BLERemoteCharacteristic.cpp:274] retrieveDescriptors(): esp_ble_gattc_get_all_descr: Unknown
[16:57:32][E][BLERemoteCharacteristic.cpp:274] retrieveDescriptors(): esp_ble_gattc_get_all_descr: Unknown
[16:57:32][E][BLERemoteCharacteristic.cpp:274] retrieveDescriptors(): esp_ble_gattc_get_all_descr: Unknown
[16:57:32][E][BLERemoteCharacteristic.cpp:274] retrieveDescriptors(): esp_ble_gattc_get_all_descr: Unknown
[16:57:32][E][BLERemoteCharacteristic.cpp:274] retrieveDescriptors(): esp_ble_gattc_get_all_descr: Unknown
[16:57:32][E][BLERemoteCharacteristic.cpp:274] retrieveDescriptors(): esp_ble_gattc_get_all_descr: Unknown
[16:57:32][E][BLERemoteCharacteristic.cpp:274] retrieveDescriptors(): esp_ble_gattc_get_all_descr: Unknown
[16:57:32][E][BLERemoteCharacteristic.cpp:274] retrieveDescriptors(): esp_ble_gattc_get_all_descr: Unknown
[16:57:32][E][BLERemoteCharacteristic.cpp:274] retrieveDescriptors(): esp_ble_gattc_get_all_descr: Unknown
[16:57:32][E][BLERemoteCharacteristic.cpp:274] retrieveDescriptors(): esp_ble_gattc_get_all_descr: Unknown
[16:57:32][E][BLERemoteCharacteristic.cpp:274] retrieveDescriptors(): esp_ble_gattc_get_all_descr: Unknown
[16:57:33][C][cometblue:038]: Connect to 'B7:1E:5A:99:BF:21' SUCCESS!
[16:57:33][D][climate:172]: 'Living Room AC' - Sending state:
[16:57:33][D][climate:175]:   Mode: OFF
[16:57:33][D][climate:186]:   Target Temperature: 0.00°C

[danielkucera]

Ok, I take it back, problem was elsewhere, it works now :)

[danielkucera]

Ok, another problem. When I run this code:

      Temperatures temp = dev->getTemperatures();
      temp.manual_temp = *call.get_target_temperature(); // target temperature in degree Celsius
      ESP_LOGCONFIG(TAG, "About to set target temperature for '%s' to %f", this->mac_.c_str(), temp.manual_temp);
      //TODO: fix 64deg currrent temp
      if (!dev->setTemperatures(temp)){
        ESP_LOGCONFIG(TAG, "Seting temperature failed!");
      }

suddenly I'm getting 64deg current temperature:

[23:23:21][D][climate:010]: 'Living Room Radiator' - Setting
[23:23:21][D][climate:017]:   Target Temperature: 23.00
[23:23:22][C][cometblue:035]: Connected to 'B7:1E:5A:99:BF:21' 'Comet Blue'
[23:23:22][C][cometblue:084]: About to set target temperature for 'B7:1E:5A:99:BF:21' to 23.000000
[23:23:22][C][cometblue:059]: Updating state of 'B7:1E:5A:99:BF:21'...
[23:23:23][C][cometblue:035]: Connected to 'B7:1E:5A:99:BF:21' 'Comet Blue'
[23:23:23][D][climate:172]: 'Living Room Radiator' - Sending state:
[23:23:23][D][climate:175]:   Mode: HEAT
[23:23:23][D][climate:177]:   Action: OFF
[23:23:23][D][climate:180]:   Current Temperature: 64.00°C
[23:23:23][D][climate:186]:   Target Temperature: 23.00°C

Does this happen to you too?

[danielkucera]

Should not here:

Comet-Blue-Thermostat/CometBlue.cpp

Line 511 in f9caae5

uint8_t tempencoded[7] = {128, temp.manual_temp * 2, temp.target_temp_low * 2, temp.target_temp_high * 2, temp.offset_temp * 2, temp.window_open_detect, temp.window_open_minutes};

be -128?

https://github.com/im-0/cometblue/blob/4d9d65c2d53564148cfdc9fcf7409be246b4dad9/cometblue/device.py#L88

[MrKrabat]

I'm not sure, I never had this problem or testet for this. The current temperature should be set by the thermostat, regardless what value we set.
This value can only take up a single byte and byte(128) equals int(-128) in two's complement.

[danielkucera]

Are you disconnecting and reconnecting after each API call?

[MrKrabat]

Yes, I disconnect after each call.
Don't want to drain too much battery, so I'm using my own temperature sensor with my esp