column layout in "sensors" is broken for voltages of 1000.00 mV

Bug #1883577 reported by Florian Hars
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
lm-sensors (Ubuntu)
Won't Fix
Wishlist
Unassigned

Bug Description

The sensors program tries to print all values neatly in columns, but there are some voltage values which it displays as 1000.00 mV, which is one digit too long.

Severity: cosmetic

ProblemType: Bug
DistroRelease: Ubuntu 20.04
Package: lm-sensors 1:3.6.0-2ubuntu1
ProcVersionSignature: Ubuntu 5.4.0-37.41-generic 5.4.41
Uname: Linux 5.4.0-37-generic x86_64
ApportVersion: 2.20.11-0ubuntu27.2
Architecture: amd64
CasperMD5CheckResult: skip
CurrentDesktop: ubuntu:GNOME
Date: Mon Jun 15 18:46:47 2020
InstallationDate: Installed on 2020-05-28 (18 days ago)
InstallationMedia: Ubuntu 20.04 LTS "Focal Fossa" - Release amd64 (20200423)
SourcePackage: lm-sensors
UpgradeStatus: No upgrade log present (probably fresh install)

Revision history for this message
Florian Hars (hars) wrote :
Revision history for this message
Sergio Durigan Junior (sergiodj) wrote :

Thank you for taking the time to file a bug report.

Could you please describe how you are running the "sensors" command? I was not able to obtain these values you mentioned, and I would like to understand how the output is being formatted in your machine, and why you think this is a problem.

Since there is not enough information in your report to begin triage or to
differentiate between a local configuration problem and a bug in Ubuntu, I
am marking this bug as "Incomplete". We would be grateful if you would:
provide a more complete description of the problem, explain why you
believe this is a bug in Ubuntu rather than a problem specific to your
system, and then change the bug status back to "New".

For local configuration issues, you can find assistance here:
http://www.ubuntu.com/support/community

Changed in lm-sensors (Ubuntu):
status: New → Incomplete
Revision history for this message
Florian Hars (hars) wrote :

I am running the "sensors" command by typing "sensors" into a terminal window and hitting Enter.

It is a bug because the column layout is broken for some voltage values which happen to be relatively common on my system, see the "in6" value:

ct6798-isa-0290
Adapter: ISA adapter
in0: 944.00 mV (min = +0.00 V, max = +1.74 V)
in1: 1.01 V (min = +0.00 V, max = +0.00 V) ALARM
in2: 3.34 V (min = +0.00 V, max = +0.00 V) ALARM
in3: 3.31 V (min = +0.00 V, max = +0.00 V) ALARM
in4: 1.02 V (min = +0.00 V, max = +0.00 V) ALARM
in5: 816.00 mV (min = +0.00 V, max = +0.00 V) ALARM
in6: 1000.00 mV (min = +0.00 V, max = +0.00 V) ALARM
in7: 3.34 V (min = +0.00 V, max = +0.00 V) ALARM

Changed in lm-sensors (Ubuntu):
status: Incomplete → New
Revision history for this message
Florian Hars (hars) wrote :

Sorry, the web form completely killed the column layout.

Revision history for this message
Florian Hars (hars) wrote :

Here it is again with underscores instead of spaces:

nct6798-isa-0290
Adapter:_ISA_adapter
in0:___________________944.00_mV_(min_=__+0.00_V,_max_=__+1.74_V)
in1:_____________________1.01_V__(min_=__+0.00_V,_max_=__+0.00_V)__ALARM
in2:_____________________3.34_V__(min_=__+0.00_V,_max_=__+0.00_V)__ALARM
in3:_____________________3.31_V__(min_=__+0.00_V,_max_=__+0.00_V)__ALARM
in4:_____________________1.02_V__(min_=__+0.00_V,_max_=__+0.00_V)__ALARM
in5:___________________816.00_mV_(min_=__+0.00_V,_max_=__+0.00_V)__ALARM
in6:___________________1000.00_mV_(min_=__+0.00_V,_max_=__+0.00_V)__ALARM
in7:_____________________3.34_V__(min_=__+0.00_V,_max_=__+0.00_V)__ALARM

Paride Legovini (paride)
Changed in lm-sensors (Ubuntu):
importance: Undecided → Low
Revision history for this message
Paride Legovini (paride) wrote :

Hi Florian, I set the Importance of this bug report to Low, as the issue is cosmetic.

Ideally this should be fixed upstream: this way the fix will be picked up by Debian/Ubuntu and other distributions without the added maintenance need of carrying an Ubuntu-specific delta.

Would you consider forwarding this bug to the lm-sensors upstream developers, and add a link to the upstream bug report (or mailing list message) to this Ubuntu bug report?

Thanks!

Changed in lm-sensors (Ubuntu):
status: New → Triaged
Revision history for this message
Sergio Durigan Junior (sergiodj) wrote :

Hi again,

After thinking a bit about this bug, I think it is fairly safe to say that we as distro won't fix it. It is entirely cosmetic, and as Paride already said, the best place to raise such issues is directly with upstream.

If you would like to report an issue with them, please follow this link:

https://github.com/lm-sensors/lm-sensors/issues/new

Ideally you will have to provide reproducible steps, but I think it should be trivial for upstream to check for the validity of the bug.

I am not going to mark this as Won't Fix just yet, but will downgrade its severity to Wishlist.

Changed in lm-sensors (Ubuntu):
importance: Low → Wishlist
Revision history for this message
Bryce Harrington (bryce) wrote :

As per the last comment, this change is best proposed upstream directly.

Changed in lm-sensors (Ubuntu):
status: Triaged → Won't Fix
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.