Bug 494019 - Grid graph minimum time between updates setting does not work
Summary: Grid graph minimum time between updates setting does not work
Status: RESOLVED FIXED
Alias: None
Product: plasma-systemmonitor
Classification: Applications
Component: general (show other bugs)
Version: 6.1.5
Platform: Arch Linux Linux
: NOR minor
Target Milestone: ---
Assignee: KSysGuard Developers
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2024-10-03 06:09 UTC by Caio
Modified: 2024-10-23 13:52 UTC (History)
4 users (show)

See Also:
Latest Commit:
Version Fixed In: 6.2.3
Sentry Crash Report:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Caio 2024-10-03 06:09:25 UTC
SUMMARY
The changes to "minimum time between updates" settings for the Grid display type do not apply. It always update to its default setting.

STEPS TO REPRODUCE
1. Create a Grid display and add sensors, e.g. [group] cpu core total usage
2. Change the "minimum time between updates" setting to a higher value, e.g. 2 seconds
3. Save the change

OBSERVED RESULT
The grid still updates at the default period 0,5 sec.

EXPECTED RESULT
The grid updates at the set period 2 secs.

SOFTWARE/OS VERSIONS
Linux/KDE Plasma: ArchLinux 6.11.1-arch1-1 (64-bit)
KDE Plasma Version: 6.1.5
KDE Frameworks Version: 6.6.0
Qt Version: 6.7.3
Graphics Platform: Wayland
Comment 1 Paul Worrall 2024-10-03 07:49:42 UTC
I can reproduce this bug, setting to CONFIRMED

ADDITIONAL INFORMATION:
The minimum time setting works OK for all other display styles except Process Table and Applications Table

SOFTWARE/OS VERSIONS:
Operating System: Arch Linux 
KDE Plasma Version: 6.1.90
KDE Frameworks Version: 6.6.0
Qt Version: 6.8.0
Graphics Platform: Wayland
Comment 2 Bug Janitor Service 2024-10-11 11:56:31 UTC
A possibly relevant merge request was started @ https://invent.kde.org/plasma/libksysguard/-/merge_requests/374
Comment 3 Arjen Hiemstra 2024-10-23 13:17:23 UTC
Git commit 9d143812093ed419ae2bb4b3dd84b8944e492f3f by Arjen Hiemstra.
Committed on 23/10/2024 at 13:13.
Pushed by ahiemstra into branch 'master'.

faces/grid: Specify updateRateLimit for FaceLoader

To ensure we pass the updateRateLimit from the parent controller on to
the internal per-face controller.

M  +1    -0    faces/facepackages/facegrid/contents/ui/FaceControl.qml

https://invent.kde.org/plasma/libksysguard/-/commit/9d143812093ed419ae2bb4b3dd84b8944e492f3f
Comment 4 Arjen Hiemstra 2024-10-23 13:37:51 UTC
Git commit 362832026668fff13e2d194d1c02e85c7fcde8f3 by Arjen Hiemstra.
Committed on 23/10/2024 at 13:33.
Pushed by ahiemstra into branch 'Plasma/6.2'.

faces/grid: Specify updateRateLimit for FaceLoader

To ensure we pass the updateRateLimit from the parent controller on to
the internal per-face controller.

M  +1    -0    faces/facepackages/facegrid/contents/ui/FaceControl.qml

https://invent.kde.org/plasma/libksysguard/-/commit/362832026668fff13e2d194d1c02e85c7fcde8f3