Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Feedback]power and temperature shown as 0.0W / 0C #694

Closed
romaniansuperhero opened this issue Nov 22, 2022 · 2 comments
Closed

[Feedback]power and temperature shown as 0.0W / 0C #694

romaniansuperhero opened this issue Nov 22, 2022 · 2 comments
Assignees
Labels
bug Something isn't working feedback

Comments

@romaniansuperhero
Copy link

Description

power and temperature shown as 0.0W / 0C

acpi -t
Thermal 0: ok, 40.0 degrees C

paste <(cat /sys/class/thermal/thermal_zone*/type) <(cat /sys/class/thermal/thermal_zone*/temp) | column -s $'\t' -t | sed 's/(.)..$/.\1°C/'
acpitz 39.0°C

bord a68n-5745
https://www.biostar.com.tw/app/en/mb/introduction.php?S_ID=857#specification
AMD A10-5745(Quad core 2.1G, turbo 2.9G) processor
AMD A70M Chipset

Additional Information

 - OS: linux
 - CasaOS Version: 0.3.8
 - Disk Total: 54490 
 - Disk Used: 9157 
 - System Info: [Ubuntu 22.04.1 LTS   \l Bit:64 Mem:7111M Core:4 Linux pi-nas 5.15.0-53-generic #59-Ubuntu SMP Mon Oct 17 18:53:30 UTC 2022 x86_64 x86_64 x86_64 GNU/Linux]
 - Remote Version: 0.3.8
 - Browser: Firefox 
 - Version: 107.0
@mojothemonkey2
Copy link
Contributor

thanks for the detailed ticket!
you provided all the info would have asked for already :)

so for this hardware, the thermal zone has a very generic name "acpitz".
instead of trying to handle all the edge cases for cpu thermal zone names, I think we'll be better off falling back to zone0, when it's not one of the obvious cpu ones.
I'll submit a change when I get a moment.

@raller1028 raller1028 moved this to Need Triage (QA+PM) in CasaOS Issues Nov 23, 2022
@raller1028 raller1028 moved this from Need Triage (QA+PM) to Planned (Dev) in CasaOS Issues Nov 23, 2022
@zhanghengxin zhanghengxin added the bug Something isn't working label Nov 23, 2022
@mojothemonkey2
Copy link
Contributor

So now if it can't match cpu name, it should do this:

{"level":"warn","ts":1669276097.285511,"msg":"CPU thermal zone not matched. Default to thermal_zone0.","func":"service.GetCPUThermalZone","file":"/code/mojothemonkey/CasaOS/service/system.go","line":335}

Repository owner moved this from Planned (Dev) to To be verified (QA) in CasaOS Issues Nov 25, 2022
@raller1028 raller1028 moved this from To be verified (QA) to Done in CasaOS Issues Nov 28, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working feedback
Projects
Archived in project
Development

No branches or pull requests

4 participants