search

pulquero / BatteryAggregator

MIT License
36 stars 7 forks source link

Rename Battery Aggregator and other little Things. #4

Closed Scheppo closed 1 year ago

Scheppo commented 1 year ago

Hi Pulquero, the „exclude Battery“ Option is working fine, thx.

Few Things:

  1. I Will Change the Battery Aggregator Name in „400AH BAT“ because my battery’s are 280 and 120 Ah.
  2. The total AH is not adjustable. 79643372-7990-4195-9940-5A2CAE4E2F89
  3. There are no Data from the Can Battery here D3198D54-6BFB-4350-9ED7-EFFCE6A90A80

Maybe I have not seen the Options to Change this, sorry.

pulquero commented 1 year ago

If you could run dbus-spy and attach screenshots of your battery services then I can check which dbus paths I am missing and add them. Total AH should be computed from the individual battery AHs, but maybe your batteries don't publish that?

Scheppo commented 1 year ago

Ok, I can see that the smartshunt don’t send the Capacity, this is strange.

Here are the Infos for the 280ah can Battery: I need only the Min and Max Cellvoltage here. AD1D8BA2-E599-4966-9821-7BFD8D7E7C84

Here ist the 280ah Can Battery over the Smartshunt: 72DD86F8-2E4D-453A-99BC-80944E1F904E

Here ist the 120ah Battery only over Smartshunt: I see no Capacity Info, maybe you can use the Name for the info? 4CCD1E41-059F-44E2-99F4-07485ABCF379

I have cut some first entry’s for Alarm to fit in one Picture.