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

Updates of the PI18 Protocol for the Mppsolar LV5048 Hyprid V2 inverter. #128

Open
wants to merge 145 commits into
base: pi18
Choose a base branch
from

Conversation

preussal
Copy link

After some training time and a lot of try and error.
C ++ is not mine.

With the update, except for the battery Bulk/Float Voltage, and battery Charge/Discharge Voltage work all important things.

The Selects can be adjusted via the YML. But these values are the one that supports the protocoll. Sensor and Device classes were adjusted so that the values in the home assistant are also suitable.

Webinterface
grafik

If someone has an idea of how to do two Select fields with the values for the battery voltage Bulk/Float, that would be great
Or a text field where you then enter "552, 540". Which is then translated into the command "^S015MCHGV552, 540".

For the current mapping solution, 2x 11025 lines would be required to enable all values for it.

@daniil226
Copy link

Hello. How can I connect your version of the library? The 007GS command does not work in the original. The inverter only responds to 005GS, but when I connect your branch, it gives an error during compilation.

@preussal
Copy link
Author

Of course you can use my branch
Here is my external source.
Can you show me the compile error, you are also welcome to open an Issue in on my Dev Repo.

external_components:
  #- source: github://preussal/esphome-pipsolar@PI18_LV5048_Hybrid_V2
  - source: github://preussal/esphome-pipsolar@PI18_LV5048_Hybrid_V2_DEV
  #- source: github://syssi/esphome-pipsolar@pi18
    refresh: 0s

I just let the build go through, and I had no build mistakes.

@daniil226
Copy link

I don’t know what you did, but now everything compiled without errors. Everything worked. Thank you very much.
Inverter AXIOMA 5 kW

@daniil226
Copy link

errors that were described in the topic:
#139

@preussal
Copy link
Author

I don’t know what you did, but now everything compiled without errors. Everything worked. Thank you very much. Inverter AXIOMA 5 kW

That was my mistake, I always built my ESP on the DEV branch.
For weeks / months.
But also not further developed because what I needed works.

And the end, i forgot to Merge all the changes to the Stable Branch.
Sorry

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

Successfully merging this pull request may close these issues.

2 participants