Skip to content
This repository has been archived by the owner on Jun 9, 2023. It is now read-only.

feat: support smartmode params #11

Merged
merged 5 commits into from
Jan 13, 2022
Merged

feat: support smartmode params #11

merged 5 commits into from
Jan 13, 2022

Conversation

barlock
Copy link
Contributor

@barlock barlock commented Mar 28, 2021

In support of: mikelawrence/senseme-hacs#33

Adds support for setting and reading smartmode parameters

@mikelawrence
Copy link
Collaborator

coolminspeed range is 0-6 not 0-7.
coolmaxspeed range is 1-7 not 0-7.

@mikelawrence
Copy link
Collaborator

I find it interesting that when auto comfort is set to cooling and you manually set the fan speed to something outside the range of coolminspeed or coolmaxspeed, coolminspeed will track the fan speed when lower and coolmaxspeed will track the fan speed when above.

@barlock
Copy link
Contributor Author

barlock commented Apr 19, 2021

Bump here in case you didn't see that I fixed your comments. If you're still busy, no worries, been running this fine from my fork.

@spf13
Copy link

spf13 commented Dec 1, 2021

I'd really like to use this functionality and understand that @mikelawrence isn't active right now.

Is there an easy way to use @barlock's fork from home assistant? I've been searching in the docs and via Google and I'm not seeing any way to do this without uninstalling the integration and then installing from another source, but doing that would require removing all my integrations.

Or alternatively would @mikelawrence be willing to add @barlock as a maintainer to help move this project forward a bit?

I'm happy to help with the occasional review if that helps.

@barlock
Copy link
Contributor Author

barlock commented Dec 22, 2021

I would happily accept a maintainership.

You can use my fork in home assistant by setting up my fork of senseme-hacs as a custom repo in hacks https://github.com/barlock/senseme-hacs. I've been using it happily ever since I made the PRs.

@spf13
Copy link

spf13 commented Dec 22, 2021 via email

@bdraco
Copy link
Owner

bdraco commented Jan 13, 2022

Thanks @barlock 👍

@bdraco bdraco merged commit a0576f3 into bdraco:master Jan 13, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants