Skip to content

Add humidity control support to climate entity#141

Open
mnaser wants to merge 1 commit intoapetrycki:masterfrom
mnaser:feature/climate-humidity-control
Open

Add humidity control support to climate entity#141
mnaser wants to merge 1 commit intoapetrycki:masterfrom
mnaser:feature/climate-humidity-control

Conversation

@mnaser
Copy link
Copy Markdown

@mnaser mnaser commented Dec 26, 2025

Summary

  • Advertise existing set_humidity() functionality via ClimateEntityFeature.TARGET_HUMIDITY
  • Only enable humidity control when ctSystemCapHumidification capability is present
  • Set humidity range to 25-65% (matching Daikin humidifier limits)

Test plan

  • Verify humidity slider appears in climate entity card when system has humidification capability
  • Test setting humidity via the slider updates the thermostat
  • Verify humidity target is properly loaded on startup and updates

🤖 Generated with Claude Code

@mnaser mnaser force-pushed the feature/climate-humidity-control branch 2 times, most recently from 216142c to 88f3501 Compare December 26, 2025 19:28
- Advertise existing set_humidity() via ClimateEntityFeature.TARGET_HUMIDITY
- Only enable when ctSystemCapHumidification capability is present
- Set humidity range to 25-65% (matching Daikin humidifier limits)
- Update local state immediately after set_humidity call

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Signed-off-by: Mohammed Naser <mnaser@vexxhost.com>
@mnaser mnaser force-pushed the feature/climate-humidity-control branch from 88f3501 to e6a841f Compare December 26, 2025 19:31
@mnaser
Copy link
Copy Markdown
Author

mnaser commented Dec 26, 2025

image image

Tested and working, seeing the update via the application.

@mnaser mnaser marked this pull request as ready for review December 26, 2025 19:35
@oneWaveAdrian
Copy link
Copy Markdown

Looks great, would also close #143 – what are we waiting for?

@mnaser
Copy link
Copy Markdown
Author

mnaser commented Jan 29, 2026

FYI I have been using this for quite sometime successfully (added automation to adjust based on outdoor temperature to avoid condensation and it's working great).

@hansdg1
Copy link
Copy Markdown

hansdg1 commented Feb 13, 2026

I just got my thermostat hooked up to the humidifier last week. This functionality is exactly what I'm looking for. Will be trying it out this weekend!

@mnaser
Copy link
Copy Markdown
Author

mnaser commented Feb 13, 2026

@apetrycki do you have a chance to look at this?

Copy link
Copy Markdown

@oneWaveAdrian oneWaveAdrian left a comment

Choose a reason for hiding this comment

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

One question, otherwise looks good (to me).

Comment on lines +484 to +485
if self.thermostat.get("ctSystemCapHumidification", False):
features |= ClimateEntityFeature.TARGET_HUMIDITY
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Curious why this "out of pattern" conditional and not amend the flag to the full SUPPORT_FLAGS list in line 216-222?

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

The goal is only to advertise the feature if the unit is advertising a humidifier capability, since not all units have one integrated to them.

mrvollger pushed a commit to mrvollger/daikinskyport that referenced this pull request Mar 26, 2026
Cherry-picked from upstream PR apetrycki#141 by mnaser
_attr_temperature_unit = UnitOfTemperature.CELSIUS
_attr_fan_modes = [FAN_AUTO, FAN_ON, FAN_LOW, FAN_MEDIUM, FAN_HIGH, FAN_SCHEDULE]
_attr_min_humidity = 25
_attr_max_humidity = 65
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.

Can you explain why these are hard coded values? I haven't looked into humidity control in HA. What are these used for?

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