-
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
Invalid values for some temperatures #12
Comments
Thanks. But I don't understand. Which of these are correct decimal numbers (without the °C)? Can you write me an example how it looks actually in iobroker - for every of these 4 entries. |
I'll make a pull request for this tommorow. I think this is easier. I found a fifth value which is wrong. There is simply missing a When using history or sql module a value like |
Hmm, okay. I just checked the file. It seems so, that there is only the value I don't know, where the other identifier came from:
Noticable is, that they are all having a german name. |
I've found a sixth value, take a look at this screenshot. I tried to show, what is wrong in my opinion: Invalid value = Has to be divided by 10 to match the current value |
@Huseriato can you please help me? Please set the language in nibeuplink adapter settings to english. Then set the log level of the nibeuplink instance to "silly".
You will see it, if you download the log. Can you please add all here - as file is the best, i think. You can remove all entries with "categoryId": "SYSTEM_INFO". I want so search for missing entries to add to the adapter. And don't forget to set the language and the log level back. Setting the language to english will add you some new entries in english - these are the entries I am looking for. You can delete them after setting language back to german. |
Sorry for my late answer. I did not see the notification. Here is my log file: iobroker.2020-12-21.log.html.zip |
Thanks. I added 10 missing parameters. You can see them here: e25ab10 It is on the master branch, so if you want you can install the nibeuplink adapter in iobroker directly from github. After installation you have to get a new auth code in the settings of the adapter. I fixed this bug, but this time you need to do this the last time. |
I tried version 0.4.0 - the new values are correctly appearing. However: The old values are still 10 times to high (those from the screenshot above). |
The english strings of this wrong values are:
This value seems to be fixed:
|
The version of master branch is 0.4.0 - but you need the last version of that 0.4.0. I don't increase the version because it is no official version - its only on github. The old, autogenerated keys (German or English) are not updated. You can delete them. |
I installed directly from GitHub yesterday. I think I'll delete the whole adapter and reinstall it (bye bye measurements) and see, what I'll get than. I think I'll use the "English" names for object keys, so debugging is much easier than. Or is this bad for testing this issue? |
You don't need to delete your objects. |
I don't want to loose my history values, so I implemented the fallback to update existing objects. Later I fix the object names in the database and than I remove the old objects. |
Because of the switch from German to English I did not know which values are correct. So I reinstalled the adapter via GitHub and removed my objects. I reassigned to logging (I'm using InfluxDB) with the same names. So there is just a small gap. Because auf the error, the most of the data is wrong and I cleared my series. After reinstall, all seems to be fine. Cool. Thank you. |
There are sadly some new values that are wrong for UNIT1: If I change back to english now to see the original object names, my objects are cluttered again. :/ Do you need the english names again? Edit: The last value of "Verdichtermodul" is also wrong. Did not see it (18 °C are wrong, correct is 1,8 °C) |
I created a second instance in english, may be that helps. This are the values that need a division by 10:
|
@Huseriato Do you have all other objects in english? I want to add all english keys. The goal is, that in all languages the keys of the objects are the same. |
I found a way to read all values with names in english. So wait - perhaps I have all. |
I added hundreds of missing parameters by reading all numbers from nibeuplink. All values with units "°C", "kW", "kWh" and "l/m" are automatically devided by 10. |
@Huseriato @steffenkuss Please check if you still have incorrect values. Then please let me know. I would like to close the ticket and make a npm version. |
the values are correct now on my side. |
I can test it today 20:30 GMT+1 and give feedback. |
Seems fine now. My object ids are "english" now, and the name is german. This i fine. All values seem to be correct. Thank you for all your work. This is really great! |
The following temperatures are multiplied by 10 (for example "430 °C" instead of "43.0 °C"):
The objects with correct decimal places (I mean the four above, without the _BT at the end) are not usable for history, influxdb or sql, because the °C is directly behind the number (so you can only log as text, which is bad).
The text was updated successfully, but these errors were encountered: