OpenBK7231T_App icon indicating copy to clipboard operation
OpenBK7231T_App copied to clipboard

CHT sensor refactoring

Open NonPIayerCharacter opened this issue 1 year ago • 4 comments

fixed negative temperature on CHT8310 (earlier was treated as unsigned byte) add CHT8315 (have no device to test, waiting for response on elektroda) build driver for bl602

NonPIayerCharacter avatar Jun 04 '24 15:06 NonPIayerCharacter

Confirmed as working on BK7231N+CHT8315, i can confirm that it is working on my BK7231N+CHT8310

NonPIayerCharacter avatar Jun 05 '24 15:06 NonPIayerCharacter

Hello, is it ready to merge? @divadiow are you able to check any of this?

openshwprojects avatar Jun 12 '24 09:06 openshwprojects

I can only test on CHT8310. but: https://www.elektroda.com/rtvforum/viewtopic.php?p=21108522#21108522

divadiow avatar Jun 12 '24 09:06 divadiow

is this good or is more testing required?

divadiow avatar Jun 24 '24 20:06 divadiow

Alert appears to be working as expected. Executing CHT_Alert 0.3 5 2 right before PinDeepSleep, the module wakes up, as soon as temperature is 0.3 degrees lower or higher, than last reading. Ready to merge i believe.

NonPIayerCharacter avatar Jul 28 '24 07:07 NonPIayerCharacter

Thank you, I don't have ability to test it but I will merge it anyway. Hopefully it was tested well.

openshwprojects avatar Jul 28 '24 11:07 openshwprojects

:tada: This PR is included in version 1.17.650 :tada:

The release is available on GitHub release

Your semantic-release bot :package::rocket:

github-actions[bot] avatar Jul 28 '24 11:07 github-actions[bot]