orblazer / plasma-applet-resources-monitor

Plasmoid for monitoring CPU, memory, network traffic, GPUs and disks IO.
https://www.pling.com/p/2143899/
GNU General Public License v3.0
57 stars 6 forks source link

[BUG] memory monitor doesn't work at 2.7 #27

Closed THMonster closed 2 years ago

THMonster commented 2 years ago

Describe the bug

To Reproduce

Steps to reproduce the behavior:

  1. Go to '...'
  2. Click on '....'
  3. See error

Expected behavior

Screenshots image

Versions (please complete the following information):

orblazer commented 2 years ago

Hello, that is strange :confused:

Could you try if with 2.6.3 that is work ? If not could you add print(column, data(indexVar, Sensors.SensorDataModel.Name), value) before this line ? https://github.com/orblazer/plasma-applet-resources-monitor/blob/0ab30cb1ddf0fa98e9cdf1f09c743c0721affd33/package/contents/ui/components/SensorGraph.qml#L34

Thanks

THMonster commented 2 years ago

Hello, that is strange confused

Could you try if with 2.6.3 that is work ? If not could you add print(column, data(indexVar, Sensors.SensorDataModel.Name), value) before this line ?

https://github.com/orblazer/plasma-applet-resources-monitor/blob/0ab30cb1ddf0fa98e9cdf1f09c743c0721affd33/package/contents/ui/components/SensorGraph.qml#L34

Thanks

Added. But how can i get the output info?

THMonster commented 2 years ago

oh i got it. image

THMonster commented 2 years ago

I tried downgrade. 2.6.3 doesn't work. 2.5.3 works.

orblazer commented 2 years ago

I tried downgrade. 2.6.3 doesn't work. 2.5.3 works.

That is normal because i have change the system in 2.6.0.

oh i got it. image

That values look like percent and only CPU...

And i can't reproduce the issue at all, so for that i need some debug :

  1. (optional) you can use dev env : https://github.com/orblazer/plasma-applet-resources-monitor#contributedebug for easiest debug.
  2. Add print(column, sensors[column]) (my previous line is not great because its follow system language...) before this line https://github.com/orblazer/plasma-applet-resources-monitor/blob/0ab30cb1ddf0fa98e9cdf1f09c743c0721affd33/package/contents/ui/components/SensorGraph.qml#L26
  3. Add print(" ", value) before this line https://github.com/orblazer/plasma-applet-resources-monitor/blob/0ab30cb1ddf0fa98e9cdf1f09c743c0721affd33/package/contents/ui/components/SensorGraph.qml#L31
  4. Print your config by adding Component.onCompleted: print(JSON.stringify(plasmoid.configuration)) before this line https://github.com/orblazer/plasma-applet-resources-monitor/blob/74a40cfeaad6ded973de605dde97974464708540/package/contents/ui/main.qml#L75

Sorry to ask you to debug that for me :disappointed: and thanks by advance.

TiGR commented 2 years ago

Version 2.7.0:

image

I've tried removing the widget and adding it again with no prevail. Memory indicator never works.

orblazer commented 2 years ago

Hello, Maybe i have found the solution, could you try with that unreleased version ? resourcesMonitor-fork-unreleased.zip

And if that not fix, could you check on System Monitor (from plasma-systemmonitor) for see if you have the value ?

Thanks.

TiGR commented 2 years ago

Okay, memory in percentage now works:

image

But not when it is not in percentage:

image

(I have no swap, just in case)

orblazer commented 2 years ago

What is your step to reproduce this case ?

TiGR commented 2 years ago
  1. Just add the widget, and it looks like this:

image

  1. Then change to display in percentage:

image

  1. Then uncheck the memory in percentage option:

image

orblazer commented 2 years ago

I can't reproduce that case :disappointed: but i have found the reason, the max value is not defined for unknown reason.

I have an question, you have some swap define on the system or is disabled ?

If you have swap, could you add debugs :

Thanks by advance

TiGR commented 2 years ago
Jan 15 09:39:45 tigr-TUXEDO plasmashell[1488233]: qml: 33001844736
Jan 15 09:39:45 tigr-TUXEDO plasmashell[1488233]: qml: 0
TiGR commented 2 years ago

@orblazer Thank you. With 2.7.1 finally everything works as intended.