derAlff / ioBroker.ve

Other
7 stars 0 forks source link

Feature request - summary of solar power #24

Open Wingsofcourage opened 5 months ago

Wingsofcourage commented 5 months ago

Can the adapter collect also the total solar power ?

com.victronenergy.system | PV - DC-coupled power | 850 | uint16 | 1 | 0 to 65536 | /Dc/Pv/Power | no | W | Summation of output power of all connected Solar Chargers

I think it's modbus address 850.

Wingsofcourage commented 5 months ago

Add something like that

    SummationOutputPowerOfAllConnectedSolarChargers: {
        RegisterName: "Summary Solar Power",
        Register: 850,
        Id: 100,
        Length: 1,
        Unit: "W",
        Factor: 1,
        Type: "uint16",
        writable: false
    }

to ve.register.victronenergy.js

Karkas66 commented 1 month ago

could you PLEASE add this to the official code. We all have a SolarCharger it is ridiculous not to request this value.

After Changing the file this Content gets overwritten and you have to manually paste it into the file again.... PLEASE just add it and commit

derAlff commented 1 month ago

@Wingsofcourage and @Karkas66 Please excuse the long wait. I am currently dealing with some difficult illnesses. That's why I can't program as much. Please bear with me.

I have added the register to the register map. Please test it and give me feedback. You can now find the register at the bottom of the list (DCCoupledPower)

image

Karkas66 commented 3 weeks ago

image

works as expected!

derAlff commented 3 weeks ago

Thank you for this Information. I should put it in the DC folder 😅.

Karkas66 commented 3 weeks ago

I dont linke the Term DCCoupledPower because the power in the DC World is usually the Battery normal voltage. Maybe you should rename it as already suggested

derAlff commented 3 weeks ago

What's your favorite name? "MPPT-SumEnergy" is my favorite

Karkas66 commented 3 weeks ago

What's your favorite name? "MPPT-SumEnergy" is my favorite

yeah that sounds good

MPPT-Total something