Skip to content
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

sort by clicking improvements and bug #1578

Closed
orena1 opened this issue Dec 27, 2019 · 3 comments · Fixed by #1759
Closed

sort by clicking improvements and bug #1578

orena1 opened this issue Dec 27, 2019 · 3 comments · Fixed by #1759
Assignees
Labels
Milestone

Comments

@orena1
Copy link
Contributor

orena1 commented Dec 27, 2019

Description

I found two issues when I used glances (great app!!!) on windows and opening the monitor on chrome.

  1. Clicking on CPU% will sort the processes by CPU usage, but in most monitoring software another click sort it by reverse (So that the lowest CPU usage will be at the top).
  2. Clicking on IOR/s or IOW/s does not sort the processes correctly.

Versions

  • Glances & psutil (glances -V):
    Glances v3.1.3 with psutil v5.6.3
  • Operating System (lsb_release -a):
(base) C:\Users\oren>lsb_release -a
'lsb_release' is not recognized as an internal or external command,
operable program or batch file.

Just kidding ;-)
Windows 10 Pro 1909

@nicolargo
Copy link
Owner

Hi @orena1

  1. It is not a bug, it is a feature. Glances layout is made to show relevant information. Who want to see the lowest CPU usage ?).

  2. Same issue on Linux OS. The IOR/r and IOR/w sort did not work on the Web UI. @notFloran can you have a look on this issue ?

@nicolargo nicolargo added the bug label Dec 28, 2019
@nicolargo nicolargo added this to the Glances 3.1.4 milestone Dec 28, 2019
@nicolargo nicolargo modified the milestones: Glances 3.1.4, Next releases Mar 3, 2020
@nicolargo nicolargo modified the milestones: Glances 3.1.5, 3.1.6 Aug 19, 2020
@nicolargo
Copy link
Owner

@notFloran Can you have a quick look on the second issue concerning the sort of IOR/r and IOR/w ?

@notFloran
Copy link
Collaborator

@nicolargo @orena1 I made a PR, can you try it ?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants