Add options for rounding vs. truncating, smaller granularity than whole amps#515
Add options for rounding vs. truncating, smaller granularity than whole amps#515RichieB2B wants to merge 1 commit intongardiner:mainfrom
Conversation
|
I guess it depends what the goal is here. Edit: I see I said the same thing a year ago, and that the content of the PR has changed since then. Oops. |
|
I actually stopped using this patch. I couldn't get the TWCmamager to offer fractions to the TWC. I might give it another try when there is more sun here. |
|
My preference is to have TWCManager use round() based on the available information provided to it. I use the api/addConsumptionOffset method and can easily add buffers to that value to avoid drawing grid power. But that's largely academic at the moment... still having issues getting the FleetAPI working. But thanks @RichieB2B for the tutorial link you provided in the other thread. The problem is with Tesla approving the app request. |
I found that using round() works much better for PV tracking. With round() I stay within 0.5A of the target. With 3-phases 240V that is a maximum inaccuracy of 360W instead of 720W when using int(). With single phase 120V it is 60W vs 120W so it doesn't matter as much.