xxxserxxx / gotop

A terminal based graphical activity monitor inspired by gtop and vtop
Other
2.8k stars 142 forks source link

Temperature not showing for M2 Air #240

Open wrinkledeth opened 2 years ago

wrinkledeth commented 2 years ago

Temperatures blank for M2 Air

image

System Info

# gotop version
kitty % gotop -v
gotop 4.2.0 (20220929T181819)

# OS Info 
% sw_vers
ProductName:    macOS
ProductVersion: 12.6
BuildVersion:   21G115

Other Details

Hardware: Mac M2 Air Terminal: Kitty

No error files found.

anatol commented 1 year ago

@wrinkledeth does it work now?

mgasparUNC commented 9 months ago

Not working on my M2 air either, I am running MacOS 14.1 and Iterm2 as my terminal.

anatol commented 9 months ago

Does your application have access to the disk device? Did you run the application as a regular user or root?

wrinkledeth commented 9 months ago

Hi @anatol ,

Gotop: 4.2.0 Terminal: Kitty M2 Air running MacOS: 14.2.1

I just tried running gotop as root and i still cannot see the temps.

anatol commented 9 months ago

I just checked the sources and see that MacOS and Unix use different codepath to fetch disk temperature. @xxxserxxx what is the reason MacOS does not use smart.go library?

LordRusk commented 2 months ago

Jumping in here, on my nixos install, i moved from 4.2.0 to the latest commit, and i can confirm on my t541, i can no longer see tempuratures, whereas with version 4.2.0, i could see tempuratures. This is all the output i get...

latest commit

[~] $ gotop 
2024/09/21 14:04:28 gopsutil reports Number of warnings: 1 
2024/09/21 14:04:28 gopsutil reports Number of warnings: 1 

image

latest commit as root

[~] $ sudo gotop 
[sudo] password for shiva: 
2024/09/21 14:07:20 gopsutil reports Number of warnings: 1 
2024/09/21 14:07:20 gopsutil reports Number of warnings: 1 

image

4.2.0

2024/09/21 14:11:25 gopsutil reports Number of warnings: 1 
2024/09/21 14:11:25 gopsutil reports Number of warnings: 1 

image


more testing would need to be done, but it for sure worked on the last release without root privilege, and it does not work properly even with root privilege on the latest commit...


EDIT: forgot my specs image