kwindrem / GuiMods

VenusOS GUI enhancements and modifications
206 stars 27 forks source link

MIssing detail from SmartSolar RS 450/100 #53

Open morphias2004 opened 2 years ago

morphias2004 commented 2 years ago

Just installed GuiMods and am seeing the following for the MPPT:

image

This is on the Cerbo GX I was just working on you with in the SetupHelper install issue thread.

The Cerbo is now on v2.85.

The MPPT is connected via VE.Can

image

I'm wondering if it's because the RS models have 2 internal MPPT's? The data displayed in the sum totals for the 2 trackers combined.

Let me know any additional details you need me to get.

kwindrem commented 2 years ago

Moved from SetupHelper to here since it's a GuiMods issue not SetupHelper.

RS MPPTs have up to 4 trackers and hence up to 4 sets of voltage and PV power parameters. There is a common (total) PV power parameter as well as an output (DC bus) power parameter. PV currents are calculated from PV power and PV voltage.

Currently the flow overview pages display name, PV power, voltage and current for between 2 and 3 PV chargers on three separate lines (name, PV power, PV voltage/current) and up to 7 with only name and PV power on a single line.

Also, the PV Charger detail page needs separate lines for each tracker.

Working on a design now. Stay tuned.

morphias2004 commented 2 years ago

Legend!

I'm guessing the field that is coming ??V NaNa is because the field mapping is slightly different for the RS?

kwindrem commented 2 years ago

Yes, for single tracker units it's /Pv/V and /Yield/Power. For multiple trackers it's /Pv/n/V and /Pv/n/P where n is a 0-based number indicating the specific tracker.

V?? indicates my code finds an invalid dbus parameter for voltage.

NaN means "not a number" which results in attempting to use an invalid number in some math (I = P / V in this case). I should be testing to make sure both P and V are valid but I do not.

kwindrem commented 2 years ago

I have pushed a beta version of GuiMods that supports multiple trackers on PV chargers. You can download this from GitHub or switch the GuiMods branch to "beta" in PackageManager.

The tracker details are shown only on the PV Charger Details page.

The voltage and current on the flow overview tile are replaced by "multiple trackers" since there could be up to 4 trackers and no space to display them. Adding specific tracker in formation to the PV Charger tile is possible with a lot more work and some additional run-time processing.

I have no way of testing the changes to please see if it works for you and report back. Thanks.

morphias2004 commented 2 years ago

Thanks mate! 👍

It will be a few weeks before I have the vehicle back to test it out, but I'll update you as soon as I do.