-
Notifications
You must be signed in to change notification settings - Fork 300
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
AMI430: Fix switch heater parameters that return 0 or 1 as a string #3650
Conversation
Co-authored-by: Mikhail Astafev <astafan8@gmail.com>
and yaml config to match
Codecov Report
@@ Coverage Diff @@
## master #3650 +/- ##
==========================================
- Coverage 65.73% 65.72% -0.01%
==========================================
Files 226 226
Lines 30547 30547
==========================================
- Hits 20080 20078 -2
- Misses 10467 10469 +2 |
@peendebak I pushed a basic test and added support for enabling disabling the switch heater in the yaml simulator. I think this should be ready to merge now? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
bors merge
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
bors merge
🕐 Waiting for PR status (Github check) to be set, probably by CI. Bors will automatically try to run when all required PR statuses are set. |
1 similar comment
🕐 Waiting for PR status (Github check) to be set, probably by CI. Bors will automatically try to run when all required PR statuses are set. |
bors merge |
The parameters from the AMI430 switch heater return a string such as
0\n
. This was parsed by stripping and then casting to bool. Butbool('0')
is True, which is not what the parameter should return.