Matt Redfearn

Results 50 comments of Matt Redfearn

> The _stub functions are for testing without inverters connected; they should not be changed to STATE_UNKNOWN as far as I understand their purpose. @WillCodeForCats is that right? As I'd...

@binsentsu There are a lot of changes proposed in multiple MRs here. How would you like to proceed (i.e. which order should we try to get things merged?)

> As far as multiple PRs, my vote is _not_ to pull in one massive change like this, but stick with the smaller incremental changes because it makes it easier...

> I deleted the _stub functions completely from my multiple inverters fork (which is just a fork with #12 added) and have not experienced that issue with the energy dashboard...

Hey @nerfherder do you think you could rebase the changes, instead of a merge? We've ended up with 23 commits in here most of which are fixups, but it's impossible...

@WillCodeForCats :-( I'm happy to attempt the work, if you can do the testing on your system? What's the gist of the change, support for multiple unit IDs, right?

Implemented in #38

Update the integration through HACS, remove and re-add the integration, and when you configure it tick the "read battery 1 information" checkbox. You should then get sensors for the battery.

@ministryofsillywalks for now, I ended up using templated sensors, but I am thinking about adding some calculated sensors to the integration itself. I've got so many PRs open I'm waiting...

@ministryofsillywalks All of the "power" sensors I use to populate a https://github.com/reptilex/tesla-style-solar-power-card. ![Screenshot_2021-10-04_14-03-28](https://user-images.githubusercontent.com/12936891/135856611-cf3bab4e-9b4c-474b-9272-ec7141a882af.png) I have populated the energy dashboard as shown here: ![Screenshot_2021-10-04_14-07-00](https://user-images.githubusercontent.com/12936891/135856820-5ee85521-be68-44ae-b855-9ce9e610d4b3.png) The import / export from grid are...