-
-
Notifications
You must be signed in to change notification settings - Fork 5
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
Some datapoints are string, but number would be better #123
Comments
The problem is that there is no real list of values (yes some are "common" but in fact all depend on the used driver). So the adapter should just guess ... and guessing means ... meehhhh Need to think about that |
Did have time to think about it already? BR |
Not really. YOu are free to use aliases or own Javascript states to convert the values as needed in the meantime |
If I prepare the appropriate PR - will you look for approve? |
Sure. I personally would go with option B because "Not breaking" ... Maybe add an option if such number parsing should be done (but can also be omitted) ... What do you think? Which ideas you had to do it backward compatible and handle type-issues (honestly I do not trust that all nut "plugins" are developed constistent on the value types? |
At the moment I don't know how I could help here. |
Datapoints, e.g. battery.voltage, input.voltage,... , are type string, but number would make more sense (IMHO).
Writing datapoints of type string to InfluxDB works, but makes live just complicated.
BR
Stefan
The text was updated successfully, but these errors were encountered: