GPU/CPU Temp

[cpdlink]

Raspberry Pi 4B running Raspberry Pi OS Lite (Release date: January 11th 2021, Kernel version: 5.4):

1. both GPU & CPU Temp labels have wrong background color when using MC color scheme - black instead of blue probably applies to all other non default color schemes; version 2.0.2 had the correct background value (please see attached sample)
2. wrong/inconsistent spelling of GPU & CPU Temp labels: i.e. Cpu & Gpu (mixed caption) instead of CPU & GPU (all CAPS) as everywhere else (please see attached sample)
3. GPU Temp shows zero degrees although CLI command shows correct value when queried "vcgencmd measure_temp"
4. degree symbol missing from both GPU and CPU Temp meters C instead of °C (please see attached sample)
5. it would be a lot more readable if you could right-align the colon character of all text meters (please see attached sample)
6. <Time> meter not available anymore - it use to be part of the version 2.0.2
   

thank you!

[avafinger]

Interesting findings.

| 3. GPU Temp shows zero degrees although CLI command shows correct value when queried "vcgencmd measure_temp"
This is similar to avafinger/htop_2.2.2-amd64#2

There will be a security breach if htop can run a script or program to find some platform specific information. I was thinking of overcoming this by passing a parameter when executing htop, so that it would be up to the user to decide on the security risks.

Another challenge is that the headers are generated automatically, with this design it is impossible to build with gcc 10.
The alignment looks very good, but it will certainly create problems if you have a screen size of 80x24, some information may be cut off.

Anyway, I will review the code and try to fix it.

[avafinger]

6. <Time> meter not available anymore - it use to be part of the version 2.0.2

To show the Time use the Clock meter.

[avafinger]

Can try to build from the source and check if it works for you?

https://github.com/avafinger/htop_2.2.2/commits/master

[cpdlink]

hi, thank you for the quick turnaround - i am really sorry for my late reply i must confess ... i'm new to git/linux/rpi, would it be possible to have some pointers on how to build the new version (the more detailed, the better) regards, ~chris

…

On Sat, Mar 6, 2021 at 10:56 AM avafinger ***@***.***> wrote: Can try to build from the source and check if it works for you? https://github.com/avafinger/htop_2.2.2/commits/master — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#2 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/ATDMNLP7AFBOYEKUM7OMSZLTCJGB7ANCNFSM4YWGLWEA> .

[roy-m-organ]

New to Github, this is a necessary add

[avafinger]

Instructions to build: https://github.com/avafinger/htop-2.1.1_enhanced-version#building-the-package-from-source-code

[roy-m-organ]

Thank you for your response

…

On Sun, 23 May 2021, 15:47 avafinger, ***@***.***> wrote: Instructions to build: https://github.com/avafinger/htop-2.1.1_enhanced-version#building-the-package-from-source-code — You are receiving this because you commented. Reply to this email directly, view it on GitHub <#2 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AQ7AGGKZMRD5WLPYYZ3BEFDTPEIO7ANCNFSM4YWGLWEA> .