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

feat: Add more i35 sensors and switches. #142

Merged
merged 11 commits into from
Nov 28, 2024
Merged

feat: Add more i35 sensors and switches. #142

merged 11 commits into from
Nov 28, 2024

Conversation

rainwoodman
Copy link
Contributor

@rainwoodman rainwoodman commented Nov 27, 2024

  • Add i35 sensors, and differentiate sensors for i35 and i7470.
  • Show model names for i35 and i7470.
  • Add a default fan speed to workaround i35 always set fan speed to zero (maybe there is a better fix)

* speed count = 64
* some of the sensors do not apply
* set a default perentage to 50% when turning on. (because device does not remember last speed)
* added a few sensors.
* added model name look up for AWS devices i35 and 7470i
* fixed a few mis-typed type annotations.
@rainwoodman rainwoodman changed the title Better i35 support WIP: Better i35 support Nov 27, 2024
the setters will and shall internally update these 'local cached state' properties
anyway.
@rainwoodman rainwoodman changed the title WIP: Better i35 support feat: Add more i35 sensors and switches. Nov 27, 2024
@rainwoodman rainwoodman changed the title feat: Add more i35 sensors and switches. WIP: feat: Add more i35 sensors and switches. Nov 27, 2024
@rainwoodman rainwoodman changed the title WIP: feat: Add more i35 sensors and switches. feat: Add more i35 sensors and switches. Nov 27, 2024
Also avoid the first, immediate action because the cloud side state
is not updated till after 5 seconds.
@dahlb dahlb merged commit a814c50 into dahlb:main Nov 28, 2024
3 checks passed
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.

2 participants