search

gysmo38 / mitsubishi2MQTT

Mitsubishi to MQTT with ESP8266 module
GNU Lesser General Public License v2.1
375 stars 134 forks source link

Change via HVAC remote get overwritten. #116

Closed Stendec1 closed 1 year ago

Stendec1 commented 3 years ago

When changing a setting via the HVAC remote, the value is published ok via ha_settings_topic, ha_state_topic and HA but afterwards overwritten by the old value.

brackw commented 3 years ago

By default SwiCago's library doesn't let remote updates propagate and reverts them back (by design he did not want remotes to override automation) but he did code in the option for it, it's just not brought out in mitsubishi2MQTT.

hp.enableExternalUpdate(); can be called to let remote commands stick.

You must have updated SwiCago's library to rev#164 or higher, as there was a bug in earlier versions that caused it not to work.

I tossed the command into my local code and it works as expected.

Someone should actually add it as a user defined option via the web interface like debug or useFahrenheit and supportHeatMode.

Stendec1 commented 3 years ago

@brackw thanks for the quick answer. I'll have a look at this.

Stendec1 commented 3 years ago

@brackw I added this to the setup() section and it did the trick! The main reason I want to have the "remote" updates stick is that I want the iSee sensor to control the absence detection. Adding it as a user defined option via the web interface might indeed be a welcome option.

Rolf-M commented 3 years ago

@Stendec1 could you add the exact change you did (the exact file and lines). I'll need to control the AC with the normal remote as well. Maybe then it'll be easier for someone to add it to the webinterface. This would be a cool feature. The simple infrared remote is a main part of the "woman acceptance factor" for me :-)

Rolf-M commented 3 years ago

@Stendec1, I think you mixed something up.... But I've seen, that the hp.allowexternalupdate() call is in the latest version from feb. 24th. so I think this issue may be closed.

Now somebody wanting to have the infrared remote disabled would need to comment this out.

Stendec1 commented 3 years ago

@Rolf-M

Sorry copied the wrong part before. This should be it:

Just put "hp.enableExternalUpdate();" near the end of the setup() section, near line 193:

// RSC allow remote and iSensor to update settings
hp.enableExternalUpdate();

} else { dnsServer.start(DNS_PORT, "*", apIP); initCaptivePortal(); } initOTA(); }

gysmo38 commented 1 year ago

Old issue. Try new version and open a new one if the issue is still there.