mirror of
https://github.com/XuehaiPan/nvitop.git
synced 2026-05-15 14:15:55 -06:00
[GH-ISSUE #155] [Feature Request] Add %GMBW in default monitor mode #97
Labels
No labels
api
bug
bug
cli / tui
dependencies
documentation
documentation
documentation
duplicate
enhancement
exporter
invalid
pull-request
pynvml
question
question
upstream
wontfix
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference: github-starred/nvitop#97
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Originally created by @XiaomoWu on GitHub (Apr 6, 2025).
Original GitHub issue: https://github.com/XuehaiPan/nvitop/issues/155
Originally assigned to: @XuehaiPan on GitHub.
Required prerequisites
Motivation
Thank you for building and maintaining such a great tool!
This issue is related to #117, but I believe it proposes a more practical solution.
Currently, memory bandwidth utilization (%GMBW) is only visible in the process detail view (i.e., after pressing Enter on a process), as shown below:
However, in the default monitor mode, %GMBW is not displayed:
Would it be possible to include %GMBW in the default monitor view, or at least provide a toggle to enable it? This metric is particularly helpful when diagnosing whether a workload is memory-bound or compute-bound.
Thanks again for your excellent work on this tool!
Solution
No response
Alternatives
No response
Additional context
No response
@XuehaiPan commented on GitHub (Apr 6, 2025):
Thanks for the suggestion. I think it is worth adding it.
One blocker is whether we'd still want to support minimal rendering width with 79 characters.
@XiaomoWu commented on GitHub (Apr 6, 2025):
@XuehaiPan Do you mean that by adding %GMBW, the total width of the columns—GPU, PID, USER, GPU-MEM, %SM, %CPU, %MEM, and TIME—would exceed 79 characters (excluding the COMMAND column)?
@XuehaiPan commented on GitHub (Apr 6, 2025):
Including the COMMAND column.
I tested it with a prototype. There is enough space to show it. I will open a PR to resolve this.