Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

add sensor with 'fault' property to dmaker.derh.22ht and dmaker.derh.50l #2362

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

damex
Copy link

@damex damex commented Feb 12, 2025

fault property can tell if compressor is at fault
or if water tank is full.
reporting about water tank in dehumifier being full is
almost mandatory when you have smart dehumidifier.

@al-one
Copy link
Owner

al-one commented Feb 13, 2025

I do not recommend specifying such a high update frequency for all dehumidifiers, as it may not be suitable for everyone. Perhaps you can customize your device through configuration.yaml.

xiaomi_miot:
  device_customizes:
    your.derh.model:
      chunk_coordinators:
        - {interval: 5, props: fault}
        - {interval: 20, props: on,mode,target_humidity,fan_level}
        - {interval: 25, props: relative_humidity,temperature}
        - {interval: 300, props: filter_life_level,filter_left_time,filter_used_time}

@damex
Copy link
Author

damex commented Feb 13, 2025

I do not recommend specifying such a high update frequency for all dehumidifiers, as it may not be suitable for everyone.

please check again which property gets such high update frequency.
that's a failure (!) notification which actually should be done with a service/entity listening to its changes it but it is not possible here.

it could be catastrophic if if compressor fails or some other failure happens and person wouldn't know that it happened until minutes or hours later.

i already customized my devices and thought it is a good idea to provide same solution for everybody else.

@al-one
Copy link
Owner

al-one commented Feb 13, 2025

  1. Not all dehumidifiers have a fault property.
  2. Not all dehumidifiers' fault property can reflect compressor/water tank issues.

You should add this customization for specific device model rather than *.derh.*.

@damex damex force-pushed the extra_property_add_fault_property_for_dehumidifiers branch from 897228b to 17fd4eb Compare February 13, 2025 09:38
@damex
Copy link
Author

damex commented Feb 13, 2025

i have here dmaker.derh.22ht and dmaker.derh.50l dehumidifiers.
added sensor with 'fault' property instead of specifying that property for all dehumidifiers.

image
image
image

@damex damex changed the title add 'fault' property with 5s refresh to all dehumidifiers add sensor with 'fault' property to dmaker.derh.22ht and dmaker.derh.50l Feb 14, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants