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 support for air conditioner AP10TW1RLR-N 006-200 #97

Closed
wants to merge 4 commits into from

Conversation

jonchampagne
Copy link

Been using this dictionary config for a week or so and it seems to be accurate.

Copy link
Owner

@oyvindwe oyvindwe left a comment

Choose a reason for hiding this comment

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

Thank your very much for your contribution ❤️

I left one comment.

device_class: switch
# Inlet temperature?
- property: f_temp_in
device_class: temperature
Copy link
Owner

Choose a reason for hiding this comment

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

Needs to be:

  sensor:
    device_class: temperature
    unit: °C # or °F

Maybe best to just leave without mapping until issue #100 is fixed

@smaddock
Copy link
Contributor

Interesting, I had just duplicated the 006-201 data dictionary for my 006-200 (almost identical to the one here) which also has f_temp_in mapped as a property, and my only complaint was I couldn’t reference it in sensor cards.

@smaddock
Copy link
Contributor

Okay to my previous comment, I set f_temp_in as:

- property: f_temp_in
  icon: mdi:temp
  climate:
    target: current_temperature

And then created a sensor template helper to get me the sensor data I wanted:

{{ state_attr("climate.portable_air_conditioner", "current_temperature") }}

@smaddock
Copy link
Contributor

A few other notes testing your config @jonchampagne:

  • daily_energy_kwh is always zero on my model, so I'd removed it
  • t_eco does nothing on my model, I try setting it but the API keeps returning zero, so I'd removed it
  • It looks like you're specifying non-core icons for t_sleep and t_super; you might want to change those to mdi:power-sleep and mdi:fast-forward respectively just for compatibility
  • I'll submit a separate PR to get the Sleep switch added to the i18n options
  • t_fan_mute is valid on my model, however it is redundant with the built-in "Disable beep" option when you configure a device with this integration, so probably only worth adding if you enable and disable the beep often. If you want to add, it can be set as:
- property: t_fan_mute
  icon: mdi:volume-variant-off
  switch:
    device_class: switch

@smaddock
Copy link
Contributor

@jonchampagne if you can’t or don’t want to make these changes I can submit a new PR as it looks like @Go2Engle also has this model, but I’d rather you get the credit for being the first to submit.

@oyvindwe
Copy link
Owner

Superseeded by #105

I'll make sure to credit both of you as well in the release notes, @jonchampagne and @smaddock :)

@oyvindwe oyvindwe closed this Aug 21, 2024
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.

3 participants