Finishing Plasma 6 System Monitor's priority setter

Hello everyone, I hope you’re all doing great. :slight_smile:

I find it annoying that “ksysguard” didn’t make the transition to Plasma 6, since "plasma-systemmonitor” is clearly not complete…

I really need the option to change process’ priority (aka. niceness), so I volunteer to help on finishing it.

But first, I would like to have some elaboration on this line:

  • Does it means the feature is unfinished (or maybe forgotten)?
  • Or, does it means it’s planed for removal (I sure hope not)… :expressionless:

Thanks for your help.

2 Likes

Your contribution would be appreciated. The stub you found in the code was intended to be used once the feature was eventually re-implemented.

1 Like

Perfect, I’m working on it, BTW.

I may submit a MR this week-end.

2 Likes

Hi. :slight_smile:

I’ve finished the first part of it. But now, to go further, I might have to modify a bit libksysguard too.

Please, don’t hesitate to make feedbacks, or even to make adjustments to my draft MR.

I tried to stay faithful to how it worked in Plasma 5’s KSysGuard:

  • Everything is implemented in proper QML, with little to no C++ modifications.
  • To set the priority, I call the setPriority functions and stuff from libksysguard.
  • Thus, it works, but currently I wasn’t able to query a process’ niceness from it’s PID, using only QML…
  • I have kept the F8 keyboard shortcut to open the priority dialog (currently hardcoded, like the others).

Here is what the “Set priority” dialog looks like. The icons will vary according to the icon theme (I’m not on Breeze). I tried to stay close to it’s Plasma 5 counterpart:

7 Likes

That’s awesome! Thanks for working on this. I left a minor comment on the MR, but overall this is looking really nice and I’m excited to see your improvement get merged.

1 Like

Hi everyone, I’ve created a little merge request on libksysguard, in continuation. :slight_smile:

Basically, I needed some kind of method to retrieve the niceness, schedulers & ionice values of a process from its PID (in QML).

I would query these on the dialog opening, to set the initial values of the sliders, radio buttons, etc.

Please let me know what you think, then I would implement this final touch in the system monitor’s merge request. :stuck_out_tongue:

Hello, I hope you’re doing good. :slight_smile:

I haven’t got any feedback on the merge request on libksysguard for two weeks… no pressure of course, I was just curious, and think it needed a bit of a bump… :stuck_out_tongue:

1 Like