greensun_hybrid.yaml Please add this new inverter definition

[xenfasa]

New Solar Inverter definition I made and use for my own inverter

greensun_hybrid.yaml

greensun_hybrid.yaml.zip

[josemonino]

Thanks a lot.

How did you do it? Did you already know the registers before making it? It would be wonderful if there were any step by step on how to do it...

I would like to do it for my Sofar inverter.

[xenfasa]

Hi,

I bugged the company until they sent me a spreadsheet with some chinese and register information. I still had to decode a lot and I only mapped the data that I was needing. There is still some info i'm not sure of like alarm codes and I did not add info for a second battery since I don't have one yet. Attached is the version of the spreadsheet I made more readable but still not easy to figure out... The original was worse.... but this gives you an idea of what I started with.

I also found some python scripts to probe the registers... Some worked and some did not. Creating my own inverter definition with ideas from the registers in the spreadsheet and seeing what numbers I got back eventually got me something useful. I would ask the company for such a spreadsheet and data.

Ben

On Sat, Oct 28, 2023 at 5:22 PM josemonino @.***> wrote:

Thanks a lot.

How did you do it? Did you already know the registers before making it? It would be wonderful if there were any step by step on how to do it...

I would like to do it for my Sofar inverter.

— Reply to this email directly, view it on GitHub https://github.com/StephanJoubert/home_assistant_solarman/issues/436#issuecomment-1783941235, or unsubscribe https://github.com/notifications/unsubscribe-auth/AIACEPET3QHMJVKL3FT3HJTYBWHTJAVCNFSM6AAAAAA6PSGO5SVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTOOBTHE2DCMRTGU . You are receiving this because you authored the thread.Message ID: @.***>

[mrboogiee]

@xenfasa in your file there is the following item under the Battery group:

    - #NEEDXXXXXXXXXXXXXXXXXXXXXXXXXXx
      name: "XBattery Status WRONG Reg"
      class: ""
      state_class: "measurement"
      uom: ""
      scale: 1
      rule: 1
      registers: [0x3603]
      isstr: true
      lookup: 
      -  key: 0
         value: "Cancel"
      -  key: 1
         value: "Charging"
      -  key: 2
         value: "Discharging"
      -  key: 2
         value: "Stand-by"
      icon: 'mdi:battery'

That reads to me like the file isn't finished yet. Same for all the # DONE comments. Is your file a work in progress?

Did you by the way know that you can create a fork from this repository, add your file and then create a pull request? That way any questions that arise can be answered by you directly. Of course I can do it as well, but I don't have the device nor the insights on any issues you still have. This means I also cannot answer any questions the maintainers might have.

[xenfasa]

What agreement where. I don't see it or am I missing something?

On Thu, Mar 28, 2024, 7:23 PM cyh0928 @.***> wrote:

This is an agreement with megarevo. Greensun’s inverter is also manufactured by megarevo.

— Reply to this email directly, view it on GitHub https://github.com/StephanJoubert/home_assistant_solarman/issues/436#issuecomment-2026433933, or unsubscribe https://github.com/notifications/unsubscribe-auth/AIACEPBSWY75WBFN72UIBZ3Y2S7BJAVCNFSM6AAAAAA6PSGO5SVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAMRWGQZTGOJTGM . You are receiving this because you were mentioned.Message ID: @.***>

[cyh0928]

r.