Skip to content

adding light capability#7

Open
rrelande wants to merge 1 commit into
dudanov:mainfrom
rrelande:adding-light
Open

adding light capability#7
rrelande wants to merge 1 commit into
dudanov:mainfrom
rrelande:adding-light

Conversation

@rrelande

Copy link
Copy Markdown

Adding support for querying display status

@dudanov dudanov left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks! Which air conditioner model uses this feature?

Comment on lines +126 to +127
/* DISPLAY LIGHT */
Display getLight() const { return static_cast<Display>(this->m_getValue(14,7,4)); }

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
/* DISPLAY LIGHT */
Display getLight() const { return static_cast<Display>(this->m_getValue(14,7,4)); }
/* DISPLAY LIGHT */
Display getLight() const { return static_cast<Display>(this->m_getValue(14, 7, 4)); }

Comment on lines +67 to +73
/// Enum for display light setting
enum Display : uint8_t {
/// Display is off
DISPLAY_OFF=7,
/// Display is on
DISPLAY_ON=0
};

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
/// Enum for display light setting
enum Display : uint8_t {
/// Display is off
DISPLAY_OFF=7,
/// Display is on
DISPLAY_ON=0
};
/// Enum for display light setting
enum Display : uint8_t {
/// Display is off
DISPLAY_OFF = 7,
/// Display is on
DISPLAY_ON = 0,
};

@rrelande

rrelande commented May 2, 2024

Copy link
Copy Markdown
Author

Thanks! Which air conditioner model uses this feature?

at least Midea air conditioners can switch off the display.

@randyoo

randyoo commented Apr 24, 2026

Copy link
Copy Markdown

Hi @dudanov -- is this PR waiting for @rrelande to accept the suggested changes in spacing? I'd love to see this PR implemented, so it can eventually be rolled into ESPHome.

And thanks for your work in helping my ESP talk to my mini split! :)

parkghost referenced this pull request in parkghost/esphome Jun 15, 2026
ESPHome 2026.2.0 selective Arduino compilation requires explicit WiFi
library declaration to make Network headers available for MideaUART.
@SoCuul

SoCuul commented Jun 15, 2026

Copy link
Copy Markdown

@dudanov I can attest that I've tried this PR with my own air conditioner as well (some Danby model, I think this one) and it worked exactly as intended with no further changes required.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants