-
-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
Screen frequently flickers when outputting to local display #2490
Comments
Thanks for the issue report (and the video). I did not reproduce on my side. Can you please try with the latest Glances version (3.4.0.2) ? |
@nicolargo I installed 3.4.0.2 via PyPi/pipx (Glances v3.4.0.2 with PsUtil v5.9.5) and was able to reproduce the issue. There was also a version 3.4.0.3 on PyPi (Glances v3.4.0.3 with PsUtil v5.9.5) that I tried and it had the same issue. But I found something else: I could reproduce the issue on another computer (running the same version of Debian), but not within an X session (even when maximizing the window). So maybe it has something to do with window management / the terminal emulator. |
Issue should be solved in the development branch. I switch back to self.term_window.erase() and add a new hot key (Ctrl-R) to force a self.term_window.clear() (related to #2211). |
Thanks! Would you be willing to accept a PR in a few weeks that periodically calls self.term_window.clear()? Maybe as a hard-coded proportion of the updates? I'm thinking every 150 updates, which would be every 5 minutes at 2 seconds between updates. I can also try to add a config option and cli argument to change that multiple. For anyone else experiencing #2211 again, a hacky workaround is to periodically echo control-r into the stdin of glances. For example |
@ATLief yes a PR with a configurable self.term_window.clear() call every n seconds can make sense (by default n should be unset or set to 0 to avoid call to self.term_window.clear(). |
For the record, I also experience this on Arch with alacritty. On every refresh, the entire terminal window very briefly flickers, most likely by being empty for a fraction of a second/one frame. When the window is very small, it becomes less noticable or not at all, presumably because refilling the few characters of the window happens fast enough that a display refresh doesn't happen between clearing the window and refilling it. Other tools like |
I'm seeing this issue with Windows Terminal 1.18.3181.0 under Debian Bookworm via WSL, Glances 3.3.1.1 deb from the Debian repos, when connecting to a different machine on my network. SSHing into that machine and running |
I confirm this is happening for me, too, with |
The same. Blinking every second. Glances v3.3.1.1 with PsUtil v5.9.4 |
Should be solved in Glances 4.0.0. |
Can confirm, thanks for the fix! |
@nicolargo Thanks again for fixing this! Since it unfortunately re-introduces #2211, would you mind re-opening that issue to help with future search-ability? |
Is it reintroduce in version 4.0.7 ? Can you also try with another terminal ? |
Describe the bug
When outputting to a local display, sections of the screen frequently flicker as the information updates. Reverting commit ed4cc5e fixes this particular issue, but re-introduces the bug in #2211. Increasing the delay between updates increases the delay between flickers. This issue doesn't happen when displaying the console on another computer over SSH.
To Reproduce
kmscon
tested)Expected behavior
The text on the screen changes without large portions of the text briefly disappearing.
Screenshots
Med_VS.mov
Environement
Additional context
Please ignore the empty space on the right side of the display; I limit the number of TTY columns to hide the VIRT and RES columns in Glances, but the problem persists even without that limitation.
If this issue can't be fixed, maybe you could add an option to use the new or old clearing behaviour, or an option to clear the screen at a slower interval (maybe every 30 update intervals, or every minute).
Thank you for fixing the other bug, and everything else you do for this awesome project!
The text was updated successfully, but these errors were encountered: