Merge branch 'dev' of https://github.com/proddy/EMS-ESP.git into MM10

.clang-format

@@ -36,4 +36,6 @@ PointerAlignment: Middle
 SpaceAfterCStyleCast: false
 SpaceBeforeAssignmentOperators: true
 SpacesInCStyleCastParentheses: false
-SpacesInParentheses: false
+SpacesInParentheses: false
+DerivePointerAlignment: false
+SortIncludes: false

.travis.yml

@@ -1,4 +1,5 @@
 os: linux
+dist: bionic
 language: python
 python:
   - "3.8"
@@ -55,7 +56,7 @@ before_deploy:
 deploy:
   provider: releases
   edge:
-    source: wenkokke/dpl
+  #  source: wenkokke/dpl
     branch: master
   token: ${GITHUB_TOKEN}
   file_glob: true

CHANGELOG.md

@@ -15,6 +15,8 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ### Fixed
 - set boiler warm water temp on Junkers/Bosch HT3
+- fixed detection of the Moduline 400 thermostat
+- RC35 setting temperature also forces the current select temp to change, irrespective of the mode
 
 ### Changed
 - improved MQTT publishing to stop flooding. `publish_time` must be at least 1 (second)

src/ems-esp.cpp

@@ -479,7 +479,7 @@ void showInfo() {
     if (ems_getMixingDeviceEnabled()) {
         myDebug_P(PSTR("")); // newline
         myDebug_P(PSTR("%sMixing module data:%s"), COLOR_BOLD_ON, COLOR_BOLD_OFF);
-        myDebug_P(PSTR("  Mixing: %s"), ems_getDeviceDescription(EMS_DEVICE_TYPE_MIXING, buffer_type,false));
+        // myDebug_P(PSTR("  Mixing: %s"), ems_getDeviceDescription(EMS_DEVICE_TYPE_MIXING, buffer_type,false));
         if (EMS_Boiler.switchTemp < EMS_VALUE_USHORT_NOTSET)
            _renderUShortValue("Switch temperature", "C", EMS_Boiler.switchTemp);
 
@@ -951,7 +951,7 @@ void do_regularUpdates() {
 // turn back on the hot water for the shower
 void _showerColdShotStop() {
     if (EMSESP_Shower.doingColdShot) {
-        myDebugLog("[Shower] finished shot of cold. hot water back on");
+        myDebugLog("[Shower] finished shot of cold water. Hot water turned back on");
         ems_setWarmTapWaterActivated(true);
         EMSESP_Shower.doingColdShot = false;
         showerColdShotStopTimer.detach(); // disable the timer
@@ -961,7 +961,7 @@ void _showerColdShotStop() {
 // turn off hot water to send a shot of cold
 void _showerColdShotStart() {
     if (EMSESP_Settings.shower_alert) {
-        myDebugLog("[Shower] doing a shot of cold water");
+        myDebugLog("[Shower] starting shot of cold water");
         ems_setWarmTapWaterActivated(false);
         EMSESP_Shower.doingColdShot = true;
         // start the timer for n seconds which will reset the water back to hot
@@ -1975,9 +1975,9 @@ void showerCheck() {
 // SETUP
 //
 void setup() {
-    // GPIO15/D8 has a pull down, so we must set it to HIGH so it doesn't bring the whole EMS bus down
+    // GPIO15/D8 has an onboard pull down resistor and we use this for Tx, so force it high so it doesn't bring the whole EMS bus down
     pinMode(D8, OUTPUT);
-    digitalWrite(D8, 1);
+    digitalWrite(D8, HIGH);
 
     // init our own parameters
     initEMSESP();

src/ems.cpp

@@ -2369,11 +2369,15 @@ void ems_setThermostatTemp(float temperature, uint8_t hc_num, uint8_t temptype)
             break;
         default:
         case 0: // automatic selection, if no type is defined, we use the standard code
+            EMS_TxTelegram.offset = EMS_OFFSET_RC35Set_seltemp;
+
+            /* commented out for https://github.com/proddy/EMS-ESP/issues/310
             if (EMS_Thermostat.hc[hc_num - 1].day_mode == 0) {
                 EMS_TxTelegram.offset = EMS_OFFSET_RC35Set_temp_night;
             } else if (EMS_Thermostat.hc[hc_num - 1].day_mode == 1) {
                 EMS_TxTelegram.offset = EMS_OFFSET_RC35Set_temp_day;
             }
+            */
             break;
         }
 
@@ -2719,7 +2723,7 @@ void ems_setWarmTapWaterActivated(bool activated) {
     EMS_TxTelegram.data[3] = EMS_TxTelegram.offset; // offset
 
     // we use the special test mode 0x1D for this. Setting the first data to 5A puts the system into test mode and
-    // a setting of 0x00 puts it back into normal operarting mode
+    // a setting of 0x00 puts it back into normal operating mode
     // when in test mode we're able to mess around with the 3-way valve settings
     if (!activated) {
         // on

src/ems_devices.h

@@ -121,9 +121,10 @@
 #define EMS_OFFSET_RC35Set_mode 7             // position of thermostat mode
 #define EMS_OFFSET_RC35Set_temp_day 2         // position of thermostat setpoint temperature for day time
 #define EMS_OFFSET_RC35Set_temp_night 1       // position of thermostat setpoint temperature for night time
-#define EMS_OFFSET_RC35Set_temp_holiday 3     // temp during holiday 0x47
+#define EMS_OFFSET_RC35Set_temp_holiday 3     // temp during holiday mode
-#define EMS_OFFSET_RC35Set_heatingtype 0      // floor heating = 3 0x47
+#define EMS_OFFSET_RC35Set_heatingtype 0      // e.g. floor heating = 3
-#define EMS_OFFSET_RC35Set_circuitcalctemp 14 // calculated circuit temperature 0x48
+#define EMS_OFFSET_RC35Set_circuitcalctemp 14 // calculated circuit temperature
+#define EMS_OFFSET_RC35Set_seltemp 37         // selected temp
 
 // Easy specific
 #define EMS_TYPE_EasyStatusMessage 0x0A          // reading values on an Easy Thermostat

src/version.h

@@ -1 +1 @@
-#define APP_VERSION "1.9.5b18m"
+#define APP_VERSION "1.9.5b19"