gsass1 / NTop

💻 htop-like system-monitor for Windows with Vi-keybindings.
GNU General Public License v3.0
1.21k stars 69 forks source link

Header with blue background jumps every update time #30

Open KorneevVitaly opened 4 years ago

KorneevVitaly commented 4 years ago

When ntop is working, the header disapeares and reappeares each update time (regardless if it's in cmd.exe or ConEmu), which looks like everything jumps up and down. I installed it from chocolatey. Here the screenshots: 3 2

gsass1 commented 4 years ago

What kind of terminal is that?

KorneevVitaly commented 4 years ago

What kind of terminal is that?

I'm using ConEmu.

Tried open ntop in regular cmd.exe and it has the same behaviour. Also, my system is Windows 8.1.

gsass1 commented 4 years ago

Well this statement https://github.com/gsass1/NTop/blob/master/ntop.c#L1767 (DISABLE_NEWLINE_AUTO_RETURN) is what should prevent the buffer from scrolling on the last line. IDK why that doesn't work on Windows 8.1 :/

KorneevVitaly commented 4 years ago

I tieed build it with build.bat and I've got an error with "error C2275: WORD:invalid use of this type" Screenshot_1

And in minwindef.h(159) it's: typedef unsigned short WORD;

Krikonildo commented 3 years ago

Work as intended with Windows 10. Same problem with Windows 7. Maybe a command line option to disable the header ?

Alex313031 commented 3 years ago

Can confirm on windows x64 7, server 2011, 8, 8.1. Havent tested in ten but probably works as Krikonildo said. I love all the "tops" for Unix/Linux i.e. top, htop, nvtop, atop. There is no other program like this and I really like it just disappointing me with the jittering up and down it can make it hard to read. Also verified on 32 bit windows 7 in command.com.

Alex313031 commented 3 years ago

Update @Krikonildo @KorneevVitaly @gsass1 WORKS as intended IF you set compatibility mode to Windows XP SP3 in the executables properties. This works for 7, and 8.1 but strangely not 8. Of course my 8 is in a VM whereas 7 and 8.1 are real machines but idk if that makes a difference. Compiling works with VS 2017 AND 2015 with platform update on win 7. Same issues and same resolution no matter what is used to compile.

Alex313031 commented 3 years ago

I have it in a directory in my path, and if I launch it from cmd (not double clicking ntop), this only resolves it IF you keep the default ~80X24 Prompt size

greatquux commented 3 years ago

Actually, while playing with your workaround, I found another one which doesn't require changing terminal size or compatibility options - just press / as if you were going to do a search, but then don't do anything, and the display calms down and keeps updating. Annoying for sure, in that if you want to re-sort you have to backspace first, but it does work. This display has been steady for many minutes now:

image

-- Michael Russo | Systems Engineer | PaperSolve, Inc. office (718) 477 8906 | direct (332) 228 1817

Randomly generated quote of the last 5 minutes:

Jim, this is Janelle. I'm flying tonight, so I can't make our date, and I gotta find a safe place for Daffy. He loves you, Jim! It's only two days, and you'll see. Great Danes are no problem!

          -- "The Rockford Files"

From: Alexander David Frick Sent: Wednesday, October 28, 2020 11:53 PM To: gsass1/NTop NTop@noreply.github.com Cc: Mike Russo mike@PaperSolve.com; Manual manual@noreply.github.com Subject: Re: [gsass1/NTop] Header with blue background jumps every update time (#30)

I have it in a directory in my path, and if I launch it from cmd (not double clicking ntop), this only resolves it IF you keep the default ~80X24 Prompt size

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHubhttps://github.com/gsass1/NTop/issues/30#issuecomment-718342746, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AAMUJBROUSUFLUHOEOIJI5LSNDRJVANCNFSM4OJP2NNA.

--

Michael Russo | Systems Engineer | PaperSolve, Inc. office (718) 477 8906 | direct (332) 228 1817

Your random quote for today:

Krikonildo commented 3 years ago

@Alex313031 Didn't work for me ( Windows 7 x64 ). Neither compatibility mode or cmd. Works without DISABLE_NEWLINE_AUTO_RETURN ( Linux / MinGW ). Even on Vista x86 recompiling with GetTickCount.

Krikonildo commented 3 years ago

@greatquux Works for me too. But it crashes if I try to select a process with the directional arrows.

gsass1 commented 3 years ago

Hello

I just pushed another commit that maybe fixes this. I would be grateful if someone could just compile the latest commit and tell me if it works since I don't have a Windows 7 VM atm.

greatquux commented 3 years ago

I compiled the latest version and it seems to work great on my Windows Server 2012 R2 now where it was previously jumping around (unless I pressed '/').

Alex313031 commented 3 years ago

@greatquux @Krikonildo The / works better because you can resize, also works on win 8. I had an SSD fail and had to recover alot of files and reinstall win 7 on a new ssd. Don't have alot of stuff set up including VS, so is there a way you could share your latest compiled version so I can try. Also cudos to @gsass1 for bein so quick about everything.

Alex313031 commented 3 years ago

@Krikonildo Your'e still using Vista in 2020!! JK I use win xp and win 98 for old stuff, and have VMs of vista. Anyway, VS 2013 and higher requires >Win 7 and 2015 and higher requires > Win 7 with 2013 Platform Update https://www.microsoft.com/en-us/download/details.aspx?id=36805 So how in the world did you compile it on that.

greatquux commented 3 years ago

@Alex313031 I will send you a link to the last compiled version I have, it was while doing some troubleshooting for the issue 34 so it leaves some stuff in the background when you quit and if it cannot find a username it displays it as "default" but otherwise works fine.

Alex313031 commented 3 years ago

@greatquux Thnx

Alex313031 commented 3 years ago

@greatquux Crashes on launch on Win 7 sp1 ultimate x64

greatquux commented 3 years ago

Unfortunately I don't have that exact version of OS to test against, but I am able to run the same binary on a Windows Server 2008 R2 SP1 (which is always x64):

image

--

Michael Russo | Systems Engineer | PaperSolve, Inc. office (718) 477 8906 | direct (332) 228 1817

Your random quote for today: There's no easy quick way out, we're gonna have to live through our whole lives, win, lose, or draw. -- Walt Kelly

-----Original Message----- From: Alexander David Frick notifications@github.com<mailto:Alexander%20David%20Frick%20%3cnotifications@github.com%3e> Reply-To: gsass1/NTop reply@reply.github.com<mailto:gsass1/NTop%20%3creply@reply.github.com%3e> To: gsass1/NTop NTop@noreply.github.com<mailto:gsass1/NTop%20%3cNTop@noreply.github.com%3e> Cc: greatquux mike@papersolve.com<mailto:greatquux%20%3cmike@papersolve.com%3e>, Mention mention@noreply.github.com<mailto:Mention%20%3cmention@noreply.github.com%3e> Subject: Re: [gsass1/NTop] Header with blue background jumps every update time (#30) Date: Sun, 29 Nov 2020 16:55:25 -0800

@greatquuxhttps://github.com/greatquux Crashes on launch on Win 7 sp1 ultimate x64

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHubhttps://github.com/gsass1/NTop/issues/30#issuecomment-735489750, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AAMUJBVYFBM2MH24ONW4FMLSSLUP3ANCNFSM4OJP2NNA.

iqlat commented 3 years ago

Same issue, title flickering on latest build (from scoop/git repo); tested as standalone .exe and in Cmder; on Windows 7 x64 Pro; it stops only when window looses focus => gif_link; the workaround with "/" crashes app.

iqlat commented 2 years ago

upd: nov, 2021 -> this issue seems to be fixed in the latest source code; you can compile it by your own or download the compiled one from google drive.

gsass1 commented 2 years ago

You can also fetch the latest build on CircleCI, for example: https://app.circleci.com/pipelines/github/gsass1/NTop/5/workflows/62fb589a-2e71-4e21-903d-6d5b9c2d8c17/jobs/5/artifacts