Closed steepestdescent closed 4 years ago
Thanks for the PR. very nicely commented. i had to manually merge it, because your PR did weird stuff like replace whole files, probably due to the space characters getting replaced. i wish there were a way to get git to ignore those.
sorry again for the big delay
Hi Sam,
Kind of you to say!
Warm regards,
Tim
On Tue, Dec 17, 2019 at 5:08 PM lincomatic notifications@github.com wrote:
Thanks for the PR. very nicely commented. i had to manually merge it, because your PR did weird stuff like replace whole files, probably due to the space characters getting replaced. i wish there were a way to get git to ignore those.
sorry again for the big delay
— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/lincomatic/open_evse/pull/109?email_source=notifications&email_token=AJ5GRVLRMCDQ4I2N67TRPT3QZFSWLA5CNFSM4JFUBG3KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEHEMLSI#issuecomment-566805961, or unsubscribe https://github.com/notifications/unsubscribe-auth/AJ5GRVKJPLPN4BPEEOKXX6DQZFSWLANCNFSM4JFUBG3A .
This pull request adds the feature Heartbeat Supervision. $SY , $GY as in SY heartbeatinterval hearbeatcurrentlimit If you do not send open_evse a heartbeat more frequently than every (uint16_t)heartbeatinterval seconds then open_evse will reduce system ampacity to (uint8_t)hearbeatcurrentlimit amperes
Testing still in progress.
Plays nice with TEMPERATURE_MONITORING. Compiles OK, but consumes incremental ROM and RAM. Testing still in progress.