andrewsali commented on pull request #322: URL: https://github.com/apache/guacamole-server/pull/322#issuecomment-755181167
Sorry, this PR is not meant to be a definitive code-base change at all - it is just a suggestion for testing and if others can reproduce the improved speed, then formalize the aspects requested in the code-review. For encoding speed / bandwidth, what we tested was to take a screenshot PNG (Full HD screen) and see how much time it is to convert to webp using the cwebp utility. Using m=0, q=30, converting 50 times ``` real 0m2.549s user 0m2.151s sys 0m0.404s ``` Output file size: 220K Using m=2, q=30, converting 50 times: ``` real 0m13.037s user 0m12.575s sys 0m0.466s ``` Output file size: 112K The original PNG file size was also 220K. Using m=0, q=90, converting 50 times: ``` real 0m3.201s user 0m2.945s sys 0m0.261s ``` Output file size: 186K. Using m=2, q=90, converting 50 times: ``` real 0m15.867s user 0m15.531s sys 0m0.333s ``` Output file size: 103K. So compared to the m=2 setting, the bandwidth requirement seems about double, but encode speeds are about x5. Given the improvements in bandwidth, it might be worthwhile to go for faster compression. In our subjective experience this has helped substantially. But it would be good for some tests in other deployments to see if it is really a good way to go. ---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: [email protected]
