diff --git a/.clang-format b/.clang-format index 2228af0b9..e84ce1c3c 100644 --- a/.clang-format +++ b/.clang-format @@ -36,4 +36,6 @@ PointerAlignment: Middle SpaceAfterCStyleCast: false SpaceBeforeAssignmentOperators: true SpacesInCStyleCastParentheses: false -SpacesInParentheses: false \ No newline at end of file +SpacesInParentheses: false +DerivePointerAlignment: false +SortIncludes: false \ No newline at end of file diff --git a/.travis.yml b/.travis.yml index f108c2321..ad0bdbab8 100644 --- a/.travis.yml +++ b/.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 diff --git a/CHANGELOG.md b/CHANGELOG.md index df6f20278..675e5a57b 100644 --- a/CHANGELOG.md +++ b/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) diff --git a/src/ems-esp.cpp b/src/ems-esp.cpp index 3a6e9b76a..476abbfe0 100644 --- a/src/ems-esp.cpp +++ b/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(); diff --git a/src/ems.cpp b/src/ems.cpp index 6ea0854c0..34c0fc343 100644 --- a/src/ems.cpp +++ b/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 diff --git a/src/ems_devices.h b/src/ems_devices.h index f30617177..6f5dcd8f6 100644 --- a/src/ems_devices.h +++ b/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_heatingtype 0 // floor heating = 3 0x47 -#define EMS_OFFSET_RC35Set_circuitcalctemp 14 // calculated circuit temperature 0x48 +#define EMS_OFFSET_RC35Set_temp_holiday 3 // temp during holiday mode +#define EMS_OFFSET_RC35Set_heatingtype 0 // e.g. floor heating = 3 +#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 diff --git a/src/version.h b/src/version.h index 7792addc0..6733def75 100644 --- a/src/version.h +++ b/src/version.h @@ -1 +1 @@ -#define APP_VERSION "1.9.5b18m" +#define APP_VERSION "1.9.5b19"