htop icon indicating copy to clipboard operation
htop copied to clipboard

`DISK READ`/`DISK WRITE` column changes

Open BenBE opened this issue 2 years ago • 4 comments

The columns DISK READ and DISK WRITE currently show as %5.2f %sB/s like e.g. 0.00 B/s. This makes those columns quite long compared to other columns. Also showing whole bytes with two significant digits is somewhat unnecessary precision.

What about shortening the column labels to RDISK/WDISK and the display of the actual column value as follows:

  1. Values below 1KiB/s are shaded
  2. Values below 1KiB/s displayed in bytes
  3. Starting with 1KiB/s values receive 3-4 significant digits and the usual unit suffices.
  4. Shading of MiB/s, GiB/s and TiB/s done similar to normal byte counts.

Currently each of the two column takes up 10 characters, when similar columns like RCHAR/WCHAR (Number of bytes read/written) use 5 chars.

BenBE avatar May 03 '22 07:05 BenBE

Hey, would love to take this up if there hasn't been any work done on it yet as this looks like a good starting point to get into this repo.

Has there been any further discussion on this on any other PRs?

NikhilReddy-2 avatar Jun 06 '22 08:06 NikhilReddy-2

There has not been any further discussion I'm aware of. Usually issues are heavily cross-referenced, thus if nothing shows here, it's unlikely there was essential discussion.

Thus feel free to take this one up. There's some documentation on the code style available in the docs folder. Also don't hesitate to ask if you've got any questions.

BenBE avatar Jun 06 '22 08:06 BenBE

Do you still want to work on this @NikhilReddy-2? If not I would gladly give it a try. From what I saw on the main branch, the proposed changes weren't done yet.

marcluque avatar Oct 10 '22 15:10 marcluque