[Mitsubishi] Not work Power OFF command #1402
Labels
Comments
|
Brand: Mitsubishi Electric, Model: MSZ-GV2519 A/C (MITSUBISHI_AC) The above devices can be turned off with the current library without any problems. The dump is shown below, but it seems that Mode is not OFF when turning it off with my Mitsubishi air conditioners. |
|
First off, I'm confused by how you are explaining the issue(s). Please be more verbose and explain more what you are doing, and what the steps are, and what the results are. For instance, are the captures you have provided, from the remote or from the ESP? (Please be more detail and verbose) What are the sequences you are doing/code you are using to generate IR messages. (please include the smallest possible/practical example so I can reproduce your problem) |
|
Sorry for lack of explanation. The problem is that I can't turn off the air conditioner with the commands I send from this library.
This was my misunderstanding. Below is a dump of the commands sent by this library. It says "Power: Off, Mode: 4 (Auto)", but one air conditioner does not seem to accept "Mode: 4 (Auto)". By setting this to "Power: Off, Mode: 1 (Heat)", I was able to turn off the air conditioner. |
Okay, but how are you telling it to go to these modes? Please provide the code and/or detailed instructions/steps etc. There are multiple ways you could be doing this and I can't reproduce or diagnose the path without knowing this. |
|
I'm currently controlling with Tasmota --https://github.com/hristo-atanasov/Tasmota-IRHVAC --HomeAssistant. When I send the OFF command in this environment, it is sent with his MQTT payload as follows. {"Vendor":"MITSUBISHI_AC","Model":-1,"Mode":"Off","Power":"Off","Celsius":"On","Temp":18,"FanSpeed":"Auto","SwingV":"Off","SwingH":"Off","Quiet":"Off","Turbo":"Off","Econo":"Off","Light":"Off","Filter":"Off","Clean":"Off","Beep":"Off","Sleep":-1}I was able to turn off the air conditioner by sending this payload part as follows: {"Vendor":"MITSUBISHI_AC","Model":-1,"Mode":"Heat","Power":"Off","Celsius":"On","Temp":18,"FanSpeed":"Auto","SwingV":"Off","SwingH":"Off","Quiet":"Off","Turbo":"Off","Econo":"Off","Light":"Off","Filter":"Off","Clean":"Off","Beep":"Off","Sleep":-1} |
|
@crankyoldgit Is this a problem with Tasmota-IRHVAC? |
|
I don't know yet. This is why we prefer people using our tools/examples or providing code as it eliminates a whole layer of complexity/potential bugs/problems. i.e. We don't debug other people's code unless it's trivially small. |
|
Yes, I'm sorry to report the problem in an ambiguous state. Also, thank you for taking your precious time. If this library does not specify the mode when turning off the air conditioner and you need instructions from the user, I think this is a problem with Tasmota-IRHVAC. |
|
I found the cause of this problem in Tasmota. When Power is Off, Mode is forcibly rewritten to Off. I rewrote this as follows, and now I can control it by issuing the appropriate command in Tasmota-IRHVAC. // Home Assistant wants mode to be off if power is also off & vice-versa.
if (state.mode == stdAc::opmode_t::kOff || !state.power) {
json.add(PSTR(D_JSON_IRHVAC_MODE), IRac::opmodeToString(state.mode));
json.add(PSTR(D_JSON_IRHVAC_POWER), IRac::boolToString(false));
} else {
json.add(PSTR(D_JSON_IRHVAC_MODE), IRac::opmodeToString(state.mode));
json.add(PSTR(D_JSON_IRHVAC_POWER), IRac::boolToString(state.power));
} |
nao-pon
added a commit
to nao-pon/Tasmota
that referenced
this issue
Feb 4, 2021
Power and mode must be individually controllable. Some devices may not function properly if the mode is also turned off when the power is turned off. see. crankyoldgit/IRremoteESP8266#1402
6 tasks
Yes, This is typically required because Home Assistant likes it to look that way. So you may be barking up the wrong tree. |
|
For the time being, I made a PR for Tasmota. HomeAssistant core-2021.1.5 doesn't seem to have any problems. Thank you very much. |
nao-pon
added a commit
to nao-pon/Tasmota-IRHVAC
that referenced
this issue
Feb 4, 2021
The final mode other than OFF is retained, and it can be saved and restored. This will do the following: - From the OFF state, when the Turn on command is accepted, the operation starts in the previous mode. - With the option "keep_mode_when_off: True", when sending an OFF command, only Power is Off and Mode sends the previous mode. - crankyoldgit/IRremoteESP8266#1402 - arendst/Tasmota#10829
Jason2866
added a commit
to Jason2866/Tasmota
that referenced
this issue
Feb 12, 2021
* Prep ESP32S2 * LITTLEFS.h casing for ESP32 * Publish more precise Free Heap metric * Zigbee command ``ZbScan`` to do an energy scan on each radio channel * Reduce code size * fix i2s audio * Update AudioOutputI2SNoDAC.cpp * Allows individual control of power and mode Power and mode must be individually controllable. Some devices may not function properly if the mode is also turned off when the power is turned off. see. crankyoldgit/IRremoteESP8266#1402 * Prep ESP32S2 * Add fragmentation translations * ESP32 increase number of relay GPIOs from 8 to 28 * Add Server to HTTP header (arendst#10831) * BLE_ESP32 (xdrv_79): Start scan as soon as we are connected to a device - saves missing a few adverts. iBeacon (xdrv_52): Clean up BLE_ESP32 based driver; add commands and advert timing. Remove HM10 code - it's in the other one..... * Add timeout initialise * Add additional #if for CONFIG_IDF_TARGET_ESP32 * update comment * Increase number of switch GPIOs from 8 to 28 * Increase number of button GPIOs from 4 to 8 * Allow BLE when ethernet only connected Allow BLE when ethernet only connected (arendst#10753) * ESP32 Increase number of interlock groups from 4 to 14 - Removed migration support for versions before v8.1.0 (Doris) - ESP32 Increase number of interlock groups from 4 to 14 * Clean settings code * Increase ESP32 SwitchText to 28 * Fix long SwitchText message logging * Update Italian language * Platformio esp8266 2.6.3 * Platformio esp8266 2.6.3 * Update af_AF.h * Create fy_NL.h * Update fy_NL.h * Update platformio.ini * Update platformio_tasmota32.ini * Update platformio_tasmota_env.ini * Update platformio_tasmota_env.ini * Update platformio_tasmota_env32.ini * Update af_AF.h * Update fy_NL.h * Applied standardization to WiFi Changed Wifi and wifi to WiFi for the purpose of standardization * Corrected some Dutch words onmiddelijk requires a double L. i and e where wrongfully ordered in Windsnelheid. * Update nl_NL.h * update max31855 json temperature name publication For using the thermostat function, publication of the measured temperature was using the wrong name. Changed to the common used name. * Add read buffer function to TasmotaSerial * if pulse is not specified, let the lib use its default value * Preview of Berry language for Tasmota32 * sns_53_sml: make ebus correctly interpret signed values Signed-off-by: Sven Ebenfeld <[email protected]> * xsns_53_sml: add option to emit ebus sync only telegrams Signed-off-by: Sven Ebenfeld <[email protected]> * fix wrong test on pulse * add variant FY and build map files * Add files via upload * Update Tasmota_build_master.yml * Update Tasmota_build_master.yml * Consolidate ESP specific functions * Enable higher PWM frequencies for ESP32 LEDC peripheral of ESP32 allows higher PWM frequencies than the ESP8266. Higher frequencies come with reduced available bit depth (as by https://esphome.io/components/output/ledc.html). New max value of 50 kHz to include at least the frequency for 4-pin pwm fans and a bit more. * Add no_share_mask support, DevGroupSend channels in hex, AND/OR operators, Use dynamic log buffer * Add support for transistor-based output without DAC Add support for transistor-based output without DAC (arendst#10875) * Add SDM72 Modbus energy meter * Update xdrv_27_shutter.ino * Update xdrv_27_shutter.ino Fix shutter pwm command to include device number and update loggin information. * Prepare RISCV * Add DGR device map support * Use device map for received power item * Update xdrv_27_shutter.ino Add #define to turn off servo in ShutterPowerOff. * Rename DevGroupDevice to DevGroupTie * Use # prefix for color channel hex value * Fix DevGroupSend invert operator default value * Bring ModeSet value out of the for loop, this ModeSet is repeated for every other value on the webpage. * move renderer, add touch display slider * warning if ESP8266 firmware is too big * Bump version to 9.2.0.7 * Default disable ESP32 BLE due to instabilities * Fix NTC calculation for ESP32 Fix NTC calculation for ESP32 (arendst#10906) * Use Arduino ESP32-S2 RC6 * fix teleinfo standard mode * add 7 seg number font * better impl of ebus negative values * json subroutines, sendmail shortcut * tasmota32-ble * Tasmota variant ESP32 bluetooth * Add dummy fade2 (arendst#10916) * Delete xdrv_85_esp32_bluetooth.ino No need for this file and it takes up precious driver numbers. * correct order * Delete tasmota_template.h * Update tasmota_template.h * Update tasmota_configurations_ESP32.h * Update tasmota_template.h * bluetooth in repo firmware * bluettoth in branch release-firmware Co-authored-by: Theo Arends <[email protected]> Co-authored-by: Barbudor <[email protected]> Co-authored-by: Stephan Hadinger <[email protected]> Co-authored-by: s-hadinger <[email protected]> Co-authored-by: gemu2015 <[email protected]> Co-authored-by: Naoki Sawada <[email protected]> Co-authored-by: Simon Hailes <[email protected]> Co-authored-by: bovirus <[email protected]> Co-authored-by: crispy78 <[email protected]> Co-authored-by: RvDesign <[email protected]> Co-authored-by: Sven Ebenfeld <[email protected]> Co-authored-by: Thomas A <[email protected]> Co-authored-by: Paul C Diem <[email protected]> Co-authored-by: Norbert Richter <[email protected]> Co-authored-by: mikep1998 <[email protected]> Co-authored-by: Yves De Muyter <[email protected]>
Jason2866
added a commit
to Jason2866/Tasmota
that referenced
this issue
Feb 12, 2021
* Fix compile error on enable sensor debug * Update NeoPool support extended types * Fix redox unit * Fix hydrolysis unit * HA discovery via MQTT for BLE MI sensor devices * Add 'MI32Option5 1' to only accept sensors if they have a BLEAlias * Refactor TX2x sensor float outputs * Clear MI sensors when setting 'mi32Option5 1' * Document CmndSetoptionBase() * Fix warnings * Add light synonyms Add commands ``ChannelRemap``, ``MultiPWM``, ``AlexaCTRange``, ``PowerOnFade``, ``PWMCT``, ``WhiteBlend``, ``VirtualCT`` as synonyms for ``SetOption37, 68, 82, 91, 92, 105 and 106`` respectively * Fix compile error * Refactor dtostrfd * Refactor dtostrfd * Add Modbus errorcode * Add Zigbee SetOption synonyms Add commands ``ZbNameKey``, ``ZbDeviceTopic``, ``ZbNoPrefix``, ``ZbEndpointSuffix``, ``ZbNoAutoBind``, ``ZbNameTopic`` as synonyms for ``SetOption83, 89, 100, 101, 110 and 112`` respectively * Make USE_BLE_ESP32 the default for sensors. * belt and braces fix test failure * Fix PMS3003 sensor message lenght and CRC chk * Refactor TX2x sensor * Refactor ext print usage * Update changelog * Clarify file name functionality * rename + soime explaination * Add Sensor83 99 cmnd write to eeprom * Refactor some IP addresses * Refactor some IP addresses * Add support for ESP32 ``Module 5`` Wireless Tag Eth01 Add support for ESP32 ``Module 5`` Wireless Tag Eth01 (arendst#9496) * Add template place holders * Support trailing silence in buzzer tune Support trailing silence in buzzer tune (arendst#10694) * Remove unwanted persistance * Add commands ``BuzzerActive``, ``BuzzerPwm`` as synonyms for ``SetOption67, 111`` respectively Add commands ``BuzzerActive``, ``BuzzerPwm`` as synonyms for ``SetOption67, 111`` respectively * refactor dtostrfd * add L1MusicSync mode * cleanup * Use dgr name for MQTT if SO88 * Added macro for SO synonyms and MQTT * Add support for once off speed and fade * Add Sonoff L1 Music Sync control Add command ``L1MusicSync <0|Off>|<1|On>|<2|Toggle>, 1..10, 1..100>`` to control Sonoff L1 Music Sync mode sensitivity and speed (arendst#10722) * Initial support for ZStack 3 * Zibee add ``SetOption120 1`` or ``ZbEndpointTopic`` to add the endpoint as suffix in topic when using ``SetOption89 1`` * Respond to fade2 and speed2 * Merge fade2 with speed2 * Forgot changelog update * Change L1 music sync defaults * Revert "Merge pull request arendst#10736 from emontnemery/once_off_speed_fade_fix" This reverts commit aa8a23e, reversing changes made to f6111e9. * Update changelog * Zigbee more synonyms * Update changelog * Fix subStr related exceptions * Zigbee fix ZbNoAutoQuery synonym * XNRG7 ADE: Set Debug log level from 3 to 4 for ADE comms It is better for visualization while checking * In case of lost comms with PZEM, avoid losing Today Energy * Refactor Sensor83 cmnd * Add Sensor83 TakeOver cmnd * fix ip crashes * Update xdrv_13_display.ino * Add Sensor83 switch light relay cmnd * NeoPool cleanup polling * Refactor command line arguments * Refactor command line arguments * Refactor command line arguments * Refactor command line arguments * Update stale-actions.yml * Use espressif32 platformio 3.0.0 * Deleted duplicated entries in i18n.h The entry: #define D_ZIGBEE_NOT_STARTED "Zigbee not started" is duplicated in i18n.h from all language files. This produces a warning while compiling due to redefinition if changing the language in my_user_config.h * Return to previous output of IPADDRESS1 command * Update af_AF.h * change solo1 to release_v3.3-solo1-bd65eb8d1 * add and fix new string ops in rules * Add strict rule scale parameters * Change NeoPool Sensor83 to prop. cmnds * Add MAX31865 specific setting per sensor * Fix IrRemote_Full JSON message (arendst#10680) * Revert comment * Add demo for driver/sensor settings file storage (arendst#10786) * Fix IrRemote_Full JSON hash value (arendst#10680) * Fix IrRemote_Full JSON leading zeroes (arendst#10680) * ESP32-S2 * Oops * s2 toolchain * Refactor File Settings Demo * Add optional SSE demo * Prep support ESP32S2 * ESP32S2 toolchain is there * Cleanup webserver * refactor + bugfixes * Prep ESP32S2 * Fix ETH_ADDR related warning * Zigbee refactor sending ZCL packets * fix regression * Update xsns_62_esp32_mi.ino * Revert ESP32S2 ethernet defines * Prep ESP32S2 * Prep ESP32S2 * Prep ESP32S2 * Prep ESP32S2 * LITTLEFS.h casing for ESP32 * Publish more precise Free Heap metric * Zigbee command ``ZbScan`` to do an energy scan on each radio channel * Reduce code size * fix i2s audio * Update AudioOutputI2SNoDAC.cpp * Allows individual control of power and mode Power and mode must be individually controllable. Some devices may not function properly if the mode is also turned off when the power is turned off. see. crankyoldgit/IRremoteESP8266#1402 * Prep ESP32S2 * Add fragmentation translations * ESP32 increase number of relay GPIOs from 8 to 28 * Add Server to HTTP header (arendst#10831) * BLE_ESP32 (xdrv_79): Start scan as soon as we are connected to a device - saves missing a few adverts. iBeacon (xdrv_52): Clean up BLE_ESP32 based driver; add commands and advert timing. Remove HM10 code - it's in the other one..... * Add timeout initialise * Add additional #if for CONFIG_IDF_TARGET_ESP32 * update comment * Increase number of switch GPIOs from 8 to 28 * Increase number of button GPIOs from 4 to 8 * Allow BLE when ethernet only connected Allow BLE when ethernet only connected (arendst#10753) * ESP32 Increase number of interlock groups from 4 to 14 - Removed migration support for versions before v8.1.0 (Doris) - ESP32 Increase number of interlock groups from 4 to 14 * Clean settings code * Increase ESP32 SwitchText to 28 * Fix long SwitchText message logging * Update Italian language * Platformio esp8266 2.6.3 * Platformio esp8266 2.6.3 * Update af_AF.h * Create fy_NL.h * Update fy_NL.h * Update platformio.ini * Update platformio_tasmota32.ini * Update platformio_tasmota_env.ini * Update platformio_tasmota_env.ini * Update platformio_tasmota_env32.ini * Update af_AF.h * Update fy_NL.h * Applied standardization to WiFi Changed Wifi and wifi to WiFi for the purpose of standardization * Corrected some Dutch words onmiddelijk requires a double L. i and e where wrongfully ordered in Windsnelheid. * Update nl_NL.h * update max31855 json temperature name publication For using the thermostat function, publication of the measured temperature was using the wrong name. Changed to the common used name. * Add read buffer function to TasmotaSerial * if pulse is not specified, let the lib use its default value * Preview of Berry language for Tasmota32 * sns_53_sml: make ebus correctly interpret signed values Signed-off-by: Sven Ebenfeld <[email protected]> * xsns_53_sml: add option to emit ebus sync only telegrams Signed-off-by: Sven Ebenfeld <[email protected]> * fix wrong test on pulse * add variant FY and build map files * Add files via upload * Update Tasmota_build_master.yml * Update Tasmota_build_master.yml * Consolidate ESP specific functions * Enable higher PWM frequencies for ESP32 LEDC peripheral of ESP32 allows higher PWM frequencies than the ESP8266. Higher frequencies come with reduced available bit depth (as by https://esphome.io/components/output/ledc.html). New max value of 50 kHz to include at least the frequency for 4-pin pwm fans and a bit more. * Add no_share_mask support, DevGroupSend channels in hex, AND/OR operators, Use dynamic log buffer * Add support for transistor-based output without DAC Add support for transistor-based output without DAC (arendst#10875) * Add SDM72 Modbus energy meter * Update xdrv_27_shutter.ino * Update xdrv_27_shutter.ino Fix shutter pwm command to include device number and update loggin information. * Prepare RISCV * Add DGR device map support * Use device map for received power item * Update xdrv_27_shutter.ino Add #define to turn off servo in ShutterPowerOff. * Rename DevGroupDevice to DevGroupTie * Use # prefix for color channel hex value * Fix DevGroupSend invert operator default value * Bring ModeSet value out of the for loop, this ModeSet is repeated for every other value on the webpage. * move renderer, add touch display slider * warning if ESP8266 firmware is too big * Bump version to 9.2.0.7 * Default disable ESP32 BLE due to instabilities * Fix NTC calculation for ESP32 Fix NTC calculation for ESP32 (arendst#10906) * Use Arduino ESP32-S2 RC6 * fix teleinfo standard mode * add 7 seg number font * better impl of ebus negative values * json subroutines, sendmail shortcut * tasmota32-ble * Tasmota variant ESP32 bluetooth * Add dummy fade2 (arendst#10916) * Delete xdrv_85_esp32_bluetooth.ino No need for this file and it takes up precious driver numbers. * correct order * Delete tasmota_template.h * Update tasmota_template.h * Update tasmota_configurations_ESP32.h * Update tasmota_template.h * bluetooth in repo firmware * bluettoth in branch release-firmware Co-authored-by: Norbert Richter <[email protected]> Co-authored-by: Simon Hailes <[email protected]> Co-authored-by: Theo Arends <[email protected]> Co-authored-by: Stephan Hadinger <[email protected]> Co-authored-by: s-hadinger <[email protected]> Co-authored-by: lonmi <[email protected]> Co-authored-by: blakadder <[email protected]> Co-authored-by: Paul C Diem <[email protected]> Co-authored-by: Erik <[email protected]> Co-authored-by: Adrian Scillato <[email protected]> Co-authored-by: gemu2015 <[email protected]> Co-authored-by: Adrian Scillato <[email protected]> Co-authored-by: crispy78 <[email protected]> Co-authored-by: Barbudor <[email protected]> Co-authored-by: Naoki Sawada <[email protected]> Co-authored-by: bovirus <[email protected]> Co-authored-by: RvDesign <[email protected]> Co-authored-by: Sven Ebenfeld <[email protected]> Co-authored-by: Thomas A <[email protected]> Co-authored-by: mikep1998 <[email protected]> Co-authored-by: Yves De Muyter <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Version/revision of the library used
Current master
ab466e4
Describe the bug
Brand: Mitsubishi Electric, Model: MLZ-RX5017AS A/C (MITSUBISHI_AC)
Brand: Mitsubishi Electric, Model: SG153/M21EDF426 remote (MITSUBISHI_AC)
It is not possible to turn off the air conditioner with the above devices.
Example code used
IRrecvDumpV3
Expected behaviour
Hope we can turn it off with this library
Output of raw data from [IRrecvDumpV3.ino]
What brand/model IR demodulator are you using?
HX1838: However, there seems to be no problem with the decoded data.
I have followed the steps in the Troubleshooting Guide & read the FAQ
Yes
Has this library/code previously worked as expected for you?
No
The text was updated successfully, but these errors were encountered: