search

mkaiser / Sungrow-SHx-Inverter-Modbus-Home-Assistant

Sungrow SH Integration for Home Assistant for SH3K6, SH4K6, SH5K-20, SH5K-V13, SH3K6-30, SH4K6-30, SH5K-30, SH3.RS, SH3.6RS, SH4.0RS, SH5.0RS, SH6.0RS, SH5.0RT, SH6.0RT, SH8.0RT, SH10RT, SH5.0RT-20, SH6.0RT-20, SH8.0RT-20, SH10RT-20, SH5.0RT-V112, SH6.0RT-V112, SH8.0RT-V112, SH10RT-V112, SH5.0RT-V122, SH6.0RT-V122, SH8.0RT-V122, SH10RT-V122, SH4.6R
328 stars 86 forks source link

combined Dashboards into single tabbed one #48

Closed andi-blafasl closed 1 year ago

andi-blafasl commented 1 year ago

I combined the two dashboard yaml-files into a single tabbed one: image Nice Icons make it easy recognisable and compact in the view. Makes updating easier, no need to check two files. It also saves space on the sidebar. Hope you like it.

mkaiser commented 1 year ago

Hi andi-blafasl,

that is awesome. I did not know that you can use tabs in a dashboard.

This opens multiple new opportunities: separate tabs for

One question before I merge:

What did you intend to make with the conditional battery charging card?

Louisbertelsmann commented 1 year ago

Already made a similar version of „history“ for my dashboard. If you want to, I can post it here this evening

mkaiser commented 1 year ago

Already made a similar version of „history“ for my dashboard. If you want to, I can post it here this evening

yes please :)

andi-blafasl commented 1 year ago

What did you intend to make with the conditional battery charging card?

I did not change anything from your .yaml files. Just combided them. This Conditional Card is already in your code 😉 After the last update i was also surprised by this new card because it was shown in the config but missing on the dashboard. Didn't know about this feature and was searching a few hours (minutes ;-) why it was not showing. 😁 As far as I understand, the card is only shown if the battery is charging. But the hard limit to 5kW is strange and the gauge would be also nice to see while discharging. Maybe we can refactor the dashboards like you mentioned above and make the overview easier.

Louisbertelsmann commented 1 year ago

I have a bit of free time right now and am working on it a bit more. Had to add two more sensors for it to make sense (Monthly and yearly battery charge) I don‘t know if it‘s possible, but can you merge the newest version of my fork into main?

mkaiser commented 1 year ago

What did you intend to make with the conditional battery charging card?

I did not change anything from your .yaml files. Just combided them. This Conditional Card is already in your code 😉 After the last update i was also surprised by this new card because it was shown in the config but missing on the dashboard. Didn't know about this feature and was searching a few hours (minutes ;-) why it was not showing. 😁 As far as I understand, the card is only shown if the battery is charging. But the hard limit to 5kW is strange and the gauge would be also nice to see while discharging. Maybe we can refactor the dashboards like you mentioned above and make the overview easier.

oh yeah, I remember putting that in sometime ago, but deleted it locally.... when I made a diff it popped up.... I guess I am very confusing sometimes.....

I have a bit of free time right now and am working on it a bit more. Had to add two more sensors for it to make sense (Monthly and yearly battery charge) I don‘t know if it‘s possible, but can you merge the newest version of my fork into main?

wil have a look at that, soon!

Louisbertelsmann commented 1 year ago

wil have a look at that, soon!

Take your time. Am still working on it. Until now I added the total pv generation for each month, the total export power for each month and the total pv generation for the current and the last year.

Louisbertelsmann commented 1 year ago

You can look at the new version here. I added some sensors and created tabs in the PV page.