Bug 494019

Summary:	Grid graph minimum time between updates setting does not work
Product:	[Applications] plasma-systemmonitor	Reporter:	Caio <caiohideo>
Component:	general	Assignee:	KSysGuard Developers <ksysguard-bugs>
Status:	RESOLVED FIXED
Severity:	minor	CC:	ahiemstra, nate, p.r.worrall, plasma-bugs-null
Priority:	NOR
Version First Reported In:	6.1.5
Target Milestone:	---
Platform:	Arch Linux
OS:	Linux
Latest Commit:	https://invent.kde.org/plasma/libksysguard/-/commit/362832026668fff13e2d194d1c02e85c7fcde8f3	Version Fixed/Implemented In:	6.2.3
Sentry Crash Report:

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