stefanhaustein / TerminalImageViewer

Small C++ program to display images in a (modern) terminal using RGB ANSI codes and unicode block graphics characters
Other
1.56k stars 111 forks source link

Does not play well with mosh #97

Closed karelbilek closed 3 years ago

karelbilek commented 3 years ago

I am trying to use tiv together with mosh - a "mobile shell", that is doing some magic to work properly in low latency environments

https://mosh.org

(basically they have a completely different protocol than SSH, that synchronizes the screens)

Now I guess it is not playing nice with unicode magic that you do here. When I try tiv with mosh, all pictures looks garbled and ugly.

karelbilek commented 3 years ago

Ah it seems like mosh is unmaintained, based on the comments here

https://github.com/mobile-shell/mosh/issues/1115

aaronliu0130 commented 3 years ago

Could you provide some screenshots? Thanks.

karelbilek commented 3 years ago
Screen Shot 2021-04-12 at 13 32 50

This is "Lenna".

karelbilek commented 3 years ago

This is on the same machine, both server and client, just through normal ssh.

Screen Shot 2021-04-12 at 13 34 34
karelbilek commented 3 years ago

Note, I think it might be something on mosh side, it is not maintained anymore and it does some complex magic to make the screen synchronization quick.

PythonNut commented 3 years ago

I imagine this is because mosh doesn't support 24bit colors (at least not using version <= 1.3.2).

karelbilek commented 3 years ago

Ah.

I used whatever version was in homebrew. ...yes, 1.3.2.

Oh, that is the latest release.

stefanhaustein commented 3 years ago

Did you try the 256 or 16 color options? Perhaps mosh doesn't support 24 bit color?

Am Mo., 12. Apr. 2021 um 08:40 Uhr schrieb Karel Bilek < @.***>:

Note, I think it might be something on mosh side, it is not maintained anymore and it does some complex magic to make the screen synchronization quick.

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/stefanhaustein/TerminalImageViewer/issues/97#issuecomment-817529913, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABAVPT4NPSFWHUQSQUODX4LTIKIWPANCNFSM4Z7JBPAQ .

karelbilek commented 3 years ago

Ahhh 256 works. That's it, thanks!

karelbilek commented 3 years ago

Closing this, as it is "solved". (good enough.)