Project

General

Profile

Bug #66520

Convert CPU temperature values from deci-Kelvin to Celsius in new UI

Added by Disk Didler almost 3 years ago. Updated over 2 years ago.

Status:
Done
Priority:
No priority
Assignee:
Damian Szidiropulosz
Category:
GUI (new)
Target version:
Severity:
New
Reason for Closing:
Reason for Blocked:
Needs QA:
No
Needs Doc:
No
Needs Merging:
No
Needs Automation:
No
Support Suite Ticket:
n/a
Hardware Configuration:
ChangeLog Required:
No

Description

See attached screenshots.

This is not good at all, one of the figures is very low and safe sounding, the other is quite high (and I suspect the actual temp)
Any ideas?

https://i.imgur.com/Fg6WX0u.jpg New

https://i.imgur.com/2ydFaPz.jpg Old

I'm trying so so hard to not be rude and to be constructive and helpful for the community but if I can just chanel some of my inner whine / rage. This kind of stuff is exactly why the vast majority of long term hardcore users, prefer the old UI.

Consistency is really important.

Happy to provide more information if required. (SuperMicro Denverton 3750)

History

#1 Updated by Dru Lavigne almost 3 years ago

  • Private changed from No to Yes
  • Reason for Blocked set to Need additional information from Author

Please attach a debug so the dev can determine which info is being passed to each UI.

#2 Updated by Disk Didler almost 3 years ago

  • File debug-freenasnew-20190101111052.tgz added

Just rebooted this morning, same thing - under 30c, vs near 70c.

#3 Updated by Dru Lavigne almost 3 years ago

  • Category changed from GUI (new) to Middleware
  • Assignee changed from Release Council to William Grzybowski
  • Reason for Blocked deleted (Need additional information from Author)

#4 Updated by William Grzybowski almost 3 years ago

  • Status changed from Unscreened to Blocked
  • Reason for Blocked set to Need additional information from Author

Can you run

sysctl dev.cpu

And verify temperatures? My suspicion is that legacy UI has it wrong.

#5 Updated by William Grzybowski almost 3 years ago

  • Status changed from Blocked to Unscreened
  • Reason for Blocked deleted (Need additional information from Author)

Nevermind, I forget debug file has it. Legacy UI has it right.

#6 Updated by William Grzybowski almost 3 years ago

  • Category changed from Middleware to GUI (new)
  • Assignee changed from William Grzybowski to Anonymous
  • Target version changed from Backlog to 11.2-U2

Erin, the data the middleware provides is in deci-Kelvins, so you have to divide by 10 and subtract 273.15 to convert to Celsius.

You get the raw data stored and middleware just pass it through. We can discuss transforming data on middleware side at some point but for now thats all we have for 11.2.

#7 Updated by Anonymous almost 3 years ago

  • Assignee changed from Anonymous to Damian Szidiropulosz

#8 Updated by Dru Lavigne almost 3 years ago

  • Target version changed from 11.2-U2 to 11.2-U3

#9 Updated by Damian Szidiropulosz almost 3 years ago

  • Status changed from Unscreened to In Progress

#10 Updated by Damian Szidiropulosz almost 3 years ago

  • Status changed from In Progress to Ready for Testing

#11 Updated by Anonymous almost 3 years ago

  • Status changed from Ready for Testing to In Progress

#12 Updated by Dru Lavigne almost 3 years ago

  • File deleted (debug-freenasnew-20190101111052.tgz)

#13 Updated by Dru Lavigne almost 3 years ago

  • Subject changed from Wildly different figures reported for CPU in new UI vs Legacy. to Convert CPU temperature values from deci-kelvin to celsius in new UI
  • Status changed from In Progress to Ready for Testing
  • Needs Doc changed from Yes to No
  • Needs Merging changed from Yes to No

#14 Updated by Dru Lavigne almost 3 years ago

  • Private changed from Yes to No

#15 Updated by Dru Lavigne almost 3 years ago

  • Has duplicate Bug #76224: Disk temp shows a nonsense total value added

#16 Updated by Dru Lavigne almost 3 years ago

  • Has duplicate deleted (Bug #76224: Disk temp shows a nonsense total value)

#17 Updated by Dru Lavigne almost 3 years ago

  • Has duplicate Bug #76521: Percentages in Reporting way too high added

#18 Updated by Dru Lavigne almost 3 years ago

  • Has duplicate deleted (Bug #76521: Percentages in Reporting way too high)

#23 Avatar?id=55038&size=24x24 Updated by Zackary Welch over 2 years ago

  • Status changed from Ready for Testing to Passed Testing
  • Needs QA changed from Yes to No

CPU temp now shows as Celsius.

#24 Updated by Dru Lavigne over 2 years ago

  • Status changed from Passed Testing to Done

#25 Updated by Dru Lavigne over 2 years ago

  • Subject changed from Convert CPU temperature values from deci-kelvin to celsius in new UI to Convert CPU temperature values from deci-Kelvin to Celsius in new UI

Also available in: Atom PDF