Double-digit thread support

[xxxserxxx]

i don't believe there's an issue related to this, but definitely close and link if there is. 😅

i have a TR1920X, which has 24-threads. running gotop -p, will list all the threads, however, i typically can't see more than the first 12 or so unless i fully maximize the window.

*would it be possible to adaptively place the cpus? (meaning, if the window which lists the cpus is able to "detect" an overflow, it repaints as a column?)

i know that i would take a temporal update hit to have that – meaning if it reduces the update frequency, that's definitely an acceptable trade-off, imo. (or maybe we can pass a flag?)

Original ticket cjbassi#158
Original submitter @ionlights

[jmuchovej]

@xxxserxxx are you planning to work on this? 🤔 (just got a notif, so i was curious.)

(i’m definitely willing to help test.)

[xxxserxxx]

@ionlights Yes, I am. Here's a bit of copypasta, since this question is (understandably) coming up frequently:

Caleb is not maintaining gotop any longer, but also doesn't want anybody else maintaining his repository, nor is he comfortable with moving issues. I'm continuing development and maintenance of gotop, so I've forked and copied the tickets so that I can keep working on them.

See this issue for more information.

[xxxserxxx]

@ionlights there have been new releases, as well.

[xxxserxxx]

@ionlights I think I understand what you mean. You want to see a layout like this:

 +- CPU Usage -----------------------+
 |                                   |
 | CPU0 10%   CPU5 10%               |
 | CPU1  5%   CPU6  5%               |
 | CPU2 10%   CPU7 10%               |
 | CPU3  5%                          |
 | CPU4 10%                          |
 +-----------------------------------+

Or were you thinking of something different?

In the meantime, can you check out a recent release and see if the ability to define your own layouts helps? If you download a binary, download this file and save it to pwd directory, or copy it to $XDG_CONFIG_HOME/gotop/, and then run gotop -l procs. The layout file has to be either in the same directory you're in, or in the gotop config directory.

The layouts/ directory has more examples, and the README explains the syntax. With this, you can make the procs widget as tall as the console, and maybe see all of your CPUs. It's not a full solution, because Ryzen ThreadRipper exists. But maybe it'll be a sufficient work-around until this ticket is closed

[jmuchovej]

So, that layout is correct. I think something that might be useful (though probably harder) would be to adapt based on the screen-size. e.g.

+- CPU Usage -------------------------+
|                                     |
| CPU00 10%   CPU03  5%   CPU06 10%   |
| CPU01  5%   CPU04 10%   CPU07  5%   |
| CPU02 10%   CPU05 10%   CPU08 10%   |
|                                     |
+-------------------------------------+

Though I'd probably classify this as a "stretch" implementation.

I'll also give your proposed interim solution a look in a few minutes as well.

Thanks for doing this! 😃

[jmuchovej]

I was finally able to update to 3.3.1 on Arch Linux (linux-lts), with kernel 5.4.21. (Need to use linux-lts because of zfs,(

[jmuchovej]

So, I'm not sure that this is supported? This machine has 24-threads. Screenshots both in regular window and tmux. This is with v3.5.1 on Arch Linux.

[xxxserxxx]

@ionlights hi John. This isn't in the 3.5 branch -- the one you're using for NVidia. I try to keep the "Milestone" field (on the left here in Github) accurate; as you can see, this was fixed in the 4.0 branch, which hasn't yet been released. If you try that branch and it still doesn't work, do let me know.

Sorry for the confusion.

[jmuchovej]

Ahh. Okay. Wasn’t sure since I saw it closed. Will checkout v4 in a few days and lyk if anything breaks. 🙂