-
-
Notifications
You must be signed in to change notification settings - Fork 44
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
101% CPU utilization #345
Comments
Here is how load is calculated: https://github.com/stsdc/monitor/blob/0dcfffe7714aebe9498ef70410bdb1c52fa25f44/src/Resources/CPU.vala#L112-L126 And here is how percentage is being calculated: https://github.com/stsdc/monitor/blob/0dcfffe7714aebe9498ef70410bdb1c52fa25f44/src/Resources/CPU.vala#L24-L28 Might be a rounding issue? |
Rounding issue sounds likely. Do you need more information to solve it, like the |
Would be nice to have, just in case of. |
top is running with Here are 3 screenshots: Here is a video. Maybe it helps to check it frame-by-frame with like VLC: https://www.youtube.com/watch?v=sW3jGuMZOgI |
Can't reproduce on my machine :( I'm debugging it with this log line: public int percentage {
get {
debug ("%f, %d", load, ((int) (Math.round (load * 100))));
return (int) (Math.round (load * 100));
}
} I have prepared a branch for this issue: https://github.com/stsdc/monitor/tree/101-percent |
I tried to build from the commit, but it fails.
Same error also at previous commit (7786ca3). Package source changed with:
|
I'm not familiar with NixOS, can't help with packaging. But maybe you can just clone the repo and try to build from sources? |
The issue was that the repo was not cloned with submodules. |
Here are the results: You should be able to reproduce the issue with:
Video: https://youtu.be/kwRwTUMl5mY |
The number seem wrong. See the individual cores!
It cycles between 99%, 100% and 101%.
Monitor 0.15.1
NixOS 22.11.2568.c43f676c938
The text was updated successfully, but these errors were encountered: