Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add native support to other version Must PV (1/3/5/10kw) #39

Open
3 tasks done
Benj-HaRRison opened this issue Sep 21, 2024 · 7 comments
Open
3 tasks done

Add native support to other version Must PV (1/3/5/10kw) #39

Benj-HaRRison opened this issue Sep 21, 2024 · 7 comments

Comments

@Benj-HaRRison
Copy link

Benj-HaRRison commented Sep 21, 2024

Checklist

  • I have filled out the template to the best of my ability.
  • This only contains 1 feature request (if you have multiple feature requests, open one feature request for each feature request).
  • This issue is not a duplicate feature request of previous feature requests.

Is your feature request related to a problem? Please describe.

This HA Integration successfully working for most of Must PV1800 models for watching to sensors. BUT this integration have have some limits for user input for another MUST devices (because this HA integration developed only to Must PV1800 3kW).

Describe the solution you'd like

If you change MAXIMUM values x5 for all user inputs - your integration successfully will working with other MUST PV1800 devices to config too (not only for sensor monitoring)

Describe alternatives you've considered

My device: Must 5kw, PV1800

Additional context

Examples (this examples for MY device):

  • ID "must_energy_inverter_battery_low_voltage" have maximum value "24", but other invertors have high of "24" - in my case i want set "42";
    If you change minimum value to "12" and maximum value to "85" - your integration will working with any PV1800.

  • ID "must_energy_battery_stop_charging_voltage" have maximum value "29", but other invertors have high of "29" - in my case i want set "55,8";
    If you change minimum value to "12" and maximum value to "110" - your integration will working with any PV1800.

  • ID "must_energy_battery_stop_discharging_voltage" have maximum value "29", but other invertors have high of "29" - in my case i want set "45,1";
    If you change minimum value to "12" and maximum value to "90" - your integration will working with any PV1800.

  • ID "must_energy_float_voltage" have maximum value "29,2", but other invertors have high of "29,2" - in my case i want set "56,1";
    If you change minimum value to "12" and maximum value to "110" - your integration will working with any PV1800.

  • ID "must_energy_battery_low_voltage" have maximum value "24", but other invertors have high of "24" - in my case i want set "34"
    If you change minimum value to "12" and maximum value to "70" - your integration will working with any PV1800.

  • ID "must_energy_grid_max_charger_current_set" have maximum value "30", but other invertors have high of "30" - in my case i want set "60"
    If you change minimum value to "10" and maximum value to "120" - your integration will working with any PV1800.

@p4block
Copy link

p4block commented Oct 31, 2024

Yep, it's working great with my 5kW 48V inverter but all the sliders stop me from configuring it. It's in a quite remote location and well, it would be great if it let me set them :D

I think the simplest way would be to let users configure the desired voltage when the integration is added?

@mukaschultze
Copy link
Owner

Yep, it's working great with my 5kW 48V inverter but all the sliders stop me from configuring it. It's in a quite remote location and well, it would be great if it let me set them :D

I think the simplest way would be to let users configure the desired voltage when the integration is added?

What's the model of your inverter? If you could paste the manual here it would be helpful.

The issue with fixing the voltages is that I don't know if they follow some sort of arithmetic progression or not. I don't know if the voltages for the 48V are exactly double of the 24V.

It doesn't need to be user-configured, the inverter itself reports whether it is 12V, 24V or 48V.

@p4block
Copy link

p4block commented Oct 31, 2024

@mukaschultze thanks for the quick response!

It's this one, MP1900
image

Let me know if you need me to do any tests. The manual is sadly sitting on top of the inverter. I will post it here if you can't find it, when I get to go the location again (maybe this weekend)

@p4block
Copy link

p4block commented Nov 1, 2024

@mukaschultze
Copy link
Owner

@mukaschultze drive.google.com/file/d/1MYyRXOiXarxxhc5QiE7EMKxLuIbvSiIB/view?usp=drivesdk

I scanned the manual, hope it helps

Thank you very much @p4block, this helps a lot

@Goolov
Copy link

Goolov commented Nov 28, 2024

Hello!
I use integration to connect Must PV18-5548 PREM.
The battery charging current of the specified inverter is from 0A-100A
But in the HA interface I can set 20A or 30A.
Is it possible to remove these restrictions in the HA interface?

@sergeysaley
Copy link

Hi there!
I researched my Must PV19-6248 EXP and found several registers that might be interesting for your project

To add:

battery state of charge (undocumented, comes from battery bms)
address: 113
unit_of_measurement: "%"
register_type: holding

battery state of health (undocumented, comes from battery bms)
address: 114
unit_of_measurement: "%"
register_type: holding

PV1 charger current (for older models - pv charger current)
address: 15207
register_type: holding
unit_of_measurement: "A"
accuracy_decimals: 1
filters:
multiply: 0.1

PV2 charger current (undocumented, but works)
address: 16207
register_type: holding
unit_of_measurement: "A"
accuracy_decimals: 1
filters:
multiply: 0.1

PV1 charger power (for older models - pv charger power)
address: 15208
register_type: holding
unit_of_measurement: "W"
device_class: power
accuracy_decimals: 1

PV2 charger power (undocumented, but works)
address: 16208
register_type: holding
unit_of_measurement: "W"
device_class: power
accuracy_decimals: 1

15208 and 16208 may be used as sources for Riemann sum sensors, useful in Energy Dashboard as Solar production sources.

You have the rest and almost everything works, so Must PV19-6248 EXP can be included in your compatibility list.
Only registers with errors and warnings do not work (n/a)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

5 participants