Raw output only in celsius even with -f switch

[Us3r]

Version: 1:3.3.4-2ubuntu1

sensors offers:
- switch -u for raw output
- switch -f for fahrenheit

both in combination return values only in celsius

[Us3r (us3r)]

Package changed.

[Nish Aravamudan (nacc)]

Thank you for this bug report! I just tried in 16.04 and:

 no parameters: raw
 -f : Farhenheit
 -u : raw values
 -f -u or -u -f : raw values

Do you want -f to override the behavior of -u? I don't think that's the intention. I agree the man-page may need to be clearer, but it's not clear to me if this is really a bug or a documentation issue. Also, have you verified if upstream lm-sensors behaves similarly?

[Us3r (us3r)]

Hey!

Have only an debian machine here for now but another version

Package: lm-sensors
Version: 1:3.3.5-2

Upstream version as I see is 3.4.0. Need to test this...

Usually I would think:

no parameters: normal output, with censius (OK)
-u: raw, celsius (OK)
-f: normal output, with fahrenheit (OK)
-u -f: raw, celsius (would expect fahrenheit)

As I am currently working on projects which integrate system statistics on integrated devices I would also suggest an option where the output could also be shown as an json array, f. e. similar to "lshw -C network -json"

However, switch -f should display Fahrenheit no matter if raw or not.
So -f should not override -u
args: -f = always fahrenheit, -u only sets the output mode

Thanks!

[Robie Basak (racb)]

If asking to report raw values, I think one would expect the raw value reported to be displayed, whichever unit that happens to be provided in, with no conversion. "Raw" to me means (amongst other things) "no conversion".

So it seems to me that -u and -f should be mutually exclusive.

Presumably this should be sent upstream, assuming that it affects upstream. Perhaps they will update the manpage or make the options mutually exclusive. I don't think this issue warrants a delta in Ubuntu. I'm marking this Won't Fix for Ubuntu for now, to make it clear that I don't expect anything to happen in Ubuntu directly about this. We can change this if upstream agrees to commit some change here.

If sending upstream, please check if the manpage comes from all the way upstream or just Debian.