-
-
Notifications
You must be signed in to change notification settings - Fork 4
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
Minor/cosmetic issue: SoC translated incorrectly to "state of capacity" (correct: "state of charge") #26
Comments
Thanks for reporting it. Migration Procedure (BUT NOT YET CHANGED IN ADAPTER!!) INFLUX: To keep old history...
|
I know that this might be quite cumbersome, but one option could be to have a translation table for the data object names which the user can configure. That would also enable users to maintain their existing database entries (like InfluxDB) in case they already have existing ones with different namings without setting up aliases in ioBroker. Alternatively the users might set up aliases directly in ioBroker and have them being written to a database (I'm not that firm with ioBroker yet to confirm, whether that'd be working). |
Already implemented. Will come in next release. |
* [#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
First of all: A VERY promising and cool adapter for ioBroker!
Keep on rocking!
Describe the bug
You translated the register 37760 to "stateOfCapacity", whereas this register is holding the current state of charge.
Register 37760 should be translated to "stateOfCharge".
Versions:
Additional context
The state of capacity would more describe the remaining total capacity compared to the battery's initial capacity (the battery health as known from mobile phones) but that value doesn't seem to be held in an externally accessible register. To my knowledge that value isn't transparent to the user, since the gross capacity and net capacity derive from each other and Huawei / LG already added some "reserves" to cover the wearing of the battery.
The text was updated successfully, but these errors were encountered: