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

temprtaure module doesnt work #19

Closed
paradox-sp opened this issue Nov 13, 2023 · 6 comments
Closed

temprtaure module doesnt work #19

paradox-sp opened this issue Nov 13, 2023 · 6 comments

Comments

@paradox-sp
Copy link

sensors command shows the temperatures but motd doesnt show the temperature

@paradox-sp
Copy link
Author

Fixed this in my fork https://github.com/paradox-sp/fancy-motd

@bcyran
Copy link
Owner

bcyran commented Nov 15, 2023

Unfortunately, your version does not work correctly on my machine, whereas mine does.

Your version:
2023-11-15_19:41:50

My version:
2023-11-15_19:41:38

For reference, this is my sensors version:

❯ sensors -v
sensors version 3.6.0+git with libsensors version 3.6.0+git

And this is relevant sensors output on my machine:

Core 0:        +40.0°C  (high = +100.0°C, crit = +100.0°C)
Core 4:        +41.0°C  (high = +100.0°C, crit = +100.0°C)
Core 8:        +38.0°C  (high = +100.0°C, crit = +100.0°C)
Core 12:       +41.0°C  (high = +100.0°C, crit = +100.0°C)
Core 16:       +38.0°C  (high = +100.0°C, crit = +100.0°C)
Core 20:       +40.0°C  (high = +100.0°C, crit = +100.0°C)
Core 24:       +38.0°C  (high = +100.0°C, crit = +100.0°C)
Core 25:       +38.0°C  (high = +100.0°C, crit = +100.0°C)
Core 26:       +38.0°C  (high = +100.0°C, crit = +100.0°C)
Core 27:       +38.0°C  (high = +100.0°C, crit = +100.0°C)
Core 28:       +43.0°C  (high = +100.0°C, crit = +100.0°C)
Core 29:       +43.0°C  (high = +100.0°C, crit = +100.0°C)
Core 30:       +43.0°C  (high = +100.0°C, crit = +100.0°C)
Core 31:       +43.0°C  (high = +100.0°C, crit = +100.0°C)

It's probably a matter of differing output between sensors versions. It would be probably possible to figure out a code to handle both of them. It would be helpful if you posted your version and how the output looks on your machine.

Maybe I'll look into it in a spare time, but I don't promise anything. PRs always welcome 😉

@quorn23
Copy link

quorn23 commented Dec 9, 2023

Mine didn't work either.
Debian box.
sensors version 3.6.0 with libsensors version 3.6.0

If i try the sensors grep line on the cli i get an issue with the symbols not showing up, running as motd the temp isn't showing up at all.

Tabby_AL4JtfbPuX

I fixed it with:
cores="$(sensors | grep Core | awk '{printf "%s ", $3} {printf "%s %s %s\n", $6, $9, $12}' | tr -d '+°,' | sed 's/)//')"

Tabby_wVHaAp4aJl

If this warrants a PR let me know and i'll do one :)

@jackra1n
Copy link

jackra1n commented Jan 4, 2024

I'm having the same issue

@jackra1n
Copy link

jackra1n commented Jan 4, 2024

in my case i had to do: cores="$(sensors | grep Tctl | awk '{printf "%s ", $2}' | tr -d '+°,' | sed 's/)//')"

@bcyran
Copy link
Owner

bcyran commented Jan 7, 2024

@quorn23 Fixed in 19b515c
@jackra1n Fixed in e3d3dff

@bcyran bcyran closed this as completed Jan 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants