muart-group / esphome

ESPHome fork for development of the mitsubishi_uart component. Check out the documentation for more info:
https://muart-group.github.io/
10 stars 1 forks source link

Fan speed names don't align #19

Closed Sammy1Am closed 4 months ago

Sammy1Am commented 7 months ago

The capabilities packet for my air handler says it should support 3 fan speeds, but the "Actual Fan Speed" sensor is reporting speeds like "Very Low" and "Quiet".

Setting the climate control to "Low" results in "Quiet", "Medium" is "Low" (IIRC).

Need to make sure these all line up properly.

KazWolfe commented 7 months ago

It gets worse. My air handler supports three fan speeds, but:

image

It cycles between OFF, Quiet, Very Low, Low, and Powerful. I haven’t explicitly tried forcing fan speeds for mapping purposes yet, but it seems to be a mess.

My mapping however:

Meanwhile, my 5-speed wall unit:

The Very Low mode seems to only be engaged on my air handler after the heating setpoint has been met, and likely is controlled by the Thermal Off Fan setting.

Sammy1Am commented 7 months ago
Supported Speeds: 5 Set Speed "Actual" Speed
Any 0 Unit is in idle
? 1 Haven't confirmed this one yet
1 6 !
2 2
3 3
5 4 (this is fine, there isn't a speed 4 🤷‍♂️ )
6 5
Supported Speeds: 3 Set Speed "Actual" Speed
Any 0 Unit is in idle
? 1 "Actual" 1 is present during pre-heating
2 2
3 3
5 4 (this is fine, there isn't a speed 4 🤷‍♂️ )

So, weirdness with there being no set-speed 4 aside:


I think it's not unreasonable to maybe change the names of "actual" speeds 2-6 so that they do match up with the set speed names. I'm in favor of renaming them to match something more traditional like "Quiet,Low,Medium,High,Very High" (we can preserve the "powerful" in code comments :P )

"Very Low" would remain a special speed that only shows up in the "Actual Speed" sensor.

KazWolfe commented 4 months ago

Closing as resolved in #42.