elementary / elementary/monitor

CPU Frequency chart maximum too restrictive

Open
#494 1 comment 0 reactions 0 assignees View on GitHub
Status: Confirmed
Dominant language
Vala
Stars
362
Forks
49
Avg merge
10d 1h
Merged PRs (30d)
2

Description

### What Happened?

The current maximum for the cpu frequency graph is at 5GHz.

Looking at the top gaming cards from AMD, this falls short:

Image

https://www.amd.com/en/products/processors/desktops/ryzen.html#enthusiast

This also is the case for intel:

Image

https://www.intel.com/content/www/us/en/products/details/processors/core.html

Image

A fix could be to upon starting the app, find the maximum frequency of a cpu and add an arbitrary value to it, e.g. + 1,000.

A workaround for this locally can be to adjust the value at line 57 in `src/Views/SystemView/SystemCPUInfoPopover.vala` from `5.0` to `X.0`

### Steps to Reproduce

1. Have a high CPU clock rate and have it show up in the app
> This can be achieved in a few different ways: update the freq to show the max value, use frequency lock (some vendors support that), or you can average the freq across used cores (which is what I have running now).

### Expected Behavior

The graph to be fully visible to higher-end cpus that are overclocked (or have auto-configured like mine does), or perhaps to be a bit more dynamic, closer to what user runs for the max.

### OS Version

Other Linux

### OS Architecture

amd64 (on most hardwares)

### Session Type

Secure Session (Wayland)

### Software Version

Compiled from git

### Log Output

```shell

```

### Hardware Info

AMD 9950x overclocked, so the range typically is above the max.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.