brndnmtthws/conky

Add support for showing temp of (more) AMD chips

Open

#2,051 opened on Oct 2, 2024

View on GitHub
 (9 comments) (1 reaction) (0 assignees)C++ (604 forks)batch import
cpufeaturegood first issue

Repository metrics

Stars
 (6,472 stars)
PR merge metrics
 (Avg merge 17d 10h) (18 merged PRs in 30d)

Description

Support for showing temperature of (at least) following AMD chips is missing when using acpitemp:

This thread elsewhere reveals that Conky's acpitemp cannot get the temperature of a Phenom2 quad-core chip. I find that similarly Conky cannot get the temperature on my new Framework 13-AMD, with a Ryzen 6 764OU processor. (See perhaps also this further report.)

One can write an external program to allow Conky to display the values on such AMD chips, but native Conky support would be nice!

Comment here is you find any other so they be added to the list.

Contributor guide

Research direction
Investigate the acpitemp implementation in the conky source code, specifically how CPU temperatures are read. Find where chip IDs are matched and add the missing AMD chip IDs (Phenom2 quad core, Ryzen 6 764OU). Verify the changes on compatible hardware or via simulation.
Tech stack
cpp
Domain
tooling
Issue type
Feature
DifficultyEstimated implementation difficulty for a new contributor, from 1 for very small changes to 5 for expert-level work.
2
Estimated timeA rough time range for an experienced contributor to investigate, implement, test, and prepare a pull request.
1-3 hours
Activity statusHow available the issue appears right now: fresh, active, stale, blocked, or waiting on maintainer input.
Active
ClarityHow clearly the issue explains the expected change, acceptance criteria, and next step.
Clear
Prerequisites
C++
Newbie friendlinessA 1-100 score estimating how approachable this issue is for first-time contributors.
70