Xpra: Ticket #645: please show actual encoding in "Session Info/Connection"

I upgraded session 0.13.9 --> 0.14.0 using command xpra upgrade :44. Then I repeated upgrade command few more times. After that I've noticed that when I change encoding in client (0.14.0) some encodings appears to be chosen but actually they're not -- for example I selected PNG/greyscale but still could see colours; with RGB encoding there were some visual artefacts still visible as it appears that H.264 were actually used... While perhaps it were a glitch that I can't reproduce after restarting session on server I'd like to suggest showing actual encoding in use under "Session Info --> Connection" where I can see "rencode + lz4 (level 1)" but not the codec from "Encoding" menu. Thanks.

Wed, 20 Aug 2014 06:00:15 GMT - onlyjob:

Something is wrong with encoding selection on 0.14.0. Even when I attach with "--encoding=png/L" Libreoffice Calc show colour background in spradsheet as well as low-quality regions so encoding used is definitely not PNG... (RGB is also affected). Seems to be regression from 0.13.9...

Wed, 20 Aug 2014 07:02:23 GMT - Antoine Martin: owner, status, version, summary changed

What does this have to do with upgrading?

FWIW: the "actual encoding" has been meaningless for some time, we choose the best encoding for the pixels we need to encode. The encoding selected in the tray is the "current encoding". For more info see #419

That said, there are some constraints: we should not be sending anything but png/L when it is selected (because it would look awful otherwise), rgb is also sticky, we try to stick to png / jpeg / webp when those are selected, but we may still use rgb instead when the compressed (or even uncompressed) raw data is smaller than wrapped in the container, etc..

I've seen the problem with png/L - will investigate.

Wed, 20 Aug 2014 07:17:36 GMT - onlyjob:

Replying to totaam:

What does this have to do with upgrading?

Probably nothing -- I was merely describing what was happening when I got this problem... When reported I had too little experience with 0.14 to say if it (not) related to upgrade... Will try to report less noise in the future... Thanks.

Wed, 20 Aug 2014 09:00:25 GMT - Antoine Martin: owner, status changed

Does r7360 fix all the problems? (will backport)

Wed, 20 Aug 2014 10:47:28 GMT - onlyjob:

Yes, definitely fixes, thank you.

I still think it might be nice to show actual combination of primary/secondary codecs but it's up to you to decide whether it's worth implementing...

Wed, 20 Aug 2014 11:02:53 GMT - Antoine Martin: status changed; resolution set

Backport in r7362 - thanks! Closing.

I still think it might be nice to show actual combination of primary/secondary codecs

The session info dialog already shows all the codecs supported by either end (in the "Features" tab), then when using a video mode you can end up using up to 6 different encodings at the same time.. Which ones get used will vary a lot. I really can't think of way of presenting this sanely.

Sat, 23 Jan 2021 05:01:49 GMT - migration script:

this ticket has been moved to: https://github.com/Xpra-org/xpra/issues/645