Initial version of a definitions file for the 3phase inverters #93
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
kellerza
pushed a commit
that referenced
this pull request
Feb 5, 2023
|
Can anyone confirm how to use this definitions file? Do I have to create a local copy of the addon and rename definitions3ph.py to definitions.py? |
|
So I cloned the repo and renamed the file as per my previous comment, but with debug turned on I can see its still requesting the same registers from the original definitions file 😕 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Not remembering last time I created a pull request so hope all went well..
As discussed in issue #63 here's a separate file with registeres for the 3 phase inverters. There are sensors removed and sensors added like the different phase values (credits to @archi for these).
The program 'mode options' I'm not sure of, might be there's a difference between inverters in this menu.
I can set the time, charge options, power and voltage/soc. Enable/disable global time-of use and set the battery max charge/discharge amps. Which was my goal. There are a couple of optional sensors, they work and someone might be interested in these.
The 3 phase inverters don't have a SD Card but if the sensor is removed it crashes because of filter line 201 so I left it.
(btw, you should remove register 107 for fault sensor, it's 'corrected Ah' not a fault register)
The way I see it there are two ways of implementing, simple drop-down list selecting single-phase / 3-phase definitions file. Or automatically by reading register 0. This will give 3 for a hybrid inverter and 5 for a 3 phase hybrid inverter. Based on that value the 'correct' definitions file can be loaded.
I will be able to add the drop-down option, it will work but I'm not a python programmer so probably will not be the correct way.
I'm using a Waveshare rs485-eth converter, and it's working great so far. It's cheap (around 25eur inc shipping) https://www.waveshare.com/product/rs232-to-eth-b.htm If you want I can take a screenshot of the settings page to add to the readme.
If there's anything wrong in the file, am forgetting or any other comments please let me know.
Thanks for creating this great plugin...