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

not all stringdata are displayed #29

Closed
micklafisch opened this issue Jan 5, 2024 · 3 comments
Closed

not all stringdata are displayed #29

micklafisch opened this issue Jan 5, 2024 · 3 comments
Labels
enhancement New feature or request

Comments

@micklafisch
Copy link

Describe the bug
Only data from string 1 and 2 are displayed with "pvXcurrent" and "pvXvoltage", but mine has in sum 4 strings

Expected behavior
show the data from all strings

Screenshots & Logfiles
adapter:
image

my nodered:
image

@daolis
Copy link
Owner

daolis commented Jan 9, 2024

Suggestion:
We could use the Register 30071 (Number of PV strings) to dynamically create the correct number of PVn-Voltage/Current states

@daolis
Copy link
Owner

daolis commented Mar 4, 2024

Impl in progress. Will come in next release.

daolis added a commit that referenced this issue Mar 10, 2024
* [#34](#34) Added Battery registers: totalCharge, totalDischarge, batteryTemperature
* [#32](#32) Fixed: No data for ChargePower and CurrentDayChargeCapacity
* [#20](#20) Read alarm registers from inverter (alarms = bits, alarmsJSON = json array with alarms (name, id, level))
* [#29](#29) Added PVn Voltage and Current for available PV Strings
* Added initial documentation - WIP
* [#26](#26) Renamed stateOfCapacity to stateOfCharge\
  Migration of historical data: see [Migration of historical data (on state name change)](docs/migration.md)
* Blocked reading of registers - faster when fetching data from inverter
@daolis
Copy link
Owner

daolis commented Mar 10, 2024

Dynamicall added number oof registers depending on Number of PV strings. Same for MPPTrackers!

@daolis daolis closed this as completed Mar 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants