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

Option to remove empty core lines doesn't work #267

Closed
sbutcher opened this issue Dec 9, 2016 · 6 comments
Closed

Option to remove empty core lines doesn't work #267

sbutcher opened this issue Dec 9, 2016 · 6 comments
Assignees

Comments

@sbutcher
Copy link

sbutcher commented Dec 9, 2016

Hello
Running

qtop -b sge -FGwrT

doesn't remove empty core lines on Centos6 and Centos7 machines i've tested on. see attached image for example.

selection_291

@sfranky
Copy link
Collaborator

sfranky commented Dec 9, 2016

Hi, sorry, there is a lack of documentation here. Empty core lines refer to lines that are entirely filled with #s, in essence cores that do not really exist for the displayed machines. In the screenshot, you also have unused cores (on the left and right hand side of the screen), so it wouldn't make sense to just remove them, it would draw an unreliable image of the cluster. You can always transpose the matrix (and maybe your monitor, if possible!!) ;)

@sbutcher
Copy link
Author

sbutcher commented Dec 9, 2016

ah, thanks for clarification, not a bug then.

@sfranky sfranky closed this as completed Dec 9, 2016
@sfranky sfranky reopened this Dec 9, 2016
@sfranky
Copy link
Collaborator

sfranky commented Dec 9, 2016

@sbutcher
stay tuned, I'm adding a double -rr to also remove ####_______ this kind of lines. That real estate is valuable!!

@sfranky
Copy link
Collaborator

sfranky commented Dec 10, 2016

@sbutcher if you want to try it out,, check PR #271
It does what you expected, plus you should be able to toggle between those states, showing all cores, hiding corelines full of '#'s and hiding corelines with '#'s and '_'s.

@sfranky
Copy link
Collaborator

sfranky commented Dec 12, 2016

fixed in latest develop branch

@sfranky sfranky closed this as completed Dec 12, 2016
@sbutcher
Copy link
Author

works for me , thanks guys!

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

No branches or pull requests

2 participants