MySensors icon indicating copy to clipboard operation
MySensors copied to clipboard

Published 20 hours ago • verified publisher icon mysensors

V_Unit_Prefix for S_weight?

Open flatsiedatsie opened this issue 5 years ago • 8 comments

Shouldn't users be able to set what weight scale they are measuring in?

For example: Kilograms, grams, or even non-metric things?

Or is that perhaps what that 'V_Impedance' is for?

flatsiedatsie avatar Apr 24 '19 14:04 flatsiedatsie

For S_TEMP / V_TEMP, the unit is decided by the controller's isMetric setting. Reference https://github.com/mysensors/MySensors/blob/a1282f358aadfe3dcd5b486778274a8ddc869ac5/core/MySensorsCore.h#L89

Would it make sense to use something similar for S_WEIGHT? Or should units be handled in some other way? The current implementation for temp does't support Kelvin for example, so maybe the current implementation is i sufficient?

mfalkvidd avatar Apr 25 '19 07:04 mfalkvidd

Are there any specifications we could adapt, instead of inventing our own? OMA LightweightM2M has this specification for a "load" sensor. They use a free-form string for specifying the unit. That approach is flexible, but maybe too flexible? Could we find other useful specifications?

mfalkvidd avatar Apr 25 '19 07:04 mfalkvidd

And no, impedance is not related. This is what impedance is used for:

In addition to measuring weight, the bathroom scale calculates body composition by sending a mild electrical impulse through the body. The measured electrical resistance is used – together with information on age, gender and height – to calculate body fat, body water, and more.

mfalkvidd avatar Apr 25 '19 11:04 mfalkvidd

Ah, so that's what its for! Interesting.

I thought it would be useful to just add V_UNIT_PREFIX? That way the device can specify in what scale it's measuring (grams/kilograms).

flatsiedatsie avatar Apr 25 '19 18:04 flatsiedatsie

Sorry, I thought you proposed a new type, but I just noticed that V_UNIT_PREFIX is already used for distance, air quality and dust level.

Yes, using V_UNIT_PREFIX for any value that can have different units would be quite useful. In addition to the existing uses and weight (as per your suggestion) I think it would make sense to also use V_UNIT_PREFIX for things like voltage, current, atmospheric pressure, power usage (although V_KWH unfortunately already implies a unit), flow and volume.

mfalkvidd avatar Apr 25 '19 20:04 mfalkvidd

Exactly!

See also this issue for some musing on making some universal V_TYPES to help presentation in the UX

flatsiedatsie avatar Apr 28 '19 14:04 flatsiedatsie

See also https://github.com/mysensors/MySensors/issues/1312 for yet another discussion on the same topic.

mfalkvidd avatar Jul 18 '19 12:07 mfalkvidd

Yes I thought I'd create an official proposal for it.

flatsiedatsie avatar Jul 20 '19 13:07 flatsiedatsie